[Fedora-directory-commits] ldapserver/ldap/servers/slapd/back-ldbm index.c, 1.6, 1.7

Richard Allen Megginson (rmeggins) fedora-directory-commits at redhat.com
Fri Aug 12 19:07:40 UTC 2005

Author: rmeggins

Update of /cvs/dirsec/ldapserver/ldap/servers/slapd/back-ldbm
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv13302

Modified Files:
Log Message:
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

Index: index.c
RCS file: /cvs/dirsec/ldapserver/ldap/servers/slapd/back-ldbm/index.c,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -r1.6 -r1.7
--- index.c	12 Aug 2005 05:49:55 -0000	1.6
+++ index.c	12 Aug 2005 19:07:37 -0000	1.7
@@ -550,10 +550,12 @@
             /* 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 */

More information about the Fedora-directory-commits mailing list