mirror/librenms-librenms
1
0
Fork 0
mirror of https://github.com/librenms/librenms.git synced 2024-10-07 16:52:45 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix error when ldap_search returns false (#14199)

Browse Source
This commit is contained in:
Jellyfrog
2022-08-14 16:14:12 +02:00
committed by GitHub
parent d3d2aa92e1
commit 2ef1c3670b
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
LibreNMS/Authentication/ActiveDirectoryAuthorizer.php
View File

@@ -158,10 +158,13 @@ class ActiveDirectoryAuthorizer extends AuthorizerBase
$this->userFilter($username),
$attributes
);
$entries = ldap_get_entries($connection, $search);
if ($entries['count']) {
return $this->getUseridFromSid($this->sidFromLdap($entries[0]['objectsid'][0]));
if ($search !== false) {
$entries = ldap_get_entries($connection, $search);
if ($entries !== false && $entries['count']) {
return $this->getUseridFromSid($this->sidFromLdap($entries[0]['objectsid'][0]));
}
}
return -1;