Browse Source

Bug(s) fixed: 165827
Bug Description: Daily Acceptance: Directory Install failed to register Directory server as a Red Hat server (81)
Reviewed by: Nathan (Thanks!)
Fix Description: The index code, in the replace case, was not checking to see if there were
actually any values to delete before attempting to delete them. This fix just
checks to see if there are any values to delete.
Platforms tested: RHEL3
Flag Day: no
Doc impact: no
QA impact: should be covered by regular nightly and manual testing with the new indexing tests
New Tests integrated into TET: none

Rich Megginson 20 years ago
parent
commit
fa9db453fb
1 changed files with 3 additions and 1 deletions
  1. 3 1
      ldap/servers/slapd/back-ldbm/index.c

+ 3 - 1
ldap/servers/slapd/back-ldbm/index.c

@@ -550,10 +550,12 @@ index_add_mods(
             }
 
             /* We need to first remove the old values from the 
-             * index. */
+             * index, if any. */
+            if (deleted_valueArray) {
                 index_addordel_values_sv( be, mods[i]->mod_type,
                                           deleted_valueArray, evals, id, 
                                           flags, txn );
+            }
 
             /* Free valuearray */
             slapi_valueset_free(mod_vals);