From 0eeb4d2ef865cefc0101fe554dacae00ce55ab4d Mon Sep 17 00:00:00 2001
From: Falk Stern <fstern@goodgamestudios.com>
Date: Tue, 10 Nov 2015 10:03:47 +0100
Subject: [PATCH] Fixed a bug from scrutinizer

---
 html/includes/authentication/active_directory.inc.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/html/includes/authentication/active_directory.inc.php b/html/includes/authentication/active_directory.inc.php
index 316017855..5aea2e11b 100644
--- a/html/includes/authentication/active_directory.inc.php
+++ b/html/includes/authentication/active_directory.inc.php
@@ -127,7 +127,7 @@ function get_userid($username) {
     global $config, $ds;
 
     $attributes = array('objectsid');
-    $result = ldap_search($ds, $config['auth_ad_base_dn'],
+    $search = ldap_search($ds, $config['auth_ad_base_dn'],
                           "(samaccountname={$username})", $attributes);    
     $entries = ldap_get_entries($ds, $search);
 
@@ -148,6 +148,7 @@ function deluser($username) {
 function get_userlist() {
     global $config, $ds;
     $userlist = array();
+    $userhash = array();
 
     $ldap_groups = get_group_list();