Make sure ldap filter is not empty when searching, use (objectclass=*) by default

ldap_search() throws errors on empty filter as find out in
http://trac.roundcube.net/ticket/1490269. As this was a configuration
issue we can simply prevent that and provide real and working default.

1 files changed, 6 insertions, 2 deletions

diff --git a/lib/Net/LDAP3.php b/lib/Net/LDAP3.php
index a825c3a..02b77ef 100644
--- a/lib/Net/LDAP3.php
+++ b/lib/Net/LDAP3.php
@@ -1552,12 +1552,16 @@ class Net_LDAP3
             return false;
         }
 
-        $this->_debug("C: Search base dn: [$base_dn] scope [$scope] with filter [$filter]");
-
         // make sure attributes list is not empty
         if (empty($attrs)) {
             $attrs = array('dn');
         }
+        // make sure filter is not empty
+        if (empty($filter)) {
+            $filter = '(objectclass=*)';
+        }
+
+        $this->_debug("C: Search base dn: [$base_dn] scope [$scope] with filter [$filter]");
 
         $function = self::scope_to_function($scope, $ns_function);