diff options
author | Rich Megginson <rmeggins@redhat.com> | 2005-08-12 19:07:37 +0000 |
---|---|---|
committer | Rich Megginson <rmeggins@redhat.com> | 2005-08-12 19:07:37 +0000 |
commit | fa9db453fba0104250e088a721faf39751fead32 (patch) | |
tree | e2181966a058c0585588bceb02e5c00f7f10c676 /ldap/servers/slapd/back-ldbm/index.c | |
parent | 2aa495ed6362c1d38b513e35b3c3d2bb1cad89e0 (diff) | |
download | ds-fa9db453fba0104250e088a721faf39751fead32.tar.gz ds-fa9db453fba0104250e088a721faf39751fead32.tar.xz ds-fa9db453fba0104250e088a721faf39751fead32.zip |
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
Diffstat (limited to 'ldap/servers/slapd/back-ldbm/index.c')
-rw-r--r-- | ldap/servers/slapd/back-ldbm/index.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/ldap/servers/slapd/back-ldbm/index.c b/ldap/servers/slapd/back-ldbm/index.c index 6e31042d..b5637032 100644 --- a/ldap/servers/slapd/back-ldbm/index.c +++ b/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); |