]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/syncrepl.c
Fix prev commit, cleanup fakeroot entry before freeing
[openldap] / servers / slapd / syncrepl.c
index bf2f436b1af980f76f197a5319e0c051540b6ab4..4d9f28b3b73e750c6b93b98a5c6cadfb4b00e828 100644 (file)
@@ -2,7 +2,7 @@
 /* $OpenLDAP$ */
 /* This work is part of OpenLDAP Software <http://www.openldap.org/>.
  *
- * Copyright 2003-2006 The OpenLDAP Foundation.
+ * Copyright 2003-2007 The OpenLDAP Foundation.
  * Portions Copyright 2003 by IBM Corporation.
  * Portions Copyright 2003 by Howard Chu, Symas Corporation.
  * All rights reserved.
@@ -37,6 +37,14 @@ struct nonpresent_entry {
        LDAP_LIST_ENTRY(nonpresent_entry) npe_link;
 };
 
+typedef struct cookie_state {
+       ldap_pvt_thread_mutex_t cs_mutex;
+       int     cs_num;
+       int cs_age;
+       struct berval *cs_vals;
+       int *cs_sids;
+} cookie_state;
+       
 #define        SYNCDATA_DEFAULT        0       /* entries are plain LDAP entries */
 #define        SYNCDATA_ACCESSLOG      1       /* entries are accesslog format */
 #define        SYNCDATA_CHANGELOG      2       /* entries are changelog format */
@@ -50,9 +58,12 @@ struct nonpresent_entry {
 #define RETRYNUM_FINITE(n)     ((n) > RETRYNUM_FOREVER)        /* not forever */
 
 typedef struct syncinfo_s {
+       struct syncinfo_s       *si_next;
        struct slap_backend_db *si_be;
+       struct slap_backend_db *si_wbe;
        struct re_s                     *si_re;
-       long                            si_rid;
+       int                                     si_rid;
+       char                            si_ridtxt[8];
        slap_bindconf           si_bindconf;
        struct berval           si_base;
        struct berval           si_logbase;
@@ -75,6 +86,8 @@ typedef struct syncinfo_s {
        int                                     *si_retrynum_init;
        int                                     *si_retrynum;
        struct sync_cookie      si_syncCookie;
+       cookie_state            *si_cookieState;
+       int                                     si_cookieAge;
        int                                     si_manageDSAit;
        int                                     si_slimit;
        int                                     si_tlimit;
@@ -83,9 +96,6 @@ typedef struct syncinfo_s {
        int                                     si_syncdata;
        int                                     si_logstate;
        int                                     si_conn_setup;
-#ifdef HAVE_TLS
-       int                                     si_check_tls;
-#endif
        Avlnode                         *si_presentlist;
        LDAP                            *si_ld;
        LDAP_LIST_HEAD(np, nonpresent_entry) si_nonpresentlist;
@@ -94,7 +104,7 @@ typedef struct syncinfo_s {
 
 static int syncuuid_cmp( const void *, const void * );
 static void avl_ber_bvfree( void * );
-static void syncrepl_del_nonpresent( Operation *, syncinfo_t *, BerVarray );
+static void syncrepl_del_nonpresent( Operation *, syncinfo_t *, BerVarray, struct berval * );
 static int syncrepl_message_to_op(
                                        syncinfo_t *, Operation *, LDAPMessage * );
 static int syncrepl_message_to_entry(
@@ -102,9 +112,7 @@ static int syncrepl_message_to_entry(
                                        Modifications **, Entry **, int );
 static int syncrepl_entry(
                                        syncinfo_t *, Operation*, Entry*,
-                                       Modifications**,int, struct berval*,
-                                       struct sync_cookie *,
-                                       struct berval * );
+                                       Modifications**,int, struct berval* );
 static int syncrepl_updateCookie(
                                        syncinfo_t *, Operation *, struct berval *,
                                        struct sync_cookie * );
@@ -140,7 +148,7 @@ init_syncrepl(syncinfo_t *si)
                if ( si->si_allattrs ) {
                        i = 0;
                        while ( attrs[i] ) {
-                               if ( !is_at_operational( at_find( attrs[i] ))) {
+                               if ( !is_at_operational( at_find( attrs[i] ) ) ) {
                                        for ( j = i; attrs[j] != NULL; j++ ) {
                                                if ( j == i )
                                                        ch_free( attrs[i] );
@@ -157,7 +165,7 @@ init_syncrepl(syncinfo_t *si)
                } else if ( si->si_allopattrs ) {
                        i = 0;
                        while ( attrs[i] ) {
-                               if ( is_at_operational( at_find( attrs[i] ))) {
+                               if ( is_at_operational( at_find( attrs[i] ) ) ) {
                                        for ( j = i; attrs[j] != NULL; j++ ) {
                                                if ( j == i )
                                                        ch_free( attrs[i] );
@@ -175,7 +183,7 @@ init_syncrepl(syncinfo_t *si)
                for ( i = 0; sync_descs[i] != NULL; i++ ) {
                        j = 0;
                        while ( attrs[j] ) {
-                               if ( !strcmp( attrs[j], sync_descs[i]->ad_cname.bv_val )) {
+                               if ( !strcmp( attrs[j], sync_descs[i]->ad_cname.bv_val ) ) {
                                        for ( k = j; attrs[k] != NULL; k++ ) {
                                                if ( k == j )
                                                        ch_free( attrs[k] );
@@ -190,9 +198,9 @@ init_syncrepl(syncinfo_t *si)
                for ( n = 0; attrs[ n ] != NULL; n++ ) /* empty */;
 
                if ( si->si_allopattrs ) {
-                       attrs = ( char ** ) ch_realloc( attrs, (n + 2)*sizeof( char * ));
+                       attrs = ( char ** ) ch_realloc( attrs, (n + 2)*sizeof( char * ) );
                } else {
-                       attrs = ( char ** ) ch_realloc( attrs, (n + 4)*sizeof( char * ));
+                       attrs = ( char ** ) ch_realloc( attrs, (n + 4)*sizeof( char * ) );
                }
 
                if ( attrs == NULL ) {
@@ -241,7 +249,7 @@ init_syncrepl(syncinfo_t *si)
                for ( i = 0; sync_descs[i] != NULL; i++ ) {
                        j = 0;
                        while ( exattrs[j] != NULL ) {
-                               if ( !strcmp( exattrs[j], sync_descs[i]->ad_cname.bv_val )) {
+                               if ( !strcmp( exattrs[j], sync_descs[i]->ad_cname.bv_val ) ) {
                                        ch_free( exattrs[j] );
                                        for ( k = j; exattrs[k] != NULL; k++ ) {
                                                exattrs[k] = exattrs[k+1];
@@ -259,7 +267,7 @@ init_syncrepl(syncinfo_t *si)
                                        k = 0;
                                        while ( oc->soc_required[k] ) {
                                                if ( !strcmp( exattrs[i],
-                                                        oc->soc_required[k]->sat_cname.bv_val )) {
+                                                        oc->soc_required[k]->sat_cname.bv_val ) ) {
                                                        ch_free( exattrs[i] );
                                                        for ( l = i; exattrs[l]; l++ ) {
                                                                exattrs[l] = exattrs[l+1];
@@ -275,7 +283,7 @@ init_syncrepl(syncinfo_t *si)
                for ( i = 0; exattrs[i] != NULL; i++ ) ;
 
                if ( i != n )
-                       exattrs = (char **) ch_realloc( exattrs, (i + 1)*sizeof(char *));
+                       exattrs = (char **) ch_realloc( exattrs, (i + 1)*sizeof(char *) );
        }
 
        si->si_exattrs = exattrs;       
@@ -333,8 +341,9 @@ ldap_sync_search(
        /* If we're using a log but we have no state, then fallback to
         * normal mode for a full refresh.
         */
-       if ( si->si_syncdata && BER_BVISEMPTY( &si->si_syncCookie.ctxcsn ))
+       if ( si->si_syncdata && !si->si_syncCookie.numcsns ) {
                si->si_logstate = SYNCLOG_FALLBACK;
+       }
 
        /* Use the log parameters if we're in log mode */
        if ( si->si_syncdata && si->si_logstate == SYNCLOG_LOGGING ) {
@@ -381,7 +390,7 @@ ldap_sync_search(
                        abs(si->si_type), rhint );
        }
 
-       if ( (rc = ber_flatten2( ber, &c[0].ldctl_value, 0 )) == LBER_ERROR ) {
+       if ( (rc = ber_flatten2( ber, &c[0].ldctl_value, 0 ) ) == LBER_ERROR ) {
                ber_free_buf( ber );
                return rc;
        }
@@ -419,126 +428,15 @@ do_syncrep1(
        int cmdline_cookie_found = 0;
 
        struct sync_cookie      *sc = NULL;
-       struct berval   *psub;
 #ifdef HAVE_TLS
        void    *ssl;
 #endif
 
-       psub = &si->si_be->be_nsuffix[0];
-
-       /* Init connection to master */
-       rc = ldap_initialize( &si->si_ld, si->si_bindconf.sb_uri.bv_val );
+       rc = slap_client_connect( &si->si_ld, &si->si_bindconf );
        if ( rc != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_ANY,
-                       "do_syncrep1: ldap_initialize failed (%s)\n",
-                       si->si_bindconf.sb_uri.bv_val, 0, 0 );
-               return rc;
-       }
-
-       op->o_protocol = LDAP_VERSION3;
-       ldap_set_option( si->si_ld, LDAP_OPT_PROTOCOL_VERSION,
-               (const void *)&op->o_protocol );
-
-#ifdef HAVE_TLS
-       if ( si->si_check_tls ) {
-               si->si_check_tls = 0;
-               rc = bindconf_tls_set( &si->si_bindconf, si->si_ld );
-       } else if ( si->si_bindconf.sb_tls_ctx ) {
-               rc = ldap_set_option( si->si_ld, LDAP_OPT_X_TLS_CTX,
-                       si->si_bindconf.sb_tls_ctx );
-       }
-       if ( rc ) {
-               Debug( LDAP_DEBUG_ANY,
-                       "do_syncrep1: TLS context initialization failed\n", 0, 0, 0 );
-               return rc;
-       }
-#endif
-
-       /* Bind to master */
-
-       if ( si->si_bindconf.sb_tls ) {
-               rc = ldap_start_tls_s( si->si_ld, NULL, NULL );
-               if( rc != LDAP_SUCCESS ) {
-                       Debug( LDAP_DEBUG_ANY,
-                               "%s: ldap_start_tls failed (%d)\n",
-                               si->si_bindconf.sb_tls == SB_TLS_CRITICAL ? "Error" : "Warning",
-                               rc, 0 );
-                       if( si->si_bindconf.sb_tls == SB_TLS_CRITICAL ) goto done;
-               }
-       }
-
-       if ( si->si_bindconf.sb_method == LDAP_AUTH_SASL ) {
-#ifdef HAVE_CYRUS_SASL
-               void *defaults;
-
-               if ( si->si_bindconf.sb_secprops != NULL ) {
-                       rc = ldap_set_option( si->si_ld,
-                               LDAP_OPT_X_SASL_SECPROPS, si->si_bindconf.sb_secprops);
-
-                       if( rc != LDAP_OPT_SUCCESS ) {
-                               Debug( LDAP_DEBUG_ANY, "Error: ldap_set_option "
-                                       "(%s,SECPROPS,\"%s\") failed!\n",
-                                       si->si_bindconf.sb_uri.bv_val, si->si_bindconf.sb_secprops, 0 );
-                               goto done;
-                       }
-               }
-
-               defaults = lutil_sasl_defaults( si->si_ld,
-                       si->si_bindconf.sb_saslmech.bv_val,
-                       si->si_bindconf.sb_realm.bv_val,
-                       si->si_bindconf.sb_authcId.bv_val,
-                       si->si_bindconf.sb_cred.bv_val,
-                       si->si_bindconf.sb_authzId.bv_val );
-
-               rc = ldap_sasl_interactive_bind_s( si->si_ld,
-                               si->si_bindconf.sb_binddn.bv_val,
-                               si->si_bindconf.sb_saslmech.bv_val,
-                               NULL, NULL,
-                               LDAP_SASL_QUIET,
-                               lutil_sasl_interact,
-                               defaults );
-
-               lutil_sasl_freedefs( defaults );
-
-               /* FIXME: different error behaviors according to
-                *      1) return code
-                *      2) on err policy : exit, retry, backoff ...
-                */
-               if ( rc != LDAP_SUCCESS ) {
-                       static struct berval bv_GSSAPI = BER_BVC( "GSSAPI" );
-
-                       Debug( LDAP_DEBUG_ANY, "do_syncrep1: "
-                               "ldap_sasl_interactive_bind_s failed (%d)\n",
-                               rc, 0, 0 );
-
-                       /* FIXME (see above comment) */
-                       /* if Kerberos credentials cache is not active, retry */
-                       if ( ber_bvcmp( &si->si_bindconf.sb_saslmech, &bv_GSSAPI ) == 0 &&
-                               rc == LDAP_LOCAL_ERROR )
-                       {
-                               rc = LDAP_SERVER_DOWN;
-                       }
-
-                       goto done;
-               }
-#else /* HAVE_CYRUS_SASL */
-               /* Should never get here, we trapped this at config time */
-               assert(0);
-               Debug( LDAP_DEBUG_SYNC, "not compiled with SASL support\n", 0, 0, 0 );
-               rc = LDAP_OTHER;
                goto done;
-#endif
-
-       } else if ( si->si_bindconf.sb_method == LDAP_AUTH_SIMPLE ) {
-               rc = ldap_sasl_bind_s( si->si_ld,
-                       si->si_bindconf.sb_binddn.bv_val, LDAP_SASL_SIMPLE,
-                       &si->si_bindconf.sb_cred, NULL, NULL, NULL );
-               if ( rc != LDAP_SUCCESS ) {
-                       Debug( LDAP_DEBUG_ANY, "do_syncrep1: "
-                               "ldap_sasl_bind_s failed (%d)\n", rc, 0, 0 );
-                       goto done;
-               }
        }
+       op->o_protocol = LDAP_VERSION3;
 
        /* Set SSF to strongest of TLS, SASL SSFs */
        op->o_sasl_ssf = 0;
@@ -555,25 +453,15 @@ do_syncrep1(
        op->o_ssf = ( op->o_sasl_ssf > op->o_tls_ssf )
                ?  op->o_sasl_ssf : op->o_tls_ssf;
 
-
-       if ( BER_BVISNULL( &si->si_syncCookie.octet_str )) {
-               /* get contextCSN shadow replica from database */
-               BerVarray csn = NULL;
-
-               assert( si->si_rid < 1000 );
-               op->o_req_ndn = op->o_bd->be_nsuffix[0];
-               op->o_req_dn = op->o_req_ndn;
-
-               /* try to read stored contextCSN */
-               backend_attribute( op, NULL, &op->o_req_ndn,
-                       slap_schema.si_ad_contextCSN, &csn, ACL_READ );
-               if ( csn ) {
-                       ch_free( si->si_syncCookie.ctxcsn.bv_val );
-                       ber_dupbv( &si->si_syncCookie.ctxcsn, csn );
-                       ber_bvarray_free_x( csn, op->o_tmpmemctx );
-               }
+       /* We've just started up, or the remote server hasn't sent us
+        * any meaningful state.
+        */
+       if ( BER_BVISNULL( &si->si_syncCookie.octet_str ) ) {
+               int i;
 
                si->si_syncCookie.rid = si->si_rid;
+               si->si_syncCookie.sid = SLAP_SINGLE_SHADOW( si->si_be ) ? -1 :
+                       slap_serverID;
 
                LDAP_STAILQ_FOREACH( sc, &slap_sync_cookie, sc_next ) {
                        if ( si->si_rid == sc->rid ) {
@@ -589,25 +477,148 @@ do_syncrep1(
 
                        /* ctxcsn wasn't parsed yet, do it now */
                        slap_parse_sync_cookie( sc, op->o_tmpmemctx );
-                       if ( BER_BVISNULL( &sc->ctxcsn ) ) {
-                               /* if cmdline cookie does not have ctxcsn */
-                               /* component, set it to an initial value */
-                               slap_init_sync_cookie_ctxcsn( sc );
-                       }
                        slap_sync_cookie_free( &si->si_syncCookie, 0 );
                        slap_dup_sync_cookie( &si->si_syncCookie, sc );
                        slap_sync_cookie_free( sc, 1 );
+               } else {
+                       ldap_pvt_thread_mutex_lock( &si->si_cookieState->cs_mutex );
+                       if ( !si->si_cookieState->cs_num ) {
+                               /* get contextCSN shadow replica from database */
+                               BerVarray csn = NULL;
+                               void *ctx = op->o_tmpmemctx;
+
+                               op->o_req_ndn = op->o_bd->be_nsuffix[0];
+                               op->o_req_dn = op->o_req_ndn;
+
+                               /* try to read stored contextCSN */
+                               op->o_tmpmemctx = NULL;
+                               backend_attribute( op, NULL, &op->o_req_ndn,
+                                       slap_schema.si_ad_contextCSN, &csn, ACL_READ );
+                               op->o_tmpmemctx = ctx;
+                               if ( csn ) {
+                                       si->si_cookieState->cs_vals = csn;
+                                       for (i=0; !BER_BVISNULL( &csn[i] ); i++);
+                                       si->si_cookieState->cs_num = i;
+                                       si->si_cookieState->cs_sids = slap_parse_csn_sids( csn, i );
+                               }
+                       }
+                       if ( si->si_cookieState->cs_num ) {
+                               ber_bvarray_free( si->si_syncCookie.ctxcsn );
+                               if ( ber_bvarray_dup_x( &si->si_syncCookie.ctxcsn,
+                                       si->si_cookieState->cs_vals, NULL )) {
+                                       rc = LDAP_NO_MEMORY;
+                                       goto done;
+                               }
+                               si->si_syncCookie.numcsns = si->si_cookieState->cs_num;
+                               si->si_syncCookie.sids = ch_malloc( si->si_cookieState->cs_num *
+                                       sizeof(int) );
+                               for ( i=0; i<si->si_syncCookie.numcsns; i++ )
+                                       si->si_syncCookie.sids[i] = si->si_cookieState->cs_sids[i];
+                       }
+                       ldap_pvt_thread_mutex_unlock( &si->si_cookieState->cs_mutex );
                }
 
                slap_compose_sync_cookie( NULL, &si->si_syncCookie.octet_str,
-                       &si->si_syncCookie.ctxcsn, si->si_syncCookie.rid );
+                       si->si_syncCookie.ctxcsn, si->si_syncCookie.rid,
+                       si->si_syncCookie.sid );
+       } else {
+               AttributeName at[2];
+               Attribute a = { slap_schema.si_ad_contextCSN };
+               Entry e = {0};
+               SlapReply rs = {0};
+               int i, j, changed = 0;
+
+               /* Look for contextCSN from syncprov overlay. If
+                * there's no overlay, this will be a no-op. That means
+                * this is a pure consumer, so local changes will not be
+                * allowed, and all changes will already be reflected in
+                * the cookieState.
+                */
+               e.e_attrs = &a;
+               e.e_name = si->si_wbe->be_suffix[0];
+               e.e_nname = si->si_wbe->be_nsuffix[0];
+               rs.sr_entry = &e;
+               rs.sr_flags = REP_ENTRY_MODIFIABLE;
+               at[0].an_name = a.a_desc->ad_cname;
+               at[0].an_desc = a.a_desc;
+               BER_BVZERO( &at[1].an_name );
+
+               ldap_pvt_thread_mutex_lock( &si->si_cookieState->cs_mutex );
+               rc = backend_operational( op, &rs );
+               if ( rc == LDAP_SUCCESS && a.a_vals ) {
+                       int num;
+                       for (i=0; !BER_BVISNULL( &a.a_vals[i] ); i++) ;
+                       num = i;
+                       /* check for differences */
+                       if ( num != si->si_cookieState->cs_num ) {
+                               changed = 1;
+                       } else {
+                               for ( i=0; i<num; i++ ) {
+                                       if ( ber_bvcmp( &a.a_vals[i],
+                                               &si->si_cookieState->cs_vals[i] )) {
+                                               changed =1;
+                                               break;
+                                       }
+                               }
+                       }
+                       if ( changed ) {
+                               ber_bvarray_free( si->si_cookieState->cs_vals );
+                               ch_free( si->si_cookieState->cs_sids );
+                               si->si_cookieState->cs_num = num;
+                               si->si_cookieState->cs_vals = a.a_vals;
+                               si->si_cookieState->cs_sids = slap_parse_csn_sids( a.a_vals,
+                                       num );
+                               si->si_cookieState->cs_age++;
+                       } else {
+                               ber_bvarray_free( a.a_vals );
+                       }
+                       changed = 0;
+               }
+               /* See if the cookieState has changed due to anything outside
+                * this particular consumer. That includes other consumers in
+                * the same context, or local changes detected above.
+                */
+               if ( si->si_cookieState->cs_num > 1 && si->si_cookieAge !=
+                       si->si_cookieState->cs_age ) {
+
+                       for (i=0; !BER_BVISNULL( &si->si_syncCookie.ctxcsn[i] ); i++) {
+                               /* bogus, just dup everything */
+                               if ( si->si_syncCookie.sids[i] == -1 ) {
+                                       ber_bvarray_free( si->si_syncCookie.ctxcsn );
+                                       ber_bvarray_dup_x( &si->si_syncCookie.ctxcsn,
+                                               si->si_cookieState->cs_vals, NULL );
+                                       changed = 1;
+                                       break;
+                               }
+                               for (j=0; j<si->si_cookieState->cs_num; j++) {
+                                       if ( si->si_syncCookie.sids[i] !=
+                                               si->si_cookieState->cs_sids[j] )
+                                               continue;
+                                       if ( bvmatch( &si->si_syncCookie.ctxcsn[i],
+                                               &si->si_cookieState->cs_vals[j] ))
+                                               break;
+                                       ber_bvreplace( &si->si_syncCookie.ctxcsn[i],
+                                               &si->si_cookieState->cs_vals[j] );
+                                       changed = 1;
+                                       break;
+                               }
+                       }
+                       if ( changed ) {
+                               ch_free( si->si_syncCookie.octet_str.bv_val );
+                               slap_compose_sync_cookie( NULL, &si->si_syncCookie.octet_str,
+                                       si->si_syncCookie.ctxcsn, si->si_syncCookie.rid,
+                                       SLAP_SINGLE_SHADOW( si->si_be ) ? -1 : slap_serverID );
+                       }
+               }
+               ldap_pvt_thread_mutex_unlock( &si->si_cookieState->cs_mutex );
        }
 
        rc = ldap_sync_search( si, op->o_tmpmemctx );
 
        if( rc != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_ANY, "do_syncrep1: "
-                       "ldap_search_ext: %s (%d)\n", ldap_err2string( rc ), rc, 0 );
+               Debug( LDAP_DEBUG_ANY, "do_syncrep1: %s "
+                       "ldap_search_ext: %s (%d)\n",
+                       si->si_ridtxt, ldap_err2string( rc ), rc );
        }
 
 done:
@@ -621,6 +632,37 @@ done:
        return rc;
 }
 
+static int
+compare_csns( struct sync_cookie *sc1, struct sync_cookie *sc2, int *which )
+{
+       int i, j, match = 0;
+       const char *text;
+
+       *which = 0;
+
+       if ( sc1->numcsns < sc2->numcsns ) {
+               *which = sc1->numcsns;
+               return -1;
+       }
+
+       for (i=0; !BER_BVISNULL( &sc1->ctxcsn[i] ); i++) {
+               for (j=0; !BER_BVISNULL( &sc2->ctxcsn[j] ); j++) {
+                       if ( sc1->sids[i] != sc2->sids[j] )
+                               continue;
+                       value_match( &match, slap_schema.si_ad_entryCSN,
+                               slap_schema.si_ad_entryCSN->ad_type->sat_ordering,
+                               SLAP_MR_VALUE_OF_ATTRIBUTE_SYNTAX,
+                               &sc1->ctxcsn[i], &sc2->ctxcsn[i], &text );
+                       if ( match < 0 ) {
+                               *which = j;
+                               return match;
+                       }
+                       break;
+               }
+       }
+       return match;
+}
+
 static int
 do_syncrep2(
        Operation *op,
@@ -642,18 +684,17 @@ do_syncrep2(
 
        int             syncstate;
        struct berval   syncUUID = BER_BVNULL;
-       struct sync_cookie      syncCookie = { BER_BVNULL };
-       struct sync_cookie      syncCookie_req = { BER_BVNULL };
+       struct sync_cookie      syncCookie = { NULL };
+       struct sync_cookie      syncCookie_req = { NULL };
        struct berval           cookie = BER_BVNULL;
 
-       int     rc, err, i;
+       int     rc, err;
        ber_len_t       len;
 
        struct berval   *psub;
        Modifications   *modlist = NULL;
 
-       const char              *text;
-       int                             match;
+       int                             match, m;
 
        struct timeval *tout_p = NULL;
        struct timeval tout = { 0, 0 };
@@ -671,7 +712,7 @@ do_syncrep2(
        ber_init2( ber, NULL, LBER_USE_DER );
        ber_set_option( ber, LBER_OPT_BER_MEMCTX, &op->o_tmpmemctx );
 
-       Debug( LDAP_DEBUG_TRACE, "=>do_syncrep2\n", 0, 0, 0 );
+       Debug( LDAP_DEBUG_TRACE, "=>do_syncrep2 %s\n", si->si_ridtxt, 0, 0 );
 
        psub = &si->si_be->be_nsuffix[0];
 
@@ -683,8 +724,8 @@ do_syncrep2(
                tout_p = NULL;
        }
 
-       while (( rc = ldap_result( si->si_ld, LDAP_RES_ANY, LDAP_MSG_ONE,
-               tout_p, &res )) > 0 )
+       while ( ( rc = ldap_result( si->si_ld, LDAP_RES_ANY, LDAP_MSG_ONE,
+               tout_p, &res ) ) > 0 )
        {
                if ( slapd_shutdown ) {
                        rc = -2;
@@ -702,21 +743,27 @@ do_syncrep2(
                        case LDAP_RES_SEARCH_ENTRY:
                                ldap_get_entry_controls( si->si_ld, msg, &rctrls );
                                /* we can't work without the control */
-                               if ( !rctrls ) {
-                                       Debug( LDAP_DEBUG_ANY, "do_syncrep2: "
+                               rctrlp = NULL;
+                               if ( rctrls ) {
+                                       /* NOTE: make sure we use the right one;
+                                        * a better approach would be to run thru
+                                        * the whole list and take care of all */
+                                       rctrlp = ldap_find_control( LDAP_CONTROL_SYNC_STATE, rctrls );
+                               }
+                               if ( rctrlp == NULL ) {
+                                       Debug( LDAP_DEBUG_ANY, "do_syncrep2: %s "
                                                "got search entry without "
-                                               "control\n", 0, 0, 0 );
+                                               "Sync State control\n", si->si_ridtxt, 0, 0 );
                                        rc = -1;
                                        goto done;
                                }
-                               rctrlp = *rctrls;
                                ber_init2( ber, &rctrlp->ldctl_value, LBER_USE_DER );
                                ber_scanf( ber, "{em" /*"}"*/, &syncstate, &syncUUID );
                                /* FIXME: what if syncUUID is NULL or empty?
                                 * (happens with back-sql...) */
                                if ( BER_BVISEMPTY( &syncUUID ) ) {
-                                       Debug( LDAP_DEBUG_ANY, "do_syncrep2: "
-                                               "got empty syncUUID\n", 0, 0, 0 );
+                                       Debug( LDAP_DEBUG_ANY, "do_syncrep2: %s "
+                                               "got empty syncUUID\n", si->si_ridtxt, 0, 0 );
                                        ldap_controls_free( rctrls );
                                        rc = -1;
                                        goto done;
@@ -735,16 +782,18 @@ do_syncrep2(
                                rc = 0;
                                if ( si->si_syncdata && si->si_logstate == SYNCLOG_LOGGING ) {
                                        modlist = NULL;
-                                       if (( rc = syncrepl_message_to_op( si, op, msg )) == LDAP_SUCCESS &&
-                                               !BER_BVISNULL( &syncCookie.ctxcsn ) ) {
+                                       if ( ( rc = syncrepl_message_to_op( si, op, msg ) ) == LDAP_SUCCESS &&
+                                               syncCookie.ctxcsn )
+                                       {
                                                rc = syncrepl_updateCookie( si, op, psub, &syncCookie );
                                        }
-                               } else if (( rc = syncrepl_message_to_entry( si, op, msg,
-                                       &modlist, &entry, syncstate )) == LDAP_SUCCESS ) {
-                                       if (( rc = syncrepl_entry( si, op, entry, &modlist,
-                                               syncstate, &syncUUID, &syncCookie_req,
-                                               &syncCookie.ctxcsn )) == LDAP_SUCCESS &&
-                                               !BER_BVISNULL( &syncCookie.ctxcsn ) ) {
+                               } else if ( ( rc = syncrepl_message_to_entry( si, op, msg,
+                                       &modlist, &entry, syncstate ) ) == LDAP_SUCCESS )
+                               {
+                                       if ( ( rc = syncrepl_entry( si, op, entry, &modlist,
+                                               syncstate, &syncUUID ) ) == LDAP_SUCCESS &&
+                                               syncCookie.ctxcsn )
+                                       {
                                                rc = syncrepl_updateCookie( si, op, psub, &syncCookie );
                                        }
                                }
@@ -758,12 +807,14 @@ do_syncrep2(
 
                        case LDAP_RES_SEARCH_REFERENCE:
                                Debug( LDAP_DEBUG_ANY,
-                                       "do_syncrep2: reference received error\n", 0, 0, 0 );
+                                       "do_syncrep2: %s reference received error\n",
+                                       si->si_ridtxt, 0, 0 );
                                break;
 
                        case LDAP_RES_SEARCH_RESULT:
                                Debug( LDAP_DEBUG_SYNC,
-                                       "do_syncrep2: LDAP_RES_SEARCH_RESULT\n", 0, 0, 0 );
+                                       "do_syncrep2: %s LDAP_RES_SEARCH_RESULT\n",
+                                       si->si_ridtxt, 0, 0 );
                                ldap_parse_result( si->si_ld, msg, &err, NULL, NULL, NULL,
                                        &rctrls, 0 );
 #ifdef LDAP_X_SYNC_REFRESH_REQUIRED
@@ -801,21 +852,13 @@ do_syncrep2(
                                        }
                                        ber_scanf( ber, /*"{"*/ "}" );
                                }
-                               if ( BER_BVISNULL( &syncCookie_req.ctxcsn )) {
+                               if ( !syncCookie_req.ctxcsn ) {
                                        match = -1;
-                               } else if ( BER_BVISNULL( &syncCookie.ctxcsn )) {
+                                       m = 0;
+                               } else if ( !syncCookie.ctxcsn ) {
                                        match = 1;
                                } else {
-                                       value_match( &match, slap_schema.si_ad_entryCSN,
-                                               slap_schema.si_ad_entryCSN->ad_type->sat_ordering,
-                                               SLAP_MR_VALUE_OF_ATTRIBUTE_SYNTAX,
-                                               &syncCookie_req.ctxcsn, &syncCookie.ctxcsn,
-                                               &text );
-                               }
-                               if ( !BER_BVISNULL( &syncCookie.ctxcsn ) &&
-                                       match < 0 && err == LDAP_SUCCESS )
-                               {
-                                       rc = syncrepl_updateCookie( si, op, psub, &syncCookie );
+                                       match = compare_csns( &syncCookie_req, &syncCookie, &m );
                                }
                                if ( rctrls ) {
                                        ldap_controls_free( rctrls );
@@ -828,12 +871,17 @@ do_syncrep2(
                                        if ( refreshDeletes == 0 && match < 0 &&
                                                err == LDAP_SUCCESS )
                                        {
-                                               syncrepl_del_nonpresent( op, si, NULL );
+                                               syncrepl_del_nonpresent( op, si, NULL,
+                                                       &syncCookie.ctxcsn[m] );
                                        } else {
                                                avl_free( si->si_presentlist, avl_ber_bvfree );
                                                si->si_presentlist = NULL;
                                        }
                                }
+                               if ( syncCookie.ctxcsn && match < 0 && err == LDAP_SUCCESS )
+                               {
+                                       rc = syncrepl_updateCookie( si, op, psub, &syncCookie );
+                               }
                                if ( err == LDAP_SUCCESS
                                        && si->si_logstate == SYNCLOG_FALLBACK ) {
                                        si->si_logstate = SYNCLOG_LOGGING;
@@ -850,23 +898,24 @@ do_syncrep2(
                                if ( !rc && !strcmp( retoid, LDAP_SYNC_INFO ) ) {
                                        ber_init2( ber, retdata, LBER_USE_DER );
 
-                                       switch ( si_tag = ber_peek_tag( ber, &len )) {
+                                       switch ( si_tag = ber_peek_tag( ber, &len ) ) {
                                        ber_tag_t tag;
                                        case LDAP_TAG_SYNC_NEW_COOKIE:
                                                Debug( LDAP_DEBUG_SYNC,
-                                                       "do_syncrep2: %s - %s%s\n", 
+                                                       "do_syncrep2: %s %s - %s\n", 
+                                                       si->si_ridtxt,
                                                        "LDAP_RES_INTERMEDIATE", 
-                                                       "NEW_COOKIE", "\n" );
+                                                       "NEW_COOKIE" );
                                                ber_scanf( ber, "tm", &tag, &cookie );
                                                break;
                                        case LDAP_TAG_SYNC_REFRESH_DELETE:
                                        case LDAP_TAG_SYNC_REFRESH_PRESENT:
                                                Debug( LDAP_DEBUG_SYNC,
-                                                       "do_syncrep2: %s - %s%s\n", 
+                                                       "do_syncrep2: %s %s - %s\n", 
+                                                       si->si_ridtxt,
                                                        "LDAP_RES_INTERMEDIATE", 
                                                        si_tag == LDAP_TAG_SYNC_REFRESH_PRESENT ?
-                                                       "REFRESH_PRESENT" : "REFRESH_DELETE",
-                                                       "\n" );
+                                                       "REFRESH_PRESENT" : "REFRESH_DELETE" );
                                                if ( si_tag == LDAP_TAG_SYNC_REFRESH_DELETE ) {
                                                        si->si_refreshDelete = 1;
                                                } else {
@@ -894,10 +943,10 @@ do_syncrep2(
                                                break;
                                        case LDAP_TAG_SYNC_ID_SET:
                                                Debug( LDAP_DEBUG_SYNC,
-                                                       "do_syncrep2: %s - %s%s\n", 
+                                                       "do_syncrep2: %s %s - %s\n", 
+                                                       si->si_ridtxt,
                                                        "LDAP_RES_INTERMEDIATE", 
-                                                       "SYNC_ID_SET",
-                                                       "\n" );
+                                                       "SYNC_ID_SET" );
                                                ber_scanf( ber, "t{" /*"}"*/, &tag );
                                                if ( ber_peek_tag( ber, &len ) ==
                                                        LDAP_TAG_SYNC_COOKIE )
@@ -910,6 +959,7 @@ do_syncrep2(
                                                        if ( !BER_BVISNULL( &syncCookie.octet_str ) )
                                                        {
                                                                slap_parse_sync_cookie( &syncCookie, NULL );
+                                                               compare_csns( &syncCookie_req, &syncCookie, &m );
                                                        }
                                                }
                                                if ( ber_peek_tag( ber, &len ) ==
@@ -920,50 +970,53 @@ do_syncrep2(
                                                ber_scanf( ber, "[W]", &syncUUIDs );
                                                ber_scanf( ber, /*"{"*/ "}" );
                                                if ( refreshDeletes ) {
-                                                       syncrepl_del_nonpresent( op, si, syncUUIDs );
+                                                       syncrepl_del_nonpresent( op, si, syncUUIDs,
+                                                               &syncCookie.ctxcsn[m] );
                                                        ber_bvarray_free_x( syncUUIDs, op->o_tmpmemctx );
                                                } else {
+                                                       int i;
                                                        for ( i = 0; !BER_BVISNULL( &syncUUIDs[i] ); i++ ) {
                                                                struct berval *syncuuid_bv;
                                                                syncuuid_bv = ber_dupbv( NULL, &syncUUIDs[i] );
                                                                slap_sl_free( syncUUIDs[i].bv_val,op->o_tmpmemctx );
-                                                               avl_insert( &si->si_presentlist,
+                                                               if ( avl_insert( &si->si_presentlist,
                                                                        (caddr_t) syncuuid_bv,
-                                                                       syncuuid_cmp, avl_dup_error );
+                                                                       syncuuid_cmp, avl_dup_error ) )
+                                                               {
+                                                                       ber_bvfree( syncuuid_bv );
+                                                               }
                                                        }
                                                        slap_sl_free( syncUUIDs, op->o_tmpmemctx );
                                                }
+                                               slap_sync_cookie_free( &syncCookie, 0 );
                                                break;
                                        default:
                                                Debug( LDAP_DEBUG_ANY,
-                                                       "do_syncrep2 : unknown syncinfo tag (%ld)\n",
-                                               (long) si_tag, 0, 0 );
+                                                       "do_syncrep2: %s unknown syncinfo tag (%ld)\n",
+                                                       si->si_ridtxt, (long) si_tag, 0 );
                                                ldap_memfree( retoid );
                                                ber_bvfree( retdata );
                                                continue;
                                        }
 
-                                       if ( BER_BVISNULL( &syncCookie_req.ctxcsn )) {
+                                       if ( !syncCookie_req.ctxcsn ) {
                                                match = -1;
-                                       } else if ( BER_BVISNULL( &syncCookie.ctxcsn )) {
+                                               m = 0;
+                                       } else if ( !syncCookie.ctxcsn ) {
                                                match = 1;
                                        } else {
-                                               value_match( &match, slap_schema.si_ad_entryCSN,
-                                                       slap_schema.si_ad_entryCSN->ad_type->sat_ordering,
-                                                       SLAP_MR_VALUE_OF_ATTRIBUTE_SYNTAX,
-                                                       &syncCookie_req.ctxcsn,
-                                                       &syncCookie.ctxcsn, &text );
+                                               match = compare_csns( &syncCookie_req, &syncCookie, &m );
                                        }
 
-                                       if ( !BER_BVISNULL( &syncCookie.ctxcsn ) &&
-                                               match < 0 )
-                                       {
-                                               rc = syncrepl_updateCookie( si, op, psub, &syncCookie);
-                                       }
+                                       if ( match < 0 ) {
+                                               if ( si->si_refreshPresent == 1 ) {
+                                                       syncrepl_del_nonpresent( op, si, NULL,
+                                                               &syncCookie.ctxcsn[m] );
+                                               }
 
-                                       if ( si->si_refreshPresent == 1 ) {
-                                               if ( match < 0 ) {
-                                                       syncrepl_del_nonpresent( op, si, NULL );
+                                               if ( syncCookie.ctxcsn )
+                                               {
+                                                       rc = syncrepl_updateCookie( si, op, psub, &syncCookie);
                                                }
                                        } 
 
@@ -972,9 +1025,9 @@ do_syncrep2(
                                        break;
 
                                } else {
-                                       Debug( LDAP_DEBUG_ANY, "do_syncrep2 : "
+                                       Debug( LDAP_DEBUG_ANY, "do_syncrep2: %s "
                                                "unknown intermediate response (%d)\n",
-                                               rc, 0, 0 );
+                                               si->si_ridtxt, rc, 0 );
                                        ldap_memfree( retoid );
                                        ber_bvfree( retdata );
                                        break;
@@ -982,12 +1035,12 @@ do_syncrep2(
                                break;
 
                        default:
-                               Debug( LDAP_DEBUG_ANY, "do_syncrep2 : "
-                                       "unknown message\n", 0, 0, 0 );
+                               Debug( LDAP_DEBUG_ANY, "do_syncrep2: %s "
+                                       "unknown message\n", si->si_ridtxt, 0, 0 );
                                break;
 
                        }
-                       if ( !BER_BVISNULL( &syncCookie.octet_str )) {
+                       if ( !BER_BVISNULL( &syncCookie.octet_str ) ) {
                                slap_sync_cookie_free( &syncCookie_req, 0 );
                                slap_dup_sync_cookie( &syncCookie_req, &syncCookie );
                                slap_sync_cookie_free( &syncCookie, 0 );
@@ -1004,7 +1057,7 @@ do_syncrep2(
                errstr = ldap_err2string( rc );
                
                Debug( LDAP_DEBUG_ANY,
-                       "do_syncrep2 : %s\n", errstr, 0, 0 );
+                       "do_syncrep2: %s %s\n", si->si_ridtxt, errstr, 0 );
        }
 
 done:
@@ -1043,14 +1096,14 @@ do_syncrepl(
        int i, defer = 1;
        Backend *be;
 
-       Debug( LDAP_DEBUG_TRACE, "=>do_syncrepl\n", 0, 0, 0 );
+       Debug( LDAP_DEBUG_TRACE, "=>do_syncrepl %s\n", si->si_ridtxt, 0, 0 );
 
        if ( si == NULL )
                return NULL;
 
        ldap_pvt_thread_mutex_lock( &si->si_mutex );
 
-       switch( abs( si->si_type )) {
+       switch( abs( si->si_type ) ) {
        case LDAP_SYNC_REFRESH_ONLY:
        case LDAP_SYNC_REFRESH_AND_PERSIST:
                break;
@@ -1084,6 +1137,17 @@ do_syncrepl(
        op->o_bd = be = si->si_be;
        op->o_dn = op->o_bd->be_rootdn;
        op->o_ndn = op->o_bd->be_rootndn;
+       if ( !si->si_schemachecking )
+               op->o_no_schema_check = 1;
+
+       /* If we're glued, send writes through the glue parent */
+       if ( !si->si_wbe ) {
+               if ( SLAP_GLUE_SUBORDINATE( be )) {
+                       si->si_wbe = select_backend( &be->be_nsuffix[0], 0, 1 );
+               } else {
+                       si->si_wbe = be;
+               }
+       }
 
        /* Establish session, do search */
        if ( !si->si_ld ) {
@@ -1103,6 +1167,13 @@ reload:
                        goto reload;
                }
 
+               /* We got deleted while running on cn=config */
+               if ( !si->si_ctype ) {
+                       if ( si->si_conn_setup )
+                               dostop = 1;
+                       rc = -1;
+               }
+
                if ( abs(si->si_type) == LDAP_SYNC_REFRESH_AND_PERSIST ) {
                        /* If we succeeded, enable the connection for further listening.
                         * If we failed, tear down the connection and reschedule.
@@ -1131,7 +1202,7 @@ reload:
         */
        ldap_pvt_thread_mutex_lock( &slapd_rq.rq_mutex );
 
-       if ( ldap_pvt_runqueue_isrunning( &slapd_rq, rtask )) {
+       if ( ldap_pvt_runqueue_isrunning( &slapd_rq, rtask ) ) {
                ldap_pvt_runqueue_stoptask( &slapd_rq, rtask );
        }
 
@@ -1157,7 +1228,8 @@ reload:
                                break;
                }
 
-               if ( !si->si_retrynum || si->si_retrynum[i] == RETRYNUM_TAIL ) {
+               if ( !si->si_ctype
+                       || !si->si_retrynum || si->si_retrynum[i] == RETRYNUM_TAIL ) {
                        ldap_pvt_runqueue_remove( &slapd_rq, rtask );
                } else if ( RETRYNUM_VALID( si->si_retrynum[i] ) ) {
                        if ( si->si_retrynum[i] > 0 )
@@ -1171,6 +1243,24 @@ reload:
        ldap_pvt_thread_mutex_unlock( &slapd_rq.rq_mutex );
        ldap_pvt_thread_mutex_unlock( &si->si_mutex );
 
+       /* Do final delete cleanup */
+       if ( !si->si_ctype ) {
+               cookie_state *cs = NULL;
+               syncinfo_t **sip;
+
+               cs = be->be_syncinfo->si_cookieState;
+               for ( sip = &be->be_syncinfo; *sip != si; sip = &(*sip)->si_next );
+               *sip = si->si_next;
+               syncinfo_free( si );
+               if ( !be->be_syncinfo ) {
+                       SLAP_DBFLAGS( be ) &= ~(SLAP_DBFLAG_SHADOW|SLAP_DBFLAG_SYNC_SHADOW);
+                       if ( cs ) {
+                               ber_bvarray_free( cs->cs_vals );
+                               ldap_pvt_thread_mutex_destroy( &cs->cs_mutex );
+                               ch_free( cs );
+                       }
+               }
+       }
        return NULL;
 }
 
@@ -1203,20 +1293,28 @@ syncrepl_accesslog_mods(
                bv = vals[i];
 
                colon = ber_bvchr( &bv, ':' );
-               if ( !colon )
-                       continue;       /* invalid */
+               if ( !colon ) {
+                       /* Invalid */
+                       continue;
+               }
+
                bv.bv_len = colon - bv.bv_val;
-               if ( slap_bv2ad( &bv, &ad, &text )) {
+               if ( slap_bv2ad( &bv, &ad, &text ) ) {
                        /* Invalid */
                        continue;
                }
+
                /* Ignore dynamically generated attrs */
-               if ( ad->ad_type->sat_flags & SLAP_AT_DYNAMIC )
+               if ( ad->ad_type->sat_flags & SLAP_AT_DYNAMIC ) {
                        continue;
+               }
+
                /* Ignore excluded attrs */
                if ( ldap_charray_inlist( si->si_exattrs,
-                       ad->ad_type->sat_cname.bv_val ))
+                       ad->ad_type->sat_cname.bv_val ) )
+               {
                        continue;
+               }
 
                switch(colon[1]) {
                case '+':       op = LDAP_MOD_ADD; break;
@@ -1227,7 +1325,7 @@ syncrepl_accesslog_mods(
                }
 
                if ( !mod || ad != mod->sml_desc || op != mod->sml_op ) {
-                       mod = (Modifications *) ch_malloc( sizeof( Modifications ));
+                       mod = (Modifications *) ch_malloc( sizeof( Modifications ) );
                        mod->sml_flags = 0;
                        mod->sml_op = op;
                        mod->sml_next = NULL;
@@ -1280,11 +1378,12 @@ syncrepl_message_to_op(
        struct berval   rdn = BER_BVNULL, sup = BER_BVNULL,
                prdn = BER_BVNULL, nrdn = BER_BVNULL,
                psup = BER_BVNULL, nsup = BER_BVNULL;
-       int             rc, deleteOldRdn = 0;
+       int             rc, deleteOldRdn = 0, freeReqDn = 0;
 
        if ( ldap_msgtype( msg ) != LDAP_RES_SEARCH_ENTRY ) {
-               Debug( LDAP_DEBUG_ANY,
-                       "Message type should be entry (%d)", ldap_msgtype( msg ), 0, 0 );
+               Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_op: %s "
+                       "Message type should be entry (%d)",
+                       si->si_ridtxt, ldap_msgtype( msg ), 0 );
                return -1;
        }
 
@@ -1297,62 +1396,69 @@ syncrepl_message_to_op(
 
        if ( rc != LDAP_SUCCESS ) {
                Debug( LDAP_DEBUG_ANY,
-                       "syncrepl_message_to_op : dn get failed (%d)", rc, 0, 0 );
+                       "syncrepl_message_to_op: %s dn get failed (%d)",
+                       si->si_ridtxt, rc, 0 );
                return rc;
        }
 
        op->o_tag = LBER_DEFAULT;
+       op->o_bd = si->si_wbe;
 
-       while (( rc = ldap_get_attribute_ber( si->si_ld, msg, ber, &bv, &bvals ))
+       while (( rc = ldap_get_attribute_ber( si->si_ld, msg, ber, &bv, &bvals ) )
                == LDAP_SUCCESS ) {
                if ( bv.bv_val == NULL )
                        break;
 
-               if ( !ber_bvstrcasecmp( &bv, &ls->ls_dn )) {
+               if ( !ber_bvstrcasecmp( &bv, &ls->ls_dn ) ) {
                        bdn = bvals[0];
                        dnPrettyNormal( NULL, &bdn, &dn, &ndn, op->o_tmpmemctx );
                        ber_dupbv( &op->o_req_dn, &dn );
                        ber_dupbv( &op->o_req_ndn, &ndn );
                        slap_sl_free( ndn.bv_val, op->o_tmpmemctx );
                        slap_sl_free( dn.bv_val, op->o_tmpmemctx );
-               } else if ( !ber_bvstrcasecmp( &bv, &ls->ls_req )) {
+                       freeReqDn = 1;
+               } else if ( !ber_bvstrcasecmp( &bv, &ls->ls_req ) ) {
                        int i = verb_to_mask( bvals[0].bv_val, modops );
                        if ( i < 0 ) {
                                Debug( LDAP_DEBUG_ANY,
-                                       "syncrepl_message_to_op : unknown op %s",
-                                       bvals[0].bv_val, 0, 0 );
+                                       "syncrepl_message_to_op: %s unknown op %s",
+                                       si->si_ridtxt, bvals[0].bv_val, 0 );
                                ch_free( bvals );
                                rc = -1;
                                goto done;
                        }
                        op->o_tag = modops[i].mask;
-               } else if ( !ber_bvstrcasecmp( &bv, &ls->ls_mod )) {
+               } else if ( !ber_bvstrcasecmp( &bv, &ls->ls_mod ) ) {
                        /* Parse attribute into modlist */
-                       if ( si->si_syncdata == SYNCDATA_ACCESSLOG )
+                       if ( si->si_syncdata == SYNCDATA_ACCESSLOG ) {
                                modlist = syncrepl_accesslog_mods( si, bvals );
-                       else
+                       } else {
                                modlist = syncrepl_changelog_mods( si, bvals );
-               } else if ( !ber_bvstrcasecmp( &bv, &ls->ls_newRdn )) {
+                       }
+               } else if ( !ber_bvstrcasecmp( &bv, &ls->ls_newRdn ) ) {
                        rdn = bvals[0];
-               } else if ( !ber_bvstrcasecmp( &bv, &ls->ls_delRdn )) {
-                       if ( !ber_bvstrcasecmp( &slap_true_bv, bvals ))
+               } else if ( !ber_bvstrcasecmp( &bv, &ls->ls_delRdn ) ) {
+                       if ( !ber_bvstrcasecmp( &slap_true_bv, bvals ) ) {
                                deleteOldRdn = 1;
-               } else if ( !ber_bvstrcasecmp( &bv, &ls->ls_newSup )) {
+                       }
+               } else if ( !ber_bvstrcasecmp( &bv, &ls->ls_newSup ) ) {
                        sup = bvals[0];
                } else if ( !ber_bvstrcasecmp( &bv,
-                       &slap_schema.si_ad_entryCSN->ad_cname )) {
+                       &slap_schema.si_ad_entryCSN->ad_cname ) )
+               {
                        slap_queue_csn( op, bvals );
                }
                ch_free( bvals );
        }
 
        /* If we didn't get a mod type or a target DN, bail out */
-       if ( op->o_tag == LBER_DEFAULT || BER_BVISNULL( &dn )) {
+       if ( op->o_tag == LBER_DEFAULT || BER_BVISNULL( &dn ) ) {
                rc = -1;
                goto done;
        }
 
        op->o_callback = &cb;
+       slap_op_time( &op->o_time, &op->o_tincr );
 
        switch( op->o_tag ) {
        case LDAP_REQ_ADD:
@@ -1360,84 +1466,120 @@ syncrepl_message_to_op(
                /* If we didn't get required data, bail */
                if ( !modlist ) goto done;
 
-               rc = slap_mods_check( modlist, &text, txtbuf, textlen, NULL );
+               rc = slap_mods_check( op, modlist, &text, txtbuf, textlen, NULL );
 
                if ( rc != LDAP_SUCCESS ) {
-                       Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_op: mods check (%s)\n",
-                               text, 0, 0 );
+                       Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_op: %s "
+                               "mods check (%s)\n",
+                               si->si_ridtxt, text, 0 );
                        goto done;
                }
 
                if ( op->o_tag == LDAP_REQ_ADD ) {
-                       op->ora_e = ( Entry * ) ch_calloc( 1, sizeof( Entry ) );
+                       Entry *e = entry_alloc();
+                       op->ora_e = e;
                        op->ora_e->e_name = op->o_req_dn;
                        op->ora_e->e_nname = op->o_req_ndn;
+                       freeReqDn = 0;
                        rc = slap_mods2entry( modlist, &op->ora_e, 1, 0, &text, txtbuf, textlen);
                        if( rc != LDAP_SUCCESS ) {
-                               Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_op: mods2entry (%s)\n",
-                                       text, 0, 0 );
+                               Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_op: %s "
+                               "mods2entry (%s)\n",
+                                       si->si_ridtxt, text, 0 );
                        } else {
                                rc = op->o_bd->be_add( op, &rs );
                                Debug( LDAP_DEBUG_SYNC,
-                                       "syncrepl_message_to_op: be_add %s (%d)\n", 
-                                       op->o_req_dn.bv_val, rc, 0 );
+                                       "syncrepl_message_to_op: %s be_add %s (%d)\n", 
+                                       si->si_ridtxt, op->o_req_dn.bv_val, rc );
                        }
-                       be_entry_release_w( op, op->ora_e );
+                       if ( e == op->ora_e )
+                               be_entry_release_w( op, op->ora_e );
                } else {
                        op->orm_modlist = modlist;
+                       op->o_bd = si->si_wbe;
                        rc = op->o_bd->be_modify( op, &rs );
-                       Debug( LDAP_DEBUG_SYNC,
-                               "syncrepl_message_to_op: be_modify %s (%d)\n", 
-                               op->o_req_dn.bv_val, rc, 0 );
+                       Debug( rc ? LDAP_DEBUG_ANY : LDAP_DEBUG_SYNC,
+                               "syncrepl_message_to_op: %s be_modify %s (%d)\n", 
+                               si->si_ridtxt, op->o_req_dn.bv_val, rc );
+                       op->o_bd = si->si_be;
                }
                break;
        case LDAP_REQ_MODRDN:
-               if ( BER_BVISNULL( &rdn )) goto done;
+               if ( BER_BVISNULL( &rdn ) ) goto done;
 
-               if ( rdnPretty( NULL, &rdn, &prdn, NULL ))
+               if ( rdnPretty( NULL, &rdn, &prdn, NULL ) ) {
                        goto done;
-               if ( rdnNormalize( 0, NULL, NULL, &rdn, &nrdn, NULL ))
+               }
+               if ( rdnNormalize( 0, NULL, NULL, &rdn, &nrdn, NULL ) ) {
                        goto done;
-               if ( !BER_BVISNULL( &sup )) {
-                       if ( dnPrettyNormal( NULL, &sup, &psup, &nsup, NULL ))
+               }
+               if ( !BER_BVISNULL( &sup ) ) {
+                       if ( dnPrettyNormal( NULL, &sup, &psup, &nsup, NULL ) ) {
                                goto done;
+                       }
                        op->orr_newSup = &psup;
                        op->orr_nnewSup = &nsup;
+               } else {
+                       op->orr_newSup = NULL;
+                       op->orr_nnewSup = NULL;
                }
                op->orr_newrdn = prdn;
                op->orr_nnewrdn = nrdn;
                op->orr_deleteoldrdn = deleteOldRdn;
                op->orr_modlist = NULL;
-               if ( slap_modrdn2mods( op, &rs ))
+               if ( slap_modrdn2mods( op, &rs ) ) {
                        goto done;
+               }
+
+               /* Append modlist for operational attrs */
+               {
+                       Modifications *m;
+
+                       for ( m = op->orr_modlist; m->sml_next; m = m->sml_next )
+                               ;
+                       m->sml_next = modlist;
+                       modlist = NULL;
+               }
                rc = op->o_bd->be_modrdn( op, &rs );
                slap_mods_free( op->orr_modlist, 1 );
-               Debug( LDAP_DEBUG_SYNC,
-                       "syncrepl_message_to_op: be_modrdn %s (%d)\n", 
-                       op->o_req_dn.bv_val, rc, 0 );
+               Debug( rc ? LDAP_DEBUG_ANY : LDAP_DEBUG_SYNC,
+                       "syncrepl_message_to_op: %s be_modrdn %s (%d)\n", 
+                       si->si_ridtxt, op->o_req_dn.bv_val, rc );
                break;
        case LDAP_REQ_DELETE:
                rc = op->o_bd->be_delete( op, &rs );
-               Debug( LDAP_DEBUG_SYNC,
-                       "syncrepl_message_to_op: be_delete %s (%d)\n", 
-                       op->o_req_dn.bv_val, rc, 0 );
+               Debug( rc ? LDAP_DEBUG_ANY : LDAP_DEBUG_SYNC,
+                       "syncrepl_message_to_op: %s be_delete %s (%d)\n", 
+                       si->si_ridtxt, op->o_req_dn.bv_val, rc );
                break;
        }
 done:
        slap_graduate_commit_csn( op );
-       if ( modlist )
+       op->o_bd = si->si_be;
+       op->o_tmpfree( op->o_csn.bv_val, op->o_tmpmemctx );
+       BER_BVZERO( &op->o_csn );
+       if ( modlist ) {
                slap_mods_free( modlist, op->o_tag != LDAP_REQ_ADD );
-       if ( !BER_BVISNULL( &rdn )) {
-               if ( !BER_BVISNULL( &nsup ))
+       }
+       if ( !BER_BVISNULL( &rdn ) ) {
+               if ( !BER_BVISNULL( &nsup ) ) {
                        ch_free( nsup.bv_val );
-               if ( !BER_BVISNULL( &psup ))
+               }
+               if ( !BER_BVISNULL( &psup ) ) {
                        ch_free( psup.bv_val );
-               if ( !BER_BVISNULL( &nrdn ))
+               }
+               if ( !BER_BVISNULL( &nrdn ) ) {
                        ch_free( nrdn.bv_val );
-               if ( !BER_BVISNULL( &prdn ))
+               }
+               if ( !BER_BVISNULL( &prdn ) ) {
                        ch_free( prdn.bv_val );
+               }
        }
-       ber_free ( ber, 0 );
+       if ( freeReqDn ) {
+               ch_free( op->o_req_ndn.bv_val );
+               ch_free( op->o_req_dn.bv_val );
+       }
+       ber_free( ber, 0 );
        return rc;
 }
 
@@ -1461,44 +1603,49 @@ syncrepl_message_to_entry(
        char txtbuf[SLAP_TEXT_BUFLEN];
        size_t textlen = sizeof txtbuf;
 
-       struct berval   bdn = {0, NULL}, dn, ndn;
-       int             rc;
+       struct berval   bdn = BER_BVNULL, dn, ndn;
+       int             rc, is_ctx;
 
        *modlist = NULL;
 
        if ( ldap_msgtype( msg ) != LDAP_RES_SEARCH_ENTRY ) {
-               Debug( LDAP_DEBUG_ANY,
-                       "Message type should be entry (%d)", ldap_msgtype( msg ), 0, 0 );
+               Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_entry: %s "
+                       "Message type should be entry (%d)",
+                       si->si_ridtxt, ldap_msgtype( msg ), 0 );
                return -1;
        }
 
        op->o_tag = LDAP_REQ_ADD;
 
        rc = ldap_get_dn_ber( si->si_ld, msg, &ber, &bdn );
-
        if ( rc != LDAP_SUCCESS ) {
                Debug( LDAP_DEBUG_ANY,
-                       "syncrepl_message_to_entry : dn get failed (%d)", rc, 0, 0 );
+                       "syncrepl_message_to_entry: %s dn get failed (%d)",
+                       si->si_ridtxt, rc, 0 );
                return rc;
        }
 
+       if ( syncstate == LDAP_SYNC_PRESENT || syncstate == LDAP_SYNC_DELETE ) {
+               /* NOTE: this could be done even before decoding the DN,
+                * although encoding errors wouldn't be detected */
+               rc = LDAP_SUCCESS;
+               goto done;
+       }
+
+       if ( entry == NULL ) {
+               rc = -1;
+               goto done;
+       }
+
        dnPrettyNormal( NULL, &bdn, &dn, &ndn, op->o_tmpmemctx );
        ber_dupbv( &op->o_req_dn, &dn );
        ber_dupbv( &op->o_req_ndn, &ndn );
        slap_sl_free( ndn.bv_val, op->o_tmpmemctx );
        slap_sl_free( dn.bv_val, op->o_tmpmemctx );
 
-       if ( syncstate == LDAP_SYNC_PRESENT || syncstate == LDAP_SYNC_DELETE ) {
-               if ( entry )
-                       *entry = NULL;
-               return LDAP_SUCCESS;
-       }
+       is_ctx = dn_match( &op->o_req_ndn, &op->o_bd->be_nsuffix[0] );
 
-       if ( entry == NULL ) {
-               return -1;
-       }
-
-       e = ( Entry * ) ch_calloc( 1, sizeof( Entry ) );
+       e = entry_alloc();
        e->e_name = op->o_req_dn;
        e->e_nname = op->o_req_ndn;
 
@@ -1509,7 +1656,16 @@ syncrepl_message_to_entry(
                        break;
                }
 
-               mod  = (Modifications *) ch_malloc( sizeof( Modifications ));
+               /* Drop all updates to the contextCSN of the context entry
+                * (ITS#4622, etc.)
+                */
+               if ( is_ctx && !strcasecmp( tmp.sml_type.bv_val,
+                       slap_schema.si_ad_contextCSN->ad_cname.bv_val )) {
+                       ber_bvarray_free( tmp.sml_values );
+                       continue;
+               }
+
+               mod  = (Modifications *) ch_malloc( sizeof( Modifications ) );
 
                mod->sml_op = LDAP_MOD_REPLACE;
                mod->sml_flags = 0;
@@ -1524,17 +1680,17 @@ syncrepl_message_to_entry(
        }
 
        if ( *modlist == NULL ) {
-               Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_entry: no attributes\n",
-                       0, 0, 0 );
+               Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_entry: %s no attributes\n",
+                       si->si_ridtxt, 0, 0 );
                rc = -1;
                goto done;
        }
 
-       rc = slap_mods_check( *modlist, &text, txtbuf, textlen, NULL );
+       rc = slap_mods_check( op, *modlist, &text, txtbuf, textlen, NULL );
 
        if ( rc != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_entry: mods check (%s)\n",
-                       text, 0, 0 );
+               Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_entry: %s mods check (%s)\n",
+                       si->si_ridtxt, text, 0 );
                goto done;
        }
 
@@ -1554,7 +1710,8 @@ syncrepl_message_to_entry(
        for ( modtail = modlist; *modtail ; ) {
                mod = *modtail;
                if ( ldap_charray_inlist( si->si_exattrs,
-                                       mod->sml_desc->ad_type->sat_cname.bv_val )) {
+                       mod->sml_desc->ad_type->sat_cname.bv_val ) )
+               {
                        *modtail = mod->sml_next;
                        slap_mod_free( &mod->sml_mod, 0 );
                        ch_free( mod );
@@ -1562,15 +1719,15 @@ syncrepl_message_to_entry(
                        modtail = &mod->sml_next;
                }
        }
-       
+
        rc = slap_mods2entry( *modlist, &e, 1, 1, &text, txtbuf, textlen);
        if( rc != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_entry: mods2entry (%s)\n",
-                       text, 0, 0 );
+               Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_entry: %s mods2entry (%s)\n",
+                       si->si_ridtxt, text, 0 );
        }
 
 done:
-       ber_free ( ber, 0 );
+       ber_free( ber, 0 );
        if ( rc != LDAP_SUCCESS ) {
                if ( e ) {
                        entry_free( e );
@@ -1599,17 +1756,18 @@ static struct berval generic_filterstr = BER_BVC("(objectclass=*)");
  * the new ones. This avoids the issue of trying to delete/add a non-leaf
  * entry.
  *
- * We don't try to otherwise distinguish ModDN from Modify; in the case of
- * a ModDN we will issue both operations on the local database.
+ * We otherwise distinguish ModDN from Modify; in the case of
+ * a ModDN we just use the CSN, modifyTimestamp and modifiersName
+ * operational attributes from the entry, and do a regular ModDN.
  */
 typedef struct dninfo {
        Entry *new_entry;
        struct berval dn;
        struct berval ndn;
        int renamed;    /* Was an existing entry renamed? */
-       int wasChanged; /* are the attributes changed? */
-       int attrs;              /* how many attribute types are in the ads list */
-       AttributeDescription **ads;
+       int delOldRDN;  /* Was old RDN deleted? */
+       Modifications **modlist;        /* the modlist we received */
+       Modifications *mods;    /* the modlist we compared */
 } dninfo;
 
 static int
@@ -1619,9 +1777,7 @@ syncrepl_entry(
        Entry* entry,
        Modifications** modlist,
        int syncstate,
-       struct berval* syncUUID,
-       struct sync_cookie* syncCookie_req,
-       struct berval* syncCSN )
+       struct berval* syncUUID )
 {
        Backend *be = op->o_bd;
        slap_callback   cb = { NULL, NULL, NULL, NULL };
@@ -1647,36 +1803,40 @@ syncrepl_entry(
 
        switch( syncstate ) {
        case LDAP_SYNC_PRESENT:
-               Debug( LDAP_DEBUG_SYNC, "%s: %s\n",
-                                       "syncrepl_entry",
+               Debug( LDAP_DEBUG_SYNC, "syncrepl_entry: %s %s\n",
+                                       si->si_ridtxt,
                                        "LDAP_RES_SEARCH_ENTRY(LDAP_SYNC_PRESENT)", 0 );
                break;
        case LDAP_SYNC_ADD:
-               Debug( LDAP_DEBUG_SYNC, "%s: %s\n",
-                                       "syncrepl_entry",
+               Debug( LDAP_DEBUG_SYNC, "syncrepl_entry: %s %s\n",
+                                       si->si_ridtxt,
                                        "LDAP_RES_SEARCH_ENTRY(LDAP_SYNC_ADD)", 0 );
                break;
        case LDAP_SYNC_DELETE:
-               Debug( LDAP_DEBUG_SYNC, "%s: %s\n",
-                                       "syncrepl_entry",
+               Debug( LDAP_DEBUG_SYNC, "syncrepl_entry: %s %s\n",
+                                       si->si_ridtxt,
                                        "LDAP_RES_SEARCH_ENTRY(LDAP_SYNC_DELETE)", 0 );
                break;
        case LDAP_SYNC_MODIFY:
-               Debug( LDAP_DEBUG_SYNC, "%s: %s\n",
-                                       "syncrepl_entry",
+               Debug( LDAP_DEBUG_SYNC, "syncrepl_entry: %s %s\n",
+                                       si->si_ridtxt,
                                        "LDAP_RES_SEARCH_ENTRY(LDAP_SYNC_MODIFY)", 0 );
                break;
        default:
-               Debug( LDAP_DEBUG_ANY, "%s: %s\n",
-                                       "syncrepl_entry",
+               Debug( LDAP_DEBUG_ANY, "syncrepl_entry: %s %s\n",
+                                       si->si_ridtxt,
                                        "LDAP_RES_SEARCH_ENTRY(UNKNOWN syncstate)", 0 );
        }
 
-       if (( syncstate == LDAP_SYNC_PRESENT || syncstate == LDAP_SYNC_ADD )) {
+       if (( syncstate == LDAP_SYNC_PRESENT || syncstate == LDAP_SYNC_ADD ) ) {
                if ( !si->si_refreshPresent ) {
                        syncuuid_bv = ber_dupbv( NULL, syncUUID );
-                       avl_insert( &si->si_presentlist, (caddr_t) syncuuid_bv,
-                               syncuuid_cmp, avl_dup_error );
+                       if ( avl_insert( &si->si_presentlist, (caddr_t) syncuuid_bv,
+                               syncuuid_cmp, avl_dup_error ) )
+                       {
+                               ber_bvfree( syncuuid_bv );
+                               syncuuid_bv = NULL;
+                       }
                }
        }
 
@@ -1688,15 +1848,34 @@ syncrepl_entry(
                }
        }
 
+       (void)slap_uuidstr_from_normalized( &syncUUID_strrep, syncUUID, op->o_tmpmemctx );
+       if ( syncstate != LDAP_SYNC_DELETE ) {
+               Attribute       *a = attr_find( entry->e_attrs, slap_schema.si_ad_entryUUID );
+
+               if ( a == NULL ) {
+                       /* add if missing */
+                       attr_merge_one( entry, slap_schema.si_ad_entryUUID,
+                               &syncUUID_strrep, syncUUID );
+
+               } else if ( !bvmatch( &a->a_nvals[0], syncUUID ) ) {
+                       /* replace only if necessary */
+                       if ( a->a_nvals != a->a_vals ) {
+                               ber_memfree( a->a_nvals[0].bv_val );
+                               ber_dupbv( &a->a_nvals[0], syncUUID );
+                       }
+                       ber_memfree( a->a_vals[0].bv_val );
+                       ber_dupbv( &a->a_vals[0], &syncUUID_strrep );
+               }
+       }
+
        f.f_choice = LDAP_FILTER_EQUALITY;
        f.f_ava = &ava;
        ava.aa_desc = slap_schema.si_ad_entryUUID;
        ava.aa_value = *syncUUID;
 
-       (void)slap_uuidstr_from_normalized( &syncUUID_strrep, syncUUID, op->o_tmpmemctx );
        if ( syncuuid_bv ) {
-               Debug( LDAP_DEBUG_SYNC, "syncrepl_entry: inserted UUID %s\n",
-                       syncUUID_strrep.bv_val, 0, 0 );
+               Debug( LDAP_DEBUG_SYNC, "syncrepl_entry: %s inserted UUID %s\n",
+                       si->si_ridtxt, syncUUID_strrep.bv_val, 0 );
        }
        op->ors_filter = &f;
 
@@ -1729,12 +1908,13 @@ syncrepl_entry(
        cb.sc_response = dn_callback;
        cb.sc_private = &dni;
        dni.new_entry = entry;
+       dni.modlist = modlist;
 
        if ( limits_check( op, &rs_search ) == 0 ) {
                rc = be->be_search( op, &rs_search );
                Debug( LDAP_DEBUG_SYNC,
-                               "syncrepl_entry: %s (%d)\n", 
-                               "be_search", rc, 0 );
+                               "syncrepl_entry: %s be_search (%d)\n", 
+                               si->si_ridtxt, rc, 0 );
        }
 
        if ( !BER_BVISNULL( &op->ors_filterstr ) ) {
@@ -1746,33 +1926,15 @@ syncrepl_entry(
 
        if ( entry && !BER_BVISNULL( &entry->e_name ) ) {
                Debug( LDAP_DEBUG_SYNC,
-                               "syncrepl_entry: %s\n",
-                               entry->e_name.bv_val, 0, 0 );
+                               "syncrepl_entry: %s %s\n",
+                               si->si_ridtxt, entry->e_name.bv_val, 0 );
        } else {
                Debug( LDAP_DEBUG_SYNC,
-                               "syncrepl_entry: %s\n",
-                               dni.dn.bv_val ? dni.dn.bv_val : "(null)", 0, 0 );
-       }
-
-       if ( syncstate != LDAP_SYNC_DELETE ) {
-               Attribute       *a = attr_find( entry->e_attrs, slap_schema.si_ad_entryUUID );
-
-               if ( a == NULL ) {
-                       /* add if missing */
-                       attr_merge_one( entry, slap_schema.si_ad_entryUUID,
-                               &syncUUID_strrep, syncUUID );
-
-               } else if ( !bvmatch( &a->a_nvals[0], syncUUID ) ) {
-                       /* replace only if necessary */
-                       if ( a->a_nvals != a->a_vals ) {
-                               ber_memfree( a->a_nvals[0].bv_val );
-                               ber_dupbv( &a->a_nvals[0], syncUUID );
-                       }
-                       ber_memfree( a->a_vals[0].bv_val );
-                       ber_dupbv( &a->a_vals[0], &syncUUID_strrep );
-               }
+                               "syncrepl_entry: %s %s\n",
+                               si->si_ridtxt, dni.dn.bv_val ? dni.dn.bv_val : "(null)", 0 );
        }
 
+       slap_op_time( &op->o_time, &op->o_tincr );
        switch ( syncstate ) {
        case LDAP_SYNC_ADD:
        case LDAP_SYNC_MODIFY:
@@ -1789,20 +1951,23 @@ syncrepl_entry(
                        }
                }
 retry_add:;
-               if ( BER_BVISNULL( &dni.dn )) {
+               if ( BER_BVISNULL( &dni.dn ) ) {
 
                        op->o_req_dn = entry->e_name;
                        op->o_req_ndn = entry->e_nname;
                        op->o_tag = LDAP_REQ_ADD;
                        op->ora_e = entry;
+                       op->o_bd = si->si_wbe;
 
-                       rc = be->be_add( op, &rs_add );
+                       rc = op->o_bd->be_add( op, &rs_add );
                        Debug( LDAP_DEBUG_SYNC,
-                                       "syncrepl_entry: %s (%d)\n", 
-                                       "be_add", rc, 0 );
+                                       "syncrepl_entry: %s be_add (%d)\n", 
+                                       si->si_ridtxt, rc, 0 );
                        switch ( rs_add.sr_err ) {
                        case LDAP_SUCCESS:
-                               be_entry_release_w( op, entry );
+                               if ( op->ora_e == entry ) {
+                                       be_entry_release_w( op, entry );
+                               }
                                entry = NULL;
                                break;
 
@@ -1827,6 +1992,7 @@ retry_add:;
                                        SlapReply       rs2 = { 0 };
                                        slap_callback   cb2 = { 0 };
 
+                                       op2.o_bd = be;
                                        op2.o_tag = LDAP_REQ_SEARCH;
                                        op2.o_req_dn = entry->e_name;
                                        op2.o_req_ndn = entry->e_nname;
@@ -1851,16 +2017,18 @@ retry_add:;
                                        if ( rc ) goto done;
 
                                        retry = 0;
+                                       slap_op_time( &op->o_time, &op->o_tincr );
                                        goto retry_add;
                                }
                                /* FALLTHRU */
 
                        default:
                                Debug( LDAP_DEBUG_ANY,
-                                       "syncrepl_entry : be_add failed (%d)\n",
-                                       rs_add.sr_err, 0, 0 );
+                                       "syncrepl_entry: %s be_add failed (%d)\n",
+                                       si->si_ridtxt, rs_add.sr_err, 0 );
                                break;
                        }
+                       op->o_bd = be;
                        goto done;
                }
                /* FALLTHRU */
@@ -1875,105 +2043,108 @@ retry_add:;
 
                        dnParent( &dni.ndn, &noldp );
                        dnParent( &entry->e_nname, &nnewp );
-                       if ( !dn_match( &noldp, &nnewp )) {
+                       if ( !dn_match( &noldp, &nnewp ) ) {
                                dnParent( &entry->e_name, &newp );
                                op->orr_newSup = &newp;
                                op->orr_nnewSup = &nnewp;
+                       } else {
+                               op->orr_newSup = NULL;
+                               op->orr_nnewSup = NULL;
                        }
-                       op->orr_deleteoldrdn = 0;
+                       op->orr_deleteoldrdn = dni.delOldRDN;
                        op->orr_modlist = NULL;
-                       if (( rc = slap_modrdn2mods( op, &rs_modify ))) {
+                       if ( ( rc = slap_modrdn2mods( op, &rs_modify ) ) ) {
                                goto done;
                        }
-                       rc = be->be_modrdn( op, &rs_modify );
-                       slap_mods_free( op->orr_modlist, 1 );
-                       Debug( LDAP_DEBUG_SYNC,
-                                       "syncrepl_entry: %s (%d)\n", 
-                                       "be_modrdn", rc, 0 );
-                       if ( rs_modify.sr_err == LDAP_SUCCESS ) {
-                               op->o_req_dn = entry->e_name;
-                               op->o_req_ndn = entry->e_nname;
-                       } else {
-                               goto done;
-                       }
-               }
-               if ( dni.wasChanged ) {
-                       Modifications *mod, *modhead = NULL;
-                       Modifications *modtail = NULL;
-                       int i;
 
-                       op->o_tag = LDAP_REQ_MODIFY;
+                       /* RDNs must be NUL-terminated for back-ldap */
+                       noldp = op->orr_newrdn;
+                       ber_dupbv_x( &op->orr_newrdn, &noldp, op->o_tmpmemctx );
+                       noldp = op->orr_nnewrdn;
+                       ber_dupbv_x( &op->orr_nnewrdn, &noldp, op->o_tmpmemctx );
 
-                       assert( *modlist != NULL );
-
-                       /* Delete all the old attrs */
-                       for ( i = 0; i < dni.attrs; i++ ) {
-                               mod = ch_malloc( sizeof( Modifications ) );
-                               mod->sml_op = LDAP_MOD_DELETE;
-                               mod->sml_flags = 0;
-                               mod->sml_desc = dni.ads[i];
-                               mod->sml_type = mod->sml_desc->ad_cname;
-                               mod->sml_values = NULL;
-                               mod->sml_nvalues = NULL;
-                               if ( !modhead ) modhead = mod;
-                               if ( modtail ) {
-                                       modtail->sml_next = mod;
-                               }
-                               modtail = mod;
-                       }
+                       /* Setup opattrs too */
+                       {
+                               AttributeDescription *opattrs[] = {
+                                       slap_schema.si_ad_entryCSN,
+                                       slap_schema.si_ad_modifiersName,
+                                       slap_schema.si_ad_modifyTimestamp,
+                                       NULL
+                               };
+                               Modifications *mod, **modtail, **ml;
+                               int i;
 
-                       /* Append passed in list to ours */
-                       if ( modtail ) {
-                               modtail->sml_next = *modlist;
-                               *modlist = modhead;
-                       } else {
-                               mod = *modlist;
-                       }
+                               for ( mod = op->orr_modlist;
+                                       mod->sml_next;
+                                       mod = mod->sml_next )
+                                       ;
+                               modtail = &mod->sml_next;
 
-                       /* Find end of this list */
-                       for ( ; mod != NULL; mod = mod->sml_next ) {
-                               modtail = mod;
+                               /* pull mod off incoming modlist, append to orr_modlist */
+                               for ( i = 0; opattrs[i]; i++ ) {
+                                       for ( ml = modlist; *ml; ml = &(*ml)->sml_next )
+                                       {
+                                               if ( (*ml)->sml_desc == opattrs[i] ) {
+                                                       mod = *ml;
+                                                       *ml = mod->sml_next;
+                                                       mod->sml_next = NULL;
+                                                       *modtail = mod;
+                                                       modtail = &mod->sml_next;
+                                                       break;
+                                               }
+                                       }
+                               }
                        }
+                       op->o_bd = si->si_wbe;
+                       rc = op->o_bd->be_modrdn( op, &rs_modify );
+                       op->o_tmpfree( op->orr_nnewrdn.bv_val, op->o_tmpmemctx );
+                       op->o_tmpfree( op->orr_newrdn.bv_val, op->o_tmpmemctx );
 
-                       mod = (Modifications *)ch_calloc(1, sizeof(Modifications));
-                       mod->sml_op = LDAP_MOD_REPLACE;
-                       mod->sml_flags = 0;
-                       mod->sml_desc = slap_schema.si_ad_entryUUID;
-                       mod->sml_type = mod->sml_desc->ad_cname;
-                       ber_dupbv( &uuid_bv, &syncUUID_strrep );
-                       ber_bvarray_add( &mod->sml_values, &uuid_bv );
-                       ber_dupbv( &uuid_bv, syncUUID );
-                       ber_bvarray_add( &mod->sml_nvalues, &uuid_bv );
-                       modtail->sml_next = mod;
-                                       
+                       slap_mods_free( op->orr_modlist, 1 );
+                       Debug( LDAP_DEBUG_SYNC,
+                                       "syncrepl_entry: %s be_modrdn (%d)\n", 
+                                       si->si_ridtxt, rc, 0 );
+                       op->o_bd = be;
+                       goto done;
+               }
+               if ( dni.mods ) {
                        op->o_tag = LDAP_REQ_MODIFY;
-                       op->orm_modlist = *modlist;
+                       op->orm_modlist = dni.mods;
+                       op->orm_no_opattrs = 1;
+                       op->o_bd = si->si_wbe;
 
-                       rc = be->be_modify( op, &rs_modify );
+                       rc = op->o_bd->be_modify( op, &rs_modify );
+                       slap_mods_free( op->orm_modlist, 1 );
                        Debug( LDAP_DEBUG_SYNC,
-                                       "syncrepl_entry: %s (%d)\n", 
-                                       "be_modify", rc, 0 );
+                                       "syncrepl_entry: %s be_modify (%d)\n", 
+                                       si->si_ridtxt, rc, 0 );
                        if ( rs_modify.sr_err != LDAP_SUCCESS ) {
                                Debug( LDAP_DEBUG_ANY,
-                                       "syncrepl_entry : be_modify failed (%d)\n",
-                                       rs_modify.sr_err, 0, 0 );
+                                       "syncrepl_entry: %s be_modify failed (%d)\n",
+                                       si->si_ridtxt, rs_modify.sr_err, 0 );
                        }
+                       op->o_bd = be;
+               } else {
+                       Debug( LDAP_DEBUG_SYNC,
+                                       "syncrepl_entry: %s entry unchanged, ignored (%s)\n", 
+                                       si->si_ridtxt, op->o_req_dn.bv_val, 0 );
                }
                goto done;
        case LDAP_SYNC_DELETE :
-               if ( !BER_BVISNULL( &dni.dn )) {
+               if ( !BER_BVISNULL( &dni.dn ) ) {
                        op->o_req_dn = dni.dn;
                        op->o_req_ndn = dni.ndn;
                        op->o_tag = LDAP_REQ_DELETE;
-                       rc = be->be_delete( op, &rs_delete );
+                       op->o_bd = si->si_wbe;
+                       rc = op->o_bd->be_delete( op, &rs_delete );
                        Debug( LDAP_DEBUG_SYNC,
-                                       "syncrepl_entry: %s (%d)\n", 
-                                       "be_delete", rc, 0 );
+                                       "syncrepl_entry: %s be_delete (%d)\n", 
+                                       si->si_ridtxt, rc, 0 );
 
                        while ( rs_delete.sr_err == LDAP_SUCCESS
                                && op->o_delete_glue_parent ) {
                                op->o_delete_glue_parent = 0;
-                               if ( !be_issuffix( op->o_bd, &op->o_req_ndn )) {
+                               if ( !be_issuffix( be, &op->o_req_ndn ) ) {
                                        slap_callback cb = { NULL };
                                        cb.sc_response = slap_null_cb;
                                        dnParent( &op->o_req_ndn, &pdn );
@@ -1985,12 +2156,13 @@ retry_add:;
                                        break;
                                }
                        }
+                       op->o_bd = be;
                }
                goto done;
 
        default :
                Debug( LDAP_DEBUG_ANY,
-                       "syncrepl_entry : unknown syncstate\n", 0, 0, 0 );
+                       "syncrepl_entry: %s unknown syncstate\n", si->si_ridtxt, 0, 0 );
                goto done;
        }
 
@@ -1999,9 +2171,6 @@ done:
                slap_sl_free( syncUUID_strrep.bv_val, op->o_tmpmemctx );
                BER_BVZERO( &syncUUID_strrep );
        }
-       if ( dni.ads ) {
-               op->o_tmpfree( dni.ads, op->o_tmpmemctx );
-       }
        if ( !BER_BVISNULL( &dni.ndn ) ) {
                op->o_tmpfree( dni.ndn.bv_val, op->o_tmpmemctx );
        }
@@ -2026,7 +2195,8 @@ static void
 syncrepl_del_nonpresent(
        Operation *op,
        syncinfo_t *si,
-       BerVarray uuids )
+       BerVarray uuids,
+       struct berval *cookiecsn )
 {
        Backend* be = op->o_bd;
        slap_callback   cb = { NULL };
@@ -2038,6 +2208,7 @@ syncrepl_del_nonpresent(
        AttributeName   an[2];
 
        struct berval pdn = BER_BVNULL;
+       struct berval csn;
 
        op->o_req_dn = si->si_base;
        op->o_req_ndn = si->si_base;
@@ -2078,6 +2249,7 @@ syncrepl_del_nonpresent(
                        slap_uuidstr_from_normalized( &uf.f_av_value, &uuids[i],
                                op->o_tmpmemctx );
                        filter2bv_x( op, op->ors_filter, &op->ors_filterstr );
+                       op->o_tmpfree( uf.f_av_value.bv_val, op->o_tmpmemctx );
                        uf.f_av_value = uuids[i];
                        rc = be->be_search( op, &rs_search );
                        op->o_tmpfree( op->ors_filterstr.bv_val, op->o_tmpmemctx );
@@ -2104,7 +2276,14 @@ syncrepl_del_nonpresent(
 
        if ( !LDAP_LIST_EMPTY( &si->si_nonpresentlist ) ) {
 
-               slap_queue_csn( op, &si->si_syncCookie.ctxcsn );
+               if ( cookiecsn && !BER_BVISNULL( cookiecsn ) ) {
+                       csn = *cookiecsn;
+               } else {
+                       csn = si->si_syncCookie.ctxcsn[0];
+               }
+
+               op->o_bd = si->si_wbe;
+               slap_queue_csn( op, &csn );
 
                np_list = LDAP_LIST_FIRST( &si->si_nonpresentlist );
                while ( np_list != NULL ) {
@@ -2119,8 +2298,8 @@ syncrepl_del_nonpresent(
                        op->o_req_ndn = *np_prev->npe_nname;
                        rc = op->o_bd->be_delete( op, &rs_delete );
                        Debug( LDAP_DEBUG_SYNC,
-                               "syncrepl_del_nonpresent: be_delete %s (%d)\n", 
-                               op->o_req_dn.bv_val, rc, 0 );
+                               "syncrepl_del_nonpresent: %s be_delete %s (%d)\n", 
+                               si->si_ridtxt, op->o_req_dn.bv_val, rc );
 
                        if ( rs_delete.sr_err == LDAP_NOT_ALLOWED_ON_NONLEAF ) {
                                Modifications mod1, mod2;
@@ -2143,13 +2322,14 @@ syncrepl_del_nonpresent(
                                op->o_tag = LDAP_REQ_MODIFY;
                                op->orm_modlist = &mod1;
 
-                               rc = be->be_modify( op, &rs_modify );
+                               rc = op->o_bd->be_modify( op, &rs_modify );
+                               if ( mod2.sml_next ) slap_mods_free( mod2.sml_next, 1 );
                        }
 
                        while ( rs_delete.sr_err == LDAP_SUCCESS &&
                                        op->o_delete_glue_parent ) {
                                op->o_delete_glue_parent = 0;
-                               if ( !be_issuffix( op->o_bd, &op->o_req_ndn )) {
+                               if ( !be_issuffix( be, &op->o_req_ndn ) ) {
                                        slap_callback cb = { NULL };
                                        cb.sc_response = slap_null_cb;
                                        dnParent( &op->o_req_ndn, &pdn );
@@ -2160,7 +2340,7 @@ syncrepl_del_nonpresent(
                                        op->o_bd->be_delete( op, &rs_delete );
                                } else {
                                        break;
-                           }
+                               }
                        }
 
                        op->o_delete_glue_parent = 0;
@@ -2171,6 +2351,7 @@ syncrepl_del_nonpresent(
                }
 
                slap_graduate_commit_csn( op );
+               op->o_bd = be;
 
                op->o_tmpfree( op->o_csn.bv_val, op->o_tmpmemctx );
                BER_BVZERO( &op->o_csn );
@@ -2251,14 +2432,13 @@ syncrepl_add_glue(
        }
 
        while ( ndn.bv_val > e->e_nname.bv_val ) {
-               glue = (Entry *) ch_calloc( 1, sizeof(Entry) );
+               glue = entry_alloc();
                ber_dupbv( &glue->e_name, &dn );
                ber_dupbv( &glue->e_nname, &ndn );
 
-               a = ch_calloc( 1, sizeof( Attribute ));
-               a->a_desc = slap_schema.si_ad_objectClass;
+               a = attr_alloc( slap_schema.si_ad_objectClass );
 
-               a->a_vals = ch_calloc( 3, sizeof( struct berval ));
+               a->a_vals = ch_calloc( 3, sizeof( struct berval ) );
                ber_dupbv( &a->a_vals[0], &gcbva[0] );
                ber_dupbv( &a->a_vals[1], &gcbva[1] );
                ber_dupbv( &a->a_vals[2], &gcbva[2] );
@@ -2268,10 +2448,9 @@ syncrepl_add_glue(
                a->a_next = glue->e_attrs;
                glue->e_attrs = a;
 
-               a = ch_calloc( 1, sizeof( Attribute ));
-               a->a_desc = slap_schema.si_ad_structuralObjectClass;
+               a = attr_alloc( slap_schema.si_ad_structuralObjectClass );
 
-               a->a_vals = ch_calloc( 2, sizeof( struct berval ));
+               a->a_vals = ch_calloc( 2, sizeof( struct berval ) );
                ber_dupbv( &a->a_vals[0], &gcbva[1] );
                ber_dupbv( &a->a_vals[1], &gcbva[2] );
 
@@ -2285,7 +2464,8 @@ syncrepl_add_glue(
                op->ora_e = glue;
                rc = be->be_add ( op, &rs_add );
                if ( rs_add.sr_err == LDAP_SUCCESS ) {
-                       be_entry_release_w( op, glue );
+                       if ( op->ora_e == glue )
+                               be_entry_release_w( op, glue );
                } else {
                /* incl. ALREADY EXIST */
                        entry_free( glue );
@@ -2318,7 +2498,8 @@ syncrepl_add_glue(
        op->ora_e = e;
        rc = be->be_add ( op, &rs_add );
        if ( rs_add.sr_err == LDAP_SUCCESS ) {
-               be_entry_release_w( op, e );
+               if ( op->ora_e == e )
+                       be_entry_release_w( op, e );
        } else {
                entry_free( e );
        }
@@ -2334,27 +2515,58 @@ syncrepl_updateCookie(
        struct sync_cookie *syncCookie )
 {
        Backend *be = op->o_bd;
-       Modifications mod = { { 0 } };
-       struct berval vals[ 2 ];
+       Modifications mod[2];
+       struct berval first = BER_BVNULL;
 
-       int rc;
+       int rc, i, j;
 
        slap_callback cb = { NULL };
        SlapReply       rs_modify = {REP_RESULT};
 
-       mod.sml_op = LDAP_MOD_REPLACE;
-       mod.sml_desc = slap_schema.si_ad_contextCSN;
-       mod.sml_type = mod.sml_desc->ad_cname;
-       mod.sml_values = vals;
-       vals[0] = syncCookie->ctxcsn;
-       BER_BVZERO( &vals[1] );
-
-       slap_queue_csn( op, &syncCookie->ctxcsn );
+       mod[0].sml_op = LDAP_MOD_DELETE;
+       mod[0].sml_desc = slap_schema.si_ad_contextCSN;
+       mod[0].sml_type = mod[0].sml_desc->ad_cname;
+       mod[0].sml_values = NULL;
+       mod[0].sml_nvalues = NULL;
+       mod[0].sml_next = &mod[1];
+
+       mod[1].sml_op = LDAP_MOD_ADD;
+       mod[1].sml_desc = slap_schema.si_ad_contextCSN;
+       mod[1].sml_type = mod[0].sml_desc->ad_cname;
+       mod[1].sml_values = NULL;
+       mod[1].sml_nvalues = NULL;
+       mod[1].sml_next = NULL;
+
+       ldap_pvt_thread_mutex_lock( &si->si_cookieState->cs_mutex );
+
+       for ( i=0; i<syncCookie->numcsns; i++ ) {
+               for ( j=0; j<si->si_cookieState->cs_num; j++ ) {
+                       if ( syncCookie->sids[i] != si->si_cookieState->cs_sids[j] )
+                               continue;
+                       if ( ber_bvcmp( &syncCookie->ctxcsn[i],
+                               &si->si_cookieState->cs_vals[j] ) > 0 ) {
+                               ber_bvarray_add_x( &mod[0].sml_values,
+                                       &si->si_cookieState->cs_vals[j], op->o_tmpmemctx );
+                               ber_bvarray_add_x( &mod[1].sml_values,
+                                       &syncCookie->ctxcsn[i], op->o_tmpmemctx );
+                               if ( BER_BVISNULL( &first ))
+                                       first = syncCookie->ctxcsn[i];
+                       }
+                       break;
+               }
+               /* there was no match for this SID, it's a new CSN */
+               if ( j == si->si_cookieState->cs_num ) {
+                       ber_bvarray_add_x( &mod[1].sml_values,
+                               &syncCookie->ctxcsn[i], op->o_tmpmemctx );
+                       if ( BER_BVISNULL( &first ))
+                               first = syncCookie->ctxcsn[i];
+               }
+       }
+       op->o_bd = si->si_wbe;
+       slap_queue_csn( op, &first );
 
        op->o_tag = LDAP_REQ_MODIFY;
 
-       assert( si->si_rid < 1000 );
-
        cb.sc_response = null_callback;
        cb.sc_private = si;
 
@@ -2364,26 +2576,191 @@ syncrepl_updateCookie(
 
        /* update contextCSN */
        op->o_msgid = SLAP_SYNC_UPDATE_MSGID;
-       op->orm_modlist = &mod;
-       rc = be->be_modify( op, &rs_modify );
+
+       if ( mod[0].sml_values )
+               op->orm_modlist = mod;
+       else
+               op->orm_modlist = &mod[1];
+
+       op->orm_no_opattrs = 1;
+       rc = op->o_bd->be_modify( op, &rs_modify );
        op->o_msgid = 0;
 
        if ( rs_modify.sr_err == LDAP_SUCCESS ) {
                slap_sync_cookie_free( &si->si_syncCookie, 0 );
                slap_dup_sync_cookie( &si->si_syncCookie, syncCookie );
+               /* If we replaced any old values */
+               if ( mod[0].sml_values ) {
+                       for ( i=0; !BER_BVISNULL( &mod[0].sml_values[i] ); i++ ) {
+                               for ( j=0; j<si->si_cookieState->cs_num; j++ ) {
+                                       if ( mod[0].sml_values[i].bv_val !=
+                                               si->si_cookieState->cs_vals[j].bv_val )
+                                               continue;
+                                       ber_bvreplace( &si->si_cookieState->cs_vals[j],
+                                               &mod[1].sml_values[i] );
+                                       break;
+                               }
+                       }
+               } else {
+                       /* Else we just added */
+                       si->si_cookieState->cs_num += syncCookie->numcsns;
+                       value_add( &si->si_cookieState->cs_vals, syncCookie->ctxcsn );
+                       free( si->si_cookieState->cs_sids );
+                       si->si_cookieState->cs_sids = slap_parse_csn_sids(
+                               si->si_cookieState->cs_vals, si->si_cookieState->cs_num );
+               }
+
+               si->si_cookieState->cs_age++;
+               si->si_cookieAge = si->si_cookieState->cs_age;
        } else {
                Debug( LDAP_DEBUG_ANY,
-                       "be_modify failed (%d)\n", rs_modify.sr_err, 0, 0 );
+                       "syncrepl_updateCookie: %s be_modify failed (%d)\n",
+                       si->si_ridtxt, rs_modify.sr_err, 0 );
        }
+       ldap_pvt_thread_mutex_unlock( &si->si_cookieState->cs_mutex );
 
        slap_graduate_commit_csn( op );
-
+       op->o_bd = be;
        op->o_tmpfree( op->o_csn.bv_val, op->o_tmpmemctx );
        BER_BVZERO( &op->o_csn );
+       if ( mod[1].sml_next ) slap_mods_free( mod[1].sml_next, 1 );
+       op->o_tmpfree( mod[1].sml_values, op->o_tmpmemctx );
+       op->o_tmpfree( mod[0].sml_values, op->o_tmpmemctx );
 
        return rc;
 }
 
+static void
+attr_cmp( Operation *op, Attribute *old, Attribute *new,
+       Modifications ***mret, Modifications ***mcur )
+{
+       int i, j;
+       Modifications *mod, **modtail;
+
+       modtail = *mret;
+
+       if ( old ) {
+               int n, o, d, a, *adds, *dels;
+               /* count old and new */
+               for ( o=0; old->a_vals[o].bv_val; o++ ) ;
+               for ( n=0; new->a_vals[n].bv_val; n++ ) ;
+
+               adds = op->o_tmpalloc( sizeof(int) * n, op->o_tmpmemctx );
+               dels = op->o_tmpalloc( sizeof(int) * o, op->o_tmpmemctx );
+               d = 0;
+               a = 0;
+               i = 0;
+               j = 0;
+
+               while ( i < o && j < n ) {
+                       int k;
+                       if ( bvmatch( &old->a_vals[i], &new->a_vals[j] ) ) {
+                               i++;
+                               j++;
+                               continue;
+                       }
+                       for ( k = j + 1; k<n; k++ ) {
+                               if ( bvmatch( &old->a_vals[i], &new->a_vals[k] ) ) {
+                                       break;
+                               }
+                       }
+                       /* an old value was deleted */
+                       if ( k == n ) {
+                               dels[d++] = i++;
+                               continue;
+                       }
+                       for ( k = i + 1; k < o; k++ ) {
+                               if ( bvmatch( &old->a_vals[k], &new->a_vals[j] ) ) {
+                                       break;
+                               }
+                       }
+                       if ( k == o ) {
+                               adds[a++] = j++;
+                       }
+               }
+               while ( i < o )
+                       dels[d++] = i++;
+               while ( j < n )
+                       adds[a++] = j++;
+
+               /* all old values were deleted, just use the replace op */
+               if ( d == o ) {
+                       i = j-1;
+               } else if ( d ) {
+               /* delete some values */
+                       mod = ch_malloc( sizeof( Modifications ) );
+                       mod->sml_op = LDAP_MOD_DELETE;
+                       mod->sml_flags = 0;
+                       mod->sml_desc = old->a_desc;
+                       mod->sml_type = mod->sml_desc->ad_cname;
+                       mod->sml_values = ch_malloc( ( d + 1 ) * sizeof(struct berval) );
+                       if ( old->a_vals != old->a_nvals ) {
+                               mod->sml_nvalues = ch_malloc( ( d + 1 ) * sizeof(struct berval) );
+                       } else {
+                               mod->sml_nvalues = NULL;
+                       }
+                       for ( i = 0; i < d; i++ ) {
+                               ber_dupbv( &mod->sml_values[i], &old->a_vals[dels[i]] );
+                               if ( mod->sml_nvalues ) {
+                                       ber_dupbv( &mod->sml_nvalues[i], &old->a_nvals[dels[i]] );
+                               }
+                       }
+                       BER_BVZERO( &mod->sml_values[i] );
+                       if ( mod->sml_nvalues ) {
+                               BER_BVZERO( &mod->sml_nvalues[i] );
+                       }
+                       *modtail = mod;
+                       modtail = &mod->sml_next;
+                       i = j;
+               }
+               op->o_tmpfree( dels, op->o_tmpmemctx );
+               /* some values were added */
+               if ( a && d < o ) {
+                       mod = ch_malloc( sizeof( Modifications ) );
+                       mod->sml_op = LDAP_MOD_ADD;
+                       mod->sml_flags = 0;
+                       mod->sml_desc = old->a_desc;
+                       mod->sml_type = mod->sml_desc->ad_cname;
+                       mod->sml_values = ch_malloc( ( a + 1 ) * sizeof(struct berval) );
+                       if ( old->a_vals != old->a_nvals ) {
+                               mod->sml_nvalues = ch_malloc( ( a + 1 ) * sizeof(struct berval) );
+                       } else {
+                               mod->sml_nvalues = NULL;
+                       }
+                       for ( i = 0; i < a; i++ ) {
+                               ber_dupbv( &mod->sml_values[i], &new->a_vals[adds[i]] );
+                               if ( mod->sml_nvalues ) {
+                                       ber_dupbv( &mod->sml_nvalues[i], &new->a_nvals[adds[i]] );
+                               }
+                       }
+                       BER_BVZERO( &mod->sml_values[i] );
+                       if ( mod->sml_nvalues ) {
+                               BER_BVZERO( &mod->sml_nvalues[i] );
+                       }
+                       *modtail = mod;
+                       modtail = &mod->sml_next;
+                       i = j;
+               }
+               op->o_tmpfree( adds, op->o_tmpmemctx );
+       } else {
+               /* new attr, just use the new mod */
+               i = 0;
+               j = 1;
+       }
+       /* advance to next element */
+       mod = **mcur;
+       if ( mod ) {
+               if ( i != j ) {
+                       **mcur = mod->sml_next;
+                       *modtail = mod;
+                       modtail = &mod->sml_next;
+               } else {
+                       *mcur = &mod->sml_next;
+               }
+       }
+       *mret = modtail;
+}
+
 static int
 dn_callback(
        Operation*      op,
@@ -2404,72 +2781,130 @@ dn_callback(
                         * in the provider are always propagated.
                         */
                        if ( dni->new_entry ) {
+                               Modifications **modtail, **ml;
                                Attribute *old, *new;
-                               int i;
+                               int i, is_ctx;
+
+                               is_ctx = dn_match( &rs->sr_entry->e_nname,
+                                       &op->o_bd->be_nsuffix[0] );
 
-                               /* Did the DN change? Note that we don't explicitly try to
-                                * discover if the deleteOldRdn argument applies here. It
-                                * would save an unnecessary Modify if we detected it, but
-                                * that's a fair amount of trouble to compare the two attr
-                                * lists in detail. (Just test normalized DN; we ignore
-                                * insignificant changes here.)
+                               /* Did the DN change?
                                 */
-                               if ( !dn_match( &rs->sr_entry->e_nname,
-                                               &dni->new_entry->e_nname ) )
+                               if ( !dn_match( &rs->sr_entry->e_name,
+                                               &dni->new_entry->e_name ) )
                                {
+                                       struct berval oldRDN, oldVal;
+                                       AttributeDescription *ad = NULL;
+                                       Attribute *a;
+
                                        dni->renamed = 1;
+                                       /* See if the oldRDN was deleted */
+                                       dnRdn( &rs->sr_entry->e_nname, &oldRDN );
+                                       oldVal.bv_val = strchr(oldRDN.bv_val, '=') + 1;
+                                       oldVal.bv_len = oldRDN.bv_len - ( oldVal.bv_val -
+                                               oldRDN.bv_val );
+                                       oldRDN.bv_len -= oldVal.bv_len + 2;
+                                       slap_bv2ad( &oldRDN, &ad, &rs->sr_text );
+                                       a = attr_find( dni->new_entry->e_attrs, ad );
+                                       if ( !a || value_find_ex( ad,
+                                               SLAP_MR_ASSERTED_VALUE_NORMALIZED_MATCH |
+                                               SLAP_MR_ATTRIBUTE_VALUE_NORMALIZED_MATCH |
+                                               SLAP_MR_VALUE_OF_SYNTAX, a->a_nvals,
+                                               &oldVal, op->o_tmpmemctx ) != LDAP_SUCCESS )
+                                       {
+                                               dni->delOldRDN = 1;
+                                       }
+                                       /* OK, this was just a modDN, we're done */
+                                       return LDAP_SUCCESS;
                                }
 
-                               for ( i = 0, old = rs->sr_entry->e_attrs;
-                                               old;
-                                               i++, old = old->a_next )
-                                       ;
-
-                               dni->attrs = i;
+                               modtail = &dni->mods;
+                               ml = dni->modlist;
+
+                               /* Make sure new entry is actually newer than old entry */
+                               old = attr_find( rs->sr_entry->e_attrs,
+                                       slap_schema.si_ad_entryCSN );
+                               new = attr_find( dni->new_entry->e_attrs,
+                                       slap_schema.si_ad_entryCSN );
+                               if ( new && old && ber_bvcmp( &old->a_vals[0],
+                                       &new->a_vals[0] ) >= 0 ) {
+                                       Debug( LDAP_DEBUG_SYNC,
+                                               "dn_callback : new entry is older than ours "
+                                               "%s ours %s, new %s\n",
+                                               rs->sr_entry->e_name.bv_val,
+                                               old->a_vals[0].bv_val,
+                                               new->a_vals[0].bv_val );
+                                       return LDAP_SUCCESS;
+                               }
 
                                /* We assume that attributes are saved in the same order
                                 * in the remote and local databases. So if we walk through
                                 * the attributeDescriptions one by one they should match in
-                                * lock step. If not, we signal a change. Otherwise we test
-                                * all the values...
+                                * lock step. If not, look for an add or delete.
                                 */
                                for ( old = rs->sr_entry->e_attrs, new = dni->new_entry->e_attrs;
-                                               old && new;
-                                               old = old->a_next, new = new->a_next )
+                                               old && new; )
                                {
-                                       if ( old->a_desc != new->a_desc ) {
-                                               dni->wasChanged = 1;
-                                               break;
+                                       /* If we've seen this before, use its mod now */
+                                       if ( new->a_flags & SLAP_ATTR_IXADD ) {
+                                               attr_cmp( op, NULL, new, &modtail, &ml );
+                                               new = new->a_next;
+                                               continue;
                                        }
-                                       for ( i = 0; ; i++ ) {
-                                               int nold, nnew;
-                                               nold = BER_BVISNULL( &old->a_vals[i] );
-                                               nnew = BER_BVISNULL( &new->a_vals[i] );
-                                               /* If both are empty, stop looking */
-                                               if ( nold && nnew ) {
-                                                       break;
-                                               }
-                                               /* If they are different, stop looking */
-                                               if ( nold != nnew ) {
-                                                       dni->wasChanged = 1;
-                                                       break;
-                                               }
-                                               if ( ber_bvcmp( &old->a_vals[i], &new->a_vals[i] )) {
-                                                       dni->wasChanged = 1;
-                                                       break;
-                                               }
+                                       /* Skip contextCSN */
+                                       if ( is_ctx && old->a_desc ==
+                                               slap_schema.si_ad_contextCSN ) {
+                                               old = old->a_next;
+                                               continue;
                                        }
-                                       if ( dni->wasChanged ) break;
-                               }
-                               if ( dni->wasChanged ) {
-                                       dni->ads = op->o_tmpalloc( dni->attrs *
-                                               sizeof(AttributeDescription *), op->o_tmpmemctx );
-                                       i = 0;
-                                       for ( old = rs->sr_entry->e_attrs; old; old = old->a_next ) {
-                                               dni->ads[i] = old->a_desc;
-                                               i++;
+
+                                       if ( old->a_desc != new->a_desc ) {
+                                               Modifications *mod;
+                                               Attribute *tmp;
+
+                                               /* If it's just been re-added later,
+                                                * remember that we've seen it.
+                                                */
+                                               tmp = attr_find( new, old->a_desc );
+                                               if ( tmp ) {
+                                                       tmp->a_flags |= SLAP_ATTR_IXADD;
+                                               } else {
+                                                       /* If it's a new attribute, pull it in.
+                                                        */
+                                                       tmp = attr_find( old, new->a_desc );
+                                                       if ( !tmp ) {
+                                                               attr_cmp( op, NULL, new, &modtail, &ml );
+                                                               new = new->a_next;
+                                                               continue;
+                                                       }
+                                                       /* Delete old attr */
+                                                       mod = ch_malloc( sizeof( Modifications ) );
+                                                       mod->sml_op = LDAP_MOD_DELETE;
+                                                       mod->sml_flags = 0;
+                                                       mod->sml_desc = old->a_desc;
+                                                       mod->sml_type = mod->sml_desc->ad_cname;
+                                                       mod->sml_values = NULL;
+                                                       mod->sml_nvalues = NULL;
+                                                       *modtail = mod;
+                                                       modtail = &mod->sml_next;
+                                               }
+                                               old = old->a_next;
+                                               continue;
                                        }
+                                       /* kludge - always update modifiersName so that it
+                                        * stays co-located with the other mod opattrs. But only
+                                        * if we know there are other valid mods.
+                                        */
+                                       if ( old->a_desc == slap_schema.si_ad_modifiersName &&
+                                               dni->mods )
+                                               attr_cmp( op, NULL, new, &modtail, &ml );
+                                       else
+                                               attr_cmp( op, old, new, &modtail, &ml );
+                                       new = new->a_next;
+                                       old = old->a_next;
                                }
+                               *modtail = *ml;
+                               *ml = NULL;
                        }
                }
        } else if ( rs->sr_type == REP_RESULT ) {
@@ -2499,22 +2934,30 @@ nonpresent_callback(
                si->si_presentlist = NULL;
 
        } else if ( rs->sr_type == REP_SEARCH ) {
-               if ( !(si->si_refreshDelete & NP_DELETE_ONE )) {
+               if ( !( si->si_refreshDelete & NP_DELETE_ONE ) ) {
+                       char buf[sizeof("rid=000 not")];
+
                        a = attr_find( rs->sr_entry->e_attrs, slap_schema.si_ad_entryUUID );
 
-                       if ( a )
+                       if ( a ) {
                                present_uuid = avl_find( si->si_presentlist, &a->a_nvals[0],
                                        syncuuid_cmp );
+                       }
+
+                       if ( slap_debug & LDAP_DEBUG_SYNC ) {
+                               sprintf( buf, "%s %s", si->si_ridtxt,
+                                       present_uuid ? "got" : "not" );
+                       }
 
-                       Debug( LDAP_DEBUG_SYNC, "nonpresent_callback: UUID %s, dn %s, %sfound\n",
-                               a ? a->a_vals[0].bv_val : "<missing>", rs->sr_entry->e_name.bv_val, present_uuid ? "" : "not " );
+                       Debug( LDAP_DEBUG_SYNC, "nonpresent_callback: %s UUID %s, dn %s\n",
+                               buf, a ? a->a_vals[0].bv_val : "<missing>", rs->sr_entry->e_name.bv_val );
 
                        if ( a == NULL ) return 0;
                }
 
                if ( present_uuid == NULL ) {
                        np_entry = (struct nonpresent_entry *)
-                               ch_calloc( 1, sizeof( struct nonpresent_entry ));
+                               ch_calloc( 1, sizeof( struct nonpresent_entry ) );
                        np_entry->npe_name = ber_dupbv( NULL, &rs->sr_entry->e_name );
                        np_entry->npe_nname = ber_dupbv( NULL, &rs->sr_entry->e_nname );
                        LDAP_LIST_INSERT_HEAD( &si->si_nonpresentlist, np_entry, npe_link );
@@ -2653,9 +3096,15 @@ syncinfo_free( syncinfo_t *sie )
        if ( sie->si_filterstr.bv_val ) {
                ch_free( sie->si_filterstr.bv_val );
        }
+       if ( sie->si_logfilterstr.bv_val ) {
+               ch_free( sie->si_logfilterstr.bv_val );
+       }
        if ( sie->si_base.bv_val ) {
                ch_free( sie->si_base.bv_val );
        }
+       if ( sie->si_logbase.bv_val ) {
+               ch_free( sie->si_logbase.bv_val );
+       }
        if ( sie->si_attrs ) {
                int i = 0;
                while ( sie->si_attrs[i] != NULL ) {
@@ -2701,7 +3150,7 @@ syncinfo_free( syncinfo_t *sie )
        if ( sie->si_presentlist ) {
            avl_free( sie->si_presentlist, avl_ber_bvfree );
        }
-       while ( !LDAP_LIST_EMPTY( &sie->si_nonpresentlist )) {
+       while ( !LDAP_LIST_EMPTY( &sie->si_nonpresentlist ) ) {
                struct nonpresent_entry* npe;
                npe = LDAP_LIST_FIRST( &sie->si_nonpresentlist );
                LDAP_LIST_REMOVE( npe, npe_link );
@@ -2739,28 +3188,20 @@ syncinfo_free( syncinfo_t *sie )
 #define SLIMITSTR              "sizelimit"
 #define TLIMITSTR              "timelimit"
 #define SYNCDATASTR            "syncdata"
+#define LOGBASESTR             "logbase"
+#define LOGFILTERSTR   "logfilter"
 
 /* FIXME: undocumented */
-#define        LOGBASESTR      "logbase"
-#define LOGFILTERSTR   "logfilter"
-#define OLDAUTHCSTR            "bindprincipal"
 #define EXATTRSSTR             "exattrs"
 #define MANAGEDSAITSTR         "manageDSAit"
 
-/* FIXME: unused */
-#define LASTMODSTR             "lastmod"
-#define LMGENSTR               "gen"
-#define LMNOSTR                        "no"
-#define LMREQSTR               "req"
-#define SRVTABSTR              "srvtab"
-#define SUFFIXSTR              "suffix"
-
 /* mandatory */
 #define GOT_ID                 0x0001
-#define GOT_PROVIDER           0x0002
+#define GOT_PROVIDER   0x0002
+#define        GOT_BASE                0x0004
 
 /* check */
-#define GOT_ALL                        (GOT_ID|GOT_PROVIDER)
+#define GOT_ALL                        (GOT_ID|GOT_PROVIDER|GOT_BASE)
 
 static struct {
        struct berval key;
@@ -2815,6 +3256,7 @@ parse_syncrepl_line(
                                return -1;
                        }
                        si->si_rid = tmp;
+                       sprintf( si->si_ridtxt, IDSTR "=%03d", si->si_rid );
                        gots |= GOT_ID;
                } else if ( !strncasecmp( c->argv[ i ], PROVIDERSTR "=",
                                        STRLENOF( PROVIDERSTR "=" ) ) )
@@ -2826,7 +3268,7 @@ parse_syncrepl_line(
                                        STRLENOF( SCHEMASTR "=" ) ) )
                {
                        val = c->argv[ i ] + STRLENOF( SCHEMASTR "=" );
-                       if ( !strncasecmp( val, "on", STRLENOF( "on" ) )) {
+                       if ( !strncasecmp( val, "on", STRLENOF( "on" ) ) ) {
                                si->si_schemachecking = 1;
                        } else if ( !strncasecmp( val, "off", STRLENOF( "off" ) ) ) {
                                si->si_schemachecking = 0;
@@ -2866,6 +3308,7 @@ parse_syncrepl_line(
                                Debug( LDAP_DEBUG_ANY, "%s: %s.\n", c->log, c->msg, 0 );
                                return -1;
                        }
+                       gots |= GOT_BASE;
                } else if ( !strncasecmp( c->argv[ i ], LOGBASESTR "=",
                                        STRLENOF( LOGBASESTR "=" ) ) )
                {
@@ -2890,8 +3333,8 @@ parse_syncrepl_line(
                {
                        int j;
                        val = c->argv[ i ] + STRLENOF( SCOPESTR "=" );
-                       for ( j=0; !BER_BVISNULL(&scopes[j].key); j++ ) {
-                               if (!strcasecmp( val, scopes[j].key.bv_val )) {
+                       for ( j = 0; !BER_BVISNULL(&scopes[j].key); j++ ) {
+                               if (!strcasecmp( val, scopes[j].key.bv_val ) ) {
                                        si->si_scope = scopes[j].val;
                                        break;
                                }
@@ -2930,11 +3373,11 @@ parse_syncrepl_line(
                                {
                                        if ( strlen(s) == 1 && *s == '*' ) {
                                                si->si_allattrs = 1;
-                                               *(val + ( s - str )) = delimstr[0];
+                                               val[ s - str ] = delimstr[0];
                                        }
                                        if ( strlen(s) == 1 && *s == '+' ) {
                                                si->si_allopattrs = 1;
-                                               *(val + ( s - str )) = delimstr[0];
+                                               val [ s - str ] = delimstr[0];
                                        }
                                }
                                ch_free( str );
@@ -2947,7 +3390,7 @@ parse_syncrepl_line(
                                        STRLENOF( EXATTRSSTR "=" ) ) )
                {
                        val = c->argv[ i ] + STRLENOF( EXATTRSSTR "=" );
-                       if ( !strncasecmp( val, ":include:", STRLENOF(":include:") )) {
+                       if ( !strncasecmp( val, ":include:", STRLENOF(":include:") ) ) {
                                char *attr_fname;
                                attr_fname = ch_strdup( val + STRLENOF(":include:") );
                                si->si_exanlist = file2anlist(
@@ -3058,7 +3501,7 @@ parse_syncrepl_line(
                        int j, k, n;
 
                        val = c->argv[ i ] + STRLENOF( RETRYSTR "=" );
-                       retry_list = (char **) ch_calloc( 1, sizeof( char * ));
+                       retry_list = (char **) ch_calloc( 1, sizeof( char * ) );
                        retry_list[0] = NULL;
 
                        slap_str2clist( &retry_list, val, " ,\t" );
@@ -3075,9 +3518,9 @@ parse_syncrepl_line(
                                ch_free( retry_list );
                                return 1;
                        }
-                       si->si_retryinterval = (time_t *) ch_calloc( n + 1, sizeof( time_t ));
-                       si->si_retrynum = (int *) ch_calloc( n + 1, sizeof( int ));
-                       si->si_retrynum_init = (int *) ch_calloc( n + 1, sizeof( int ));
+                       si->si_retryinterval = (time_t *) ch_calloc( n + 1, sizeof( time_t ) );
+                       si->si_retrynum = (int *) ch_calloc( n + 1, sizeof( int ) );
+                       si->si_retrynum_init = (int *) ch_calloc( n + 1, sizeof( int ) );
                        for ( j = 0; j < n; j++ ) {
                                unsigned long   t;
                                if ( lutil_atoul( &t, retry_list[j*2] ) != 0 ) {
@@ -3174,7 +3617,7 @@ parse_syncrepl_line(
                } else if ( bindconf_parse( c->argv[i], &si->si_bindconf ) ) {
                        snprintf( c->msg, sizeof( c->msg ),
                                "Error: parse_syncrepl_line: "
-                               "unknown keyword \"%s\"\n", c->argv[ i ] );
+                               "unable to parse \"%s\"\n", c->argv[ i ] );
                        Debug( LDAP_DEBUG_ANY, "%s: %s.\n", c->log, c->msg, 0 );
                        return -1;
                }
@@ -3182,7 +3625,10 @@ parse_syncrepl_line(
 
        if ( gots != GOT_ALL ) {
                snprintf( c->msg, sizeof( c->msg ),
-                       "Error: Malformed \"syncrepl\" line in slapd config file" );
+                       "Error: Malformed \"syncrepl\" line in slapd config file, missing%s%s%s",
+                       gots & GOT_ID ? "" : " "IDSTR,
+                       gots & GOT_PROVIDER ? "" : " "PROVIDERSTR,
+                       gots & GOT_BASE ? "" : " "SEARCHBASESTR );
                Debug( LDAP_DEBUG_ANY, "%s: %s.\n", c->log, c->msg, 0 );
                return -1;
        }
@@ -3198,8 +3644,14 @@ add_syncrepl(
        int     rc = 0;
 
        if ( !( c->be->be_search && c->be->be_add && c->be->be_modify && c->be->be_delete ) ) {
-               Debug( LDAP_DEBUG_ANY, "%s: database %s does not support operations "
-                       "required for syncrepl\n", c->log, c->be->be_type, 0 );
+               snprintf( c->msg, sizeof(c->msg), "database %s does not support "
+                       "operations required for syncrepl", c->be->be_type );
+               Debug( LDAP_DEBUG_ANY, "%s: %s\n", c->log, c->msg, 0 );
+               return 1;
+       }
+       if ( BER_BVISEMPTY( &c->be->be_rootdn ) ) {
+               strcpy( c->msg, "rootDN must be defined before syncrepl may be used" );
+               Debug( LDAP_DEBUG_ANY, "%s: %s\n", c->log, c->msg, 0 );
                return 1;
        }
        si = (syncinfo_t *) ch_calloc( 1, sizeof( syncinfo_t ) );
@@ -3217,8 +3669,8 @@ add_syncrepl(
        si->si_base.bv_val = NULL;
        si->si_scope = LDAP_SCOPE_SUBTREE;
        si->si_attrsonly = 0;
-       si->si_anlist = (AttributeName *) ch_calloc( 1, sizeof( AttributeName ));
-       si->si_exanlist = (AttributeName *) ch_calloc( 1, sizeof( AttributeName ));
+       si->si_anlist = (AttributeName *) ch_calloc( 1, sizeof( AttributeName ) );
+       si->si_exanlist = (AttributeName *) ch_calloc( 1, sizeof( AttributeName ) );
        si->si_attrs = NULL;
        si->si_allattrs = 0;
        si->si_allopattrs = 0;
@@ -3233,10 +3685,6 @@ add_syncrepl(
        si->si_slimit = 0;
        si->si_conn_setup = 0;
 
-#ifdef HAVE_TLS
-       si->si_check_tls = 1;
-#endif
-
        si->si_presentlist = NULL;
        LDAP_LIST_INIT( &si->si_nonpresentlist );
        ldap_pvt_thread_mutex_init( &si->si_mutex );
@@ -3244,13 +3692,60 @@ add_syncrepl(
        rc = parse_syncrepl_line( c, si );
 
        if ( rc == 0 ) {
+               /* Must be LDAPv3 because we need controls */
+               switch ( si->si_bindconf.sb_version ) {
+               case 0:
+                       /* not explicitly set */
+                       si->si_bindconf.sb_version = LDAP_VERSION3;
+                       break;
+               case 3:
+                       /* explicitly set */
+                       break;
+               default:
+                       Debug( LDAP_DEBUG_ANY,
+                               "version %d incompatible with syncrepl\n",
+                               si->si_bindconf.sb_version, 0, 0 );
+                       syncinfo_free( si );    
+                       return 1;
+               }
+
                si->si_be = c->be;
-               init_syncrepl( si );
-               si->si_re = ldap_pvt_runqueue_insert( &slapd_rq, si->si_interval,
-                       do_syncrepl, si, "do_syncrepl", c->be->be_suffix[0].bv_val );
-               if ( !si->si_re )
-                       rc = -1;
+               if ( slapMode & SLAP_SERVER_MODE ) {
+                       Listener **l = slapd_get_listeners();
+                       int isMe = 0;
+
+                       /* check if URL points to current server. If so, ignore
+                        * this configuration. We require an exact match. Just
+                        * in case they really want to do this, they can vary
+                        * the case of the URL to allow it.
+                        */
+                       if ( l && !SLAP_DBHIDDEN( c->be ) ) {
+                               int i;
+                               for ( i=0; l[i]; i++ ) {
+                                       if ( bvmatch( &l[i]->sl_url, &si->si_bindconf.sb_uri ) ) {
+                                               isMe = 1;
+                                               break;
+                                       }
+                               }
+                       }
+
+                       if ( !isMe ) {
+                               init_syncrepl( si );
+                               si->si_re = ldap_pvt_runqueue_insert( &slapd_rq,
+                                       si->si_interval, do_syncrepl, si, "do_syncrepl",
+                                       si->si_ridtxt );
+                               if ( si->si_re )
+                                       rc = config_sync_shadow( c ) ? -1 : 0;
+                               else
+                                       rc = -1;
+                       }
+               }
        }
+
+#ifdef HAVE_TLS
+       /* Use main slapd defaults */
+       bindconf_tls_defaults( &si->si_bindconf );
+#endif
        if ( rc < 0 ) {
                Debug( LDAP_DEBUG_ANY, "failed to add syncinfo\n", 0, 0, 0 );
                syncinfo_free( si );    
@@ -3263,6 +3758,13 @@ add_syncrepl(
                if ( !si->si_schemachecking ) {
                        SLAP_DBFLAGS(c->be) |= SLAP_DBFLAG_NO_SCHEMA_CHECK;
                }
+               if ( c->be->be_syncinfo ) {
+                       si->si_cookieState = c->be->be_syncinfo->si_cookieState;
+               } else {
+                       si->si_cookieState = ch_calloc( 1, sizeof( cookie_state ));
+                       ldap_pvt_thread_mutex_init( &si->si_cookieState->cs_mutex );
+               }
+               si->si_next = c->be->be_syncinfo;
                c->be->be_syncinfo = si;
                return 0;
        }
@@ -3282,14 +3784,16 @@ syncrepl_unparse( syncinfo_t *si, struct berval *bv )
        /* temporarily inhibit bindconf from printing URI */
        uri = si->si_bindconf.sb_uri;
        BER_BVZERO( &si->si_bindconf.sb_uri );
+       si->si_bindconf.sb_version = 0;
        bindconf_unparse( &si->si_bindconf, &bc );
        si->si_bindconf.sb_uri = uri;
+       si->si_bindconf.sb_version = LDAP_VERSION3;
 
        ptr = buf;
        ptr += snprintf( ptr, WHATSLEFT, IDSTR "=%03ld " PROVIDERSTR "=%s",
                si->si_rid, si->si_bindconf.sb_uri.bv_val );
        if ( ptr - buf >= sizeof( buf ) ) return;
-       if ( !BER_BVISNULL( &bc )) {
+       if ( !BER_BVISNULL( &bc ) ) {
                if ( WHATSLEFT <= bc.bv_len ) {
                        free( bc.bv_val );
                        return;
@@ -3297,25 +3801,25 @@ syncrepl_unparse( syncinfo_t *si, struct berval *bv )
                ptr = lutil_strcopy( ptr, bc.bv_val );
                free( bc.bv_val );
        }
-       if ( !BER_BVISEMPTY( &si->si_filterstr )) {
+       if ( !BER_BVISEMPTY( &si->si_filterstr ) ) {
                if ( WHATSLEFT <= STRLENOF( " " FILTERSTR "=\"" "\"" ) + si->si_filterstr.bv_len ) return;
                ptr = lutil_strcopy( ptr, " " FILTERSTR "=\"" );
                ptr = lutil_strcopy( ptr, si->si_filterstr.bv_val );
                *ptr++ = '"';
        }
-       if ( !BER_BVISNULL( &si->si_base )) {
+       if ( !BER_BVISNULL( &si->si_base ) ) {
                if ( WHATSLEFT <= STRLENOF( " " SEARCHBASESTR "=\"" "\"" ) + si->si_base.bv_len ) return;
                ptr = lutil_strcopy( ptr, " " SEARCHBASESTR "=\"" );
                ptr = lutil_strcopy( ptr, si->si_base.bv_val );
                *ptr++ = '"';
        }
-       if ( !BER_BVISEMPTY( &si->si_logfilterstr )) {
+       if ( !BER_BVISEMPTY( &si->si_logfilterstr ) ) {
                if ( WHATSLEFT <= STRLENOF( " " LOGFILTERSTR "=\"" "\"" ) + si->si_logfilterstr.bv_len ) return;
                ptr = lutil_strcopy( ptr, " " LOGFILTERSTR "=\"" );
                ptr = lutil_strcopy( ptr, si->si_logfilterstr.bv_val );
                *ptr++ = '"';
        }
-       if ( !BER_BVISNULL( &si->si_logbase )) {
+       if ( !BER_BVISNULL( &si->si_logbase ) ) {
                if ( WHATSLEFT <= STRLENOF( " " LOGBASESTR "=\"" "\"" ) + si->si_logbase.bv_len ) return;
                ptr = lutil_strcopy( ptr, " " LOGBASESTR "=\"" );
                ptr = lutil_strcopy( ptr, si->si_logbase.bv_val );
@@ -3339,7 +3843,7 @@ syncrepl_unparse( syncinfo_t *si, struct berval *bv )
                ptr = lutil_strcopy( ptr, si->si_anfile );
                *ptr++ = '"';
        } else if ( si->si_allattrs || si->si_allopattrs ||
-               ( si->si_anlist && !BER_BVISNULL(&si->si_anlist[0].an_name) ))
+               ( si->si_anlist && !BER_BVISNULL(&si->si_anlist[0].an_name) ) )
        {
                char *old;
 
@@ -3435,26 +3939,59 @@ syncrepl_config( ConfigArgs *c )
        if (c->op == SLAP_CONFIG_EMIT) {
                if ( c->be->be_syncinfo ) {
                        struct berval bv;
-                       syncrepl_unparse( c->be->be_syncinfo, &bv ); 
-                       ber_bvarray_add( &c->rvalue_vals, &bv );
+                       syncinfo_t *si;
+
+                       for ( si = c->be->be_syncinfo; si; si=si->si_next ) {
+                               syncrepl_unparse( si, &bv ); 
+                               ber_bvarray_add( &c->rvalue_vals, &bv );
+                       }
                        return 0;
                }
                return 1;
        } else if ( c->op == LDAP_MOD_DELETE ) {
+               cookie_state *cs = NULL;
                if ( c->be->be_syncinfo ) {
-                       syncinfo_free( c->be->be_syncinfo );
-                       c->be->be_syncinfo = NULL;
+                       syncinfo_t *si, **sip;
+                       int i;
+
+                       cs = c->be->be_syncinfo->si_cookieState;
+                       for ( sip = &c->be->be_syncinfo, i=0; *sip; i++ ) {
+                               si = *sip;
+                               if ( c->valx == -1 || i == c->valx ) {
+                                       *sip = si->si_next;
+                                       /* If the task is currently active, we have to leave
+                                        * it running. It will exit on its own. This will only
+                                        * happen when running on the cn=config DB.
+                                        */
+                                       if ( si->si_re &&
+                                               ldap_pvt_runqueue_isrunning( &slapd_rq, si->si_re ) ) {
+                                               si->si_ctype = 0;
+                                       } else {
+                                               syncinfo_free( si );
+                                       }
+                                       if ( i == c->valx )
+                                               break;
+                               } else {
+                                       sip = &si->si_next;
+                               }
+                       }
+               }
+               if ( !c->be->be_syncinfo ) {
+                       SLAP_DBFLAGS( c->be ) &= ~(SLAP_DBFLAG_SHADOW|SLAP_DBFLAG_SYNC_SHADOW);
+                       if ( cs ) {
+                               ber_bvarray_free( cs->cs_vals );
+                               ldap_pvt_thread_mutex_destroy( &cs->cs_mutex );
+                               ch_free( cs );
+                       }
                }
-               SLAP_DBFLAGS( c->be ) &= ~(SLAP_DBFLAG_SHADOW|SLAP_DBFLAG_SYNC_SHADOW);
                return 0;
        }
-       if ( SLAP_SHADOW( c->be ) ) {
+       if ( SLAP_SLURP_SHADOW( c->be ) ) {
                Debug(LDAP_DEBUG_ANY, "%s: "
                        "syncrepl: database already shadowed.\n",
                        c->log, 0, 0);
                return(1);
-       } else if ( add_syncrepl( c ) ) {
-               return(1);
+       } else {
+               return add_syncrepl( c );
        }
-       return config_sync_shadow( c );
 }