From: Quanah Gibson-Mount <quanah@openldap.org>
Date: Thu, 30 Apr 2015 03:30:17 +0000 (-0500)
Subject: ITS#8120 Move final CHECK_CSN block to before the mutex is unlocked.
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=2c34d2fda687c490d78af3fd23bf3d245d239ffe;p=openldap

ITS#8120 Move final CHECK_CSN block to before the mutex is unlocked.
---

diff --git a/servers/slapd/syncrepl.c b/servers/slapd/syncrepl.c
index 04e3862847..255ca5bcc7 100644
--- a/servers/slapd/syncrepl.c
+++ b/servers/slapd/syncrepl.c
@@ -3926,12 +3926,6 @@ syncrepl_updateCookie(
 		ch_free( sc.sids );
 		ber_bvarray_free( sc.ctxcsn );
 	}
-	ldap_pvt_thread_mutex_unlock( &si->si_cookieState->cs_mutex );
-
-	op->o_bd = be;
-	op->o_tmpfree( op->o_csn.bv_val, op->o_tmpmemctx );
-	BER_BVZERO( &op->o_csn );
-	if ( mod.sml_next ) slap_mods_free( mod.sml_next, 1 );
 
 #ifdef CHECK_CSN
 	for ( i=0; i<si->si_cookieState->cs_num; i++ ) {
@@ -3939,6 +3933,13 @@ syncrepl_updateCookie(
 	}
 #endif
 
+	ldap_pvt_thread_mutex_unlock( &si->si_cookieState->cs_mutex );
+
+	op->o_bd = be;
+	op->o_tmpfree( op->o_csn.bv_val, op->o_tmpmemctx );
+	BER_BVZERO( &op->o_csn );
+	if ( mod.sml_next ) slap_mods_free( mod.sml_next, 1 );
+
 	return rc;
 }