From: Howard Chu <hyc@openldap.org>
Date: Thu, 13 Nov 2008 22:46:49 +0000 (+0000)
Subject: ITS#5809 renames are not exclusive from modifies
X-Git-Tag: ACLCHECK_0~1111
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=51c87bb9c738866b82f5516e6fd064ca15e2c7b2;p=openldap

ITS#5809 renames are not exclusive from modifies
---

diff --git a/servers/slapd/syncrepl.c b/servers/slapd/syncrepl.c
index 476cdae553..96b5754e87 100644
--- a/servers/slapd/syncrepl.c
+++ b/servers/slapd/syncrepl.c
@@ -2273,7 +2273,7 @@ retry_add:;
 					"syncrepl_entry: %s be_modrdn (%d)\n", 
 					si->si_ridtxt, rc, 0 );
 			op->o_bd = be;
-			goto done;
+			/* Renamed entries may still have other mods so just fallthru */
 		}
 		if ( dni.mods ) {
 			op->o_tag = LDAP_REQ_MODIFY;
@@ -3049,8 +3049,10 @@ dn_callback(
 					{
 						dni->delOldRDN = 1;
 					}
-					/* OK, this was just a modDN, we're done */
-					return LDAP_SUCCESS;
+					/* A ModDN has happened, but other changes may have
+					 * occurred before we picked it up. So fallthru to
+					 * regular Modify processing.
+					 */
 				}
 
 				modtail = &dni->mods;