Bug on AuthLdap class

[cristos77]

hello

i found a bug on the authldap class.
In getAllUsers function, a variable is not set correctly:

https://github.com/glpi-project/glpi/blob/9.2/bugfixes/inc/authldap.class.php#L1767-L1768

$userfound=false;
if (!empty($ldap_users[$user[$field_for_db]])
        || ($userfound = self::dnExistsInLdap($user_infos, $user['user_dn']))) {
   ....

the "userfound" variable can be not set if the first test is true. The second test is not checked, so the userfound variable stay at 'false'.

i suggest to set variable during declaration:

$userfound=self::dnExistsInLdap($user_infos, $user['user_dn']);
if (!empty($ldap_users[$user[$field_for_db]]) {
   ....

best regards
cristo