Make LDAP_ALLOW_NULL_SEARCH_BASE the last resort. (from -devel)

diff --git a/servers/slapd/backend.c b/servers/slapd/backend.c
index c2f4656d741eb13bb3e936e8b760b541a5b7106d..9bc09c94b21c3bc109a16b9fff6a3da510327a0b 100644 (file)
--- a/servers/slapd/backend.c
+++ b/servers/slapd/backend.c
@@ -128,18 +128,6 @@ select_backend( char * dn )
                for ( j = 0; backends[i].be_suffix != NULL &&
                    backends[i].be_suffix[j] != NULL; j++ )
                {
-#ifdef LDAP_ALLOW_NULL_SEARCH_BASE
-                       /* Add greg@greg.rim.or.jp
-                        * It's quick hack for cheap client
-                        * Some browser offer a NULL base at ldap_search
-                        */
-                       if(dnlen == 0) {
-                               Debug( LDAP_DEBUG_TRACE,
-                                       "select_backend: use default backend\n", 0, 0, 0 );
-                               return (&backends[i]);
-                       }
-#endif /* LDAP_ALLOW_NULL_SEARCH_BASE */
-
                        len = strlen( backends[i].be_suffix[j] );
 
                        if ( len > dnlen ) {
@@ -173,6 +161,19 @@ select_backend( char * dn )
                 }
         }
 
+#ifdef LDAP_ALLOW_NULL_SEARCH_BASE
+       /* Add greg@greg.rim.or.jp
+        * It's quick hack for cheap client
+        * Some browser offer a NULL base at ldap_search
+        *
+        * Should only be used as a last resort. -Kdz
+        */
+       if(dnlen == 0) {
+               Debug( LDAP_DEBUG_TRACE,
+                       "select_backend: use default backend\n", 0, 0, 0 );
+               return( &backends[0] );
+       }
+#endif /* LDAP_ALLOW_NULL_SEARCH_BASE */
 
        return( NULL );
 }