From: Howard Chu <hyc@openldap.org>
Date: Tue, 17 Feb 2009 22:08:00 +0000 (+0000)
Subject: ITS#5954 from jclarke@linagora.com: fix be_syncinfo order
X-Git-Tag: ACLCHECK_0~815
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=c50729c4fd2ac4b3ab23d1503a99cd2af850aa3a;p=openldap

ITS#5954 from jclarke@linagora.com: fix be_syncinfo order
---

diff --git a/servers/slapd/syncrepl.c b/servers/slapd/syncrepl.c
index d99af0ef2e..b6686958f3 100644
--- a/servers/slapd/syncrepl.c
+++ b/servers/slapd/syncrepl.c
@@ -4325,13 +4325,22 @@ add_syncrepl(
 			BER_BVISNULL( &si->si_bindconf.sb_uri ) ?
 			"(null)" : si->si_bindconf.sb_uri.bv_val, 0, 0 );
 		if ( c->be->be_syncinfo ) {
+			syncinfo_t *sip;
+
 			si->si_cookieState = c->be->be_syncinfo->si_cookieState;
+
+			// add new syncrepl to end of list (same order as when deleting)
+			for ( sip = c->be->be_syncinfo; sip->si_next; sip = sip->si_next );
+			sip->si_next = si;
 		} else {
 			si->si_cookieState = ch_calloc( 1, sizeof( cookie_state ));
 			ldap_pvt_thread_mutex_init( &si->si_cookieState->cs_mutex );
+
+			c->be->be_syncinfo = si;
 		}
-		si->si_next = c->be->be_syncinfo;
-		c->be->be_syncinfo = si;
+
+		si->si_next = NULL;
+
 		return 0;
 	}
 }