From 652089dd78f007bf00539abaee621c875dd49d4c Mon Sep 17 00:00:00 2001 From: evbevz Date: Thu, 15 Sep 2016 10:26:38 +0300 Subject: [PATCH] LDAP fix: handle legal boolean return result from searchUsers() --- src/opnsense/mvc/app/library/OPNsense/Auth/LDAP.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/opnsense/mvc/app/library/OPNsense/Auth/LDAP.php b/src/opnsense/mvc/app/library/OPNsense/Auth/LDAP.php index 098d8b098..c0267887c 100644 --- a/src/opnsense/mvc/app/library/OPNsense/Auth/LDAP.php +++ b/src/opnsense/mvc/app/library/OPNsense/Auth/LDAP.php @@ -333,7 +333,7 @@ class LDAP implements IAuthConnector $ldap_is_connected = $this->connect($this->ldapBindURL, $this->ldapBindDN, $this->ldapBindPassword); if ($ldap_is_connected) { $result = $this->searchUsers($username, $this->ldapAttributeUser, $this->ldapExtendedQuery); - if (count($result) > 0) { + if ($result !== false && count($result) > 0) { $ldap_is_connected = $this->connect($this->ldapBindURL, $result[0]['dn'], $password); return $ldap_is_connected; }