[Date Prev][Date Next] [Chronological] [Thread] [Top]

slapindex check for be functions (ITS#673)



slapindex checks to see if the backend db has the functions necessary for
reindexing. But the check should fail if ANY are missing, not just the
special case where ALL are missing. The logic is wrong. Right?

-Mark Adamson
 Carnegie Mellon




--- .orig/servers/slapd/tools/slapindex.c       Wed Jul 26 11:23:07 2000
+++ servers/slapd/tools/slapindex.c     Fri Aug 18 09:57:46 2000
@@ -24,10 +24,10 @@
 
        slap_tool_init( "slapindex", SLAPINDEX, argc, argv );
 
-       if( !be->be_entry_open &&
-               !be->be_entry_close &&
-               !be->be_entry_first &&
-               !be->be_entry_next &&
+       if( !be->be_entry_open ||
+               !be->be_entry_close ||
+               !be->be_entry_first ||
+               !be->be_entry_next  ||
                !be->be_entry_reindex )
        {
                fprintf( stderr, "%s: database doesn't support necessary
operations.\n",