]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/syncrepl.c
declare oc_bvfind_undef()
[openldap] / servers / slapd / syncrepl.c
index 4d7187ced05eb9c523332baeb8f8936deaf89735..fe4e41e47bd9ea917d411ee1f9c1583c35a4d9c5 100644 (file)
 #include <ac/string.h>
 #include <ac/socket.h>
 
-#include "ldap_pvt.h"
 #include "lutil.h"
 #include "slap.h"
 #include "lutil_ldap.h"
 
 #include "ldap_rq.h"
 
+/* FIXME: for ldap_ld_free() */
+#undef ldap_debug
+#include "../../libraries/libldap/ldap-int.h"
+
 #define SYNCREPL_STR   "syncreplxxx"
 #define CN_STR "cn="
 
@@ -38,7 +41,7 @@ static const struct berval slap_syncrepl_cn_bvc = BER_BVC(CN_STR SYNCREPL_STR);
 
 static int syncuuid_cmp( const void *, const void * );
 static void avl_ber_bvfree( void * );
-static void syncrepl_del_nonpresent( Operation *, syncinfo_t * );
+static void syncrepl_del_nonpresent( Operation *, syncinfo_t *, BerVarray );
 
 /* callback functions */
 static int dn_callback( struct slap_op *, struct slap_rep * );
@@ -47,14 +50,11 @@ static int null_callback( struct slap_op *, struct slap_rep * );
 
 static AttributeDescription *sync_descs[4];
 
-struct runqueue_s syncrepl_rq;
-
 void
 init_syncrepl(syncinfo_t *si)
 {
        int i, j, k, l, n;
        char **attrs, **exattrs;
-       ObjectClass *oc;
 
        if ( !sync_descs[0] ) {
                sync_descs[0] = slap_schema.si_ad_objectClass;
@@ -69,7 +69,6 @@ init_syncrepl(syncinfo_t *si)
                attrs = anlist2attrs( si->si_anlist );
 
        if ( attrs ) {
-
                if ( si->si_allattrs ) {
                        i = 0;
                        while ( attrs[i] ) {
@@ -83,7 +82,10 @@ init_syncrepl(syncinfo_t *si)
                                        i++;
                                }
                        }
-                       attrs = ( char ** ) ch_realloc( attrs, (i + 1)*sizeof( char * ));
+                       attrs = ( char ** ) ch_realloc( attrs, (i + 2)*sizeof( char * ) );
+                       attrs[i] = ch_strdup("*");
+                       attrs[i + 1] = NULL;
+
                } else if ( si->si_allopattrs ) {
                        i = 0;
                        while ( attrs[i] ) {
@@ -97,46 +99,68 @@ init_syncrepl(syncinfo_t *si)
                                        i++;
                                }
                        }
-                       attrs = ( char ** ) ch_realloc( attrs, (i + 1)*sizeof( char * ));
+                       attrs = ( char ** ) ch_realloc( attrs, (i + 2)*sizeof( char * ) );
+                       attrs[i] = ch_strdup("+");
+                       attrs[i + 1] = NULL;
                }
 
-               if ( !si->si_allopattrs ) {
-                       for ( i = 0; sync_descs[i] != NULL; i++ ) {
-                               j = 0;
-                               while ( attrs[j] ) {
-                                       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] );
-                                                       attrs[k] = attrs[k+1];
-                                               }
-                                       } else {
-                                               j++;
+               for ( i = 0; sync_descs[i] != NULL; i++ ) {
+                       j = 0;
+                       while ( attrs[j] ) {
+                               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] );
+                                               attrs[k] = attrs[k+1];
                                        }
+                               } else {
+                                       j++;
                                }
-
                        }
-                       for ( n = 0; attrs[ n ] != NULL; n++ ) /* empty */;
+               }
+
+               for ( n = 0; attrs[ n ] != NULL; n++ ) /* empty */;
+
+               if ( si->si_allopattrs ) {
+                       attrs = ( char ** ) ch_realloc( attrs, (n + 2)*sizeof( char * ));
+               } else {
                        attrs = ( char ** ) ch_realloc( attrs, (n + 4)*sizeof( char * ));
-                       if ( attrs == NULL ) {
-                               Debug( LDAP_DEBUG_ANY, "out of memory\n", 0,0,0 );
-                       }
-       
-                       /* Add Attributes */
+               }
+
+               if ( attrs == NULL ) {
+                       Debug( LDAP_DEBUG_ANY, "out of memory\n", 0,0,0 );
+               }
+
+               /* Add Attributes */
+               if ( si->si_allopattrs ) {
+                       attrs[n++] = ch_strdup( sync_descs[0]->ad_cname.bv_val );
+               } else {
                        for ( i = 0; sync_descs[ i ] != NULL; i++ ) {
                                attrs[ n++ ] = ch_strdup ( sync_descs[i]->ad_cname.bv_val );
-                               attrs[ n ] = NULL;
                        }
                }
+               attrs[ n ] = NULL;
 
        } else {
-               attrs = ( char ** ) ch_realloc( attrs, 3 * sizeof( char * ));
-               if ( attrs == NULL ) {
-                       Debug( LDAP_DEBUG_ANY, "out of memory\n", 0,0,0 );
+
+               i = 0;
+               if ( si->si_allattrs == si->si_allopattrs ) {
+                       attrs = (char**) ch_malloc( 3 * sizeof(char*) );
+                       attrs[i++] = ch_strdup( "*" );
+                       attrs[i++] = ch_strdup( "+" );
+               } else if ( si->si_allattrs && !si->si_allopattrs ) {
+                       for ( n = 0; sync_descs[ n ] != NULL; n++ ) ;
+                       attrs = (char**) ch_malloc( (n+1)* sizeof(char*) );
+                       attrs[i++] = ch_strdup( "*" );
+                       for ( j = 1; sync_descs[ j ] != NULL; j++ ) {
+                               attrs[i++] = ch_strdup ( sync_descs[j]->ad_cname.bv_val );
+                       }
+               } else if ( !si->si_allattrs && si->si_allopattrs ) {
+                       attrs = (char**) ch_malloc( 3 * sizeof(char*) );
+                       attrs[i++] = ch_strdup( "+" );
+                       attrs[i++] = ch_strdup( sync_descs[0]->ad_cname.bv_val );
                }
-               attrs[0] = ch_strdup( "*" );
-               attrs[1] = ch_strdup( "+" );
-               attrs[2] = NULL;
+               attrs[i] = NULL;
        }
        
        si->si_attrs = attrs;
@@ -144,6 +168,7 @@ init_syncrepl(syncinfo_t *si)
        exattrs = anlist2attrs( si->si_exanlist );
 
        if ( exattrs ) {
+               for ( n = 0; exattrs[n] != NULL; n++ ) ;
 
                for ( i = 0; sync_descs[i] != NULL; i++ ) {
                        j = 0;
@@ -162,7 +187,8 @@ init_syncrepl(syncinfo_t *si)
 
                for ( i = 0; exattrs[i] != NULL; i++ ) {
                        for ( j = 0; si->si_anlist[j].an_name.bv_val; j++ ) {
-                               if ( oc = si->si_anlist[j].an_oc ) {
+                               ObjectClass     *oc;
+                               if ( ( oc = si->si_anlist[j].an_oc ) ) {
                                        k = 0;
                                        while ( oc->soc_required[k] ) {
                                                if ( !strcmp( exattrs[i],
@@ -181,7 +207,9 @@ init_syncrepl(syncinfo_t *si)
                }
 
                for ( i = 0; exattrs[i] != NULL; i++ ) ;
-               exattrs = (char **) ch_realloc( exattrs, (i + 1)*sizeof(char *));
+
+               if ( i != n )
+                       exattrs = (char **) ch_realloc( exattrs, (i + 1)*sizeof(char *));
        }
 
        si->si_exattrs = exattrs;       
@@ -204,7 +232,7 @@ ldap_sync_search(
        ber_set_option( ber, LBER_OPT_BER_MEMCTX, &ctx );
 
        if ( si->si_syncCookie.octet_str &&
-               si->si_syncCookie.octet_str[0].bv_val )
+               !BER_BVISNULL( &si->si_syncCookie.octet_str[0] ) )
        {
                ber_printf( ber, "{eO}",
                        abs(si->si_type),
@@ -255,7 +283,6 @@ do_syncrep1(
        char syncrepl_cbuf[sizeof(CN_STR SYNCREPL_STR)];
        struct berval syncrepl_cn_bv;
        struct sync_cookie      *sc = NULL;
-       struct sync_cookie      syncCookie = { NULL, -1, NULL };
        struct berval   *psub;
 #ifdef HAVE_TLS
        void    *ssl;
@@ -372,7 +399,7 @@ do_syncrep1(
        /* get syncrepl cookie of shadow replica from subentry */
        assert( si->si_rid < 1000 );
        syncrepl_cn_bv.bv_val = syncrepl_cbuf;
-       syncrepl_cn_bv.bv_len = snprintf(syncrepl_cbuf, sizeof(syncrepl_cbuf),
+       syncrepl_cn_bv.bv_len = snprintf( syncrepl_cbuf, sizeof(syncrepl_cbuf),
                CN_STR "syncrepl%ld", si->si_rid );
        build_new_dn( &op->o_req_ndn, psub, &syncrepl_cn_bv, op->o_tmpmemctx );
        op->o_req_dn = op->o_req_ndn;
@@ -399,7 +426,7 @@ do_syncrep1(
                if ( !cookie ) {
                        /* no stored cookie */
                        if ( sc->ctxcsn == NULL ||
-                                sc->ctxcsn->bv_val == NULL ) {
+                                BER_BVISNULL( sc->ctxcsn ) ) {
                                /* if cmdline cookie does not have ctxcsn */
                                /* component, set it to an initial value */
                                slap_init_sync_cookie_ctxcsn( sc );
@@ -559,6 +586,9 @@ do_syncrep2(
                                ldap_get_entry_controls( si->si_ld, msg, &rctrls );
                                /* we can't work without the control */
                                if ( !rctrls ) {
+                                       Debug( LDAP_DEBUG_ANY, "do_syncrep2: "
+                                               "got search entry without "
+                                               "control\n", 0, 0, 0 );
                                        rc = -1;
                                        goto done;
                                }
@@ -567,21 +597,23 @@ do_syncrep2(
                                ber_scanf( ber, "{em" /*"}"*/, &syncstate, &syncUUID );
                                if ( ber_peek_tag( ber, &len ) == LDAP_TAG_SYNC_COOKIE ) {
                                        ber_scanf( ber, /*"{"*/ "m}", &cookie );
-                                       if ( cookie.bv_val ) {
+                                       if ( !BER_BVISNULL( &cookie ) ) {
                                                struct berval tmp_bv;
                                                ber_dupbv( &tmp_bv, &cookie );
                                                ber_bvarray_add( &syncCookie.octet_str, &tmp_bv );
                                        }
                                        if ( syncCookie.octet_str &&
-                                                       syncCookie.octet_str[0].bv_val )
+                                                       !BER_BVISNULL( &syncCookie.octet_str[0] ) )
+                                       {
                                                slap_parse_sync_cookie( &syncCookie );
+                                       }
                                }
                                if ( syncrepl_message_to_entry( si, op, msg,
                                        &modlist, &entry, syncstate ) == LDAP_SUCCESS ) {
-                                       rc_efree = syncrepl_entry( si, op, entry, modlist,
-                                               syncstate, &syncUUID, &syncCookie_req );
+                                       rc_efree = syncrepl_entry( si, op, entry, &modlist,
+                                               syncstate, &syncUUID, &syncCookie_req, syncCookie.ctxcsn );
                                        if ( syncCookie.octet_str &&
-                                               syncCookie.octet_str[0].bv_val )
+                                               !BER_BVISNULL( &syncCookie.octet_str[0] ) )
                                        {
                                                syncrepl_updateCookie( si, op, psub, &syncCookie );
                                        }
@@ -598,10 +630,12 @@ do_syncrep2(
 
                        case LDAP_RES_SEARCH_REFERENCE:
                                Debug( LDAP_DEBUG_ANY,
-                                       "do_syncrep2 : reference received\n", 0, 0, 0 );
+                                       "do_syncrep2: reference received error\n", 0, 0, 0 );
                                break;
 
                        case LDAP_RES_SEARCH_RESULT:
+                               Debug( LDAP_DEBUG_SYNC,
+                                       "do_syncrep2: LDAP_RES_SEARCH_RESULT\n", 0, 0, 0 );
                                ldap_parse_result( si->si_ld, msg, &err, NULL, NULL, NULL,
                                        &rctrls, 0 );
                                if ( rctrls ) {
@@ -611,13 +645,13 @@ do_syncrep2(
                                        ber_scanf( ber, "{" /*"}"*/);
                                        if ( ber_peek_tag( ber, &len ) == LDAP_TAG_SYNC_COOKIE ) {
                                                ber_scanf( ber, "m", &cookie );
-                                               if ( cookie.bv_val ) {
+                                               if ( !BER_BVISNULL( &cookie ) ) {
                                                        struct berval tmp_bv;
                                                        ber_dupbv( &tmp_bv, &cookie );
                                                        ber_bvarray_add( &syncCookie.octet_str, &tmp_bv);
                                                }
                                                if ( syncCookie.octet_str &&
-                                                       syncCookie.octet_str[0].bv_val )
+                                                       !BER_BVISNULL( &syncCookie.octet_str[0] ) )
                                                {
                                                        slap_parse_sync_cookie( &syncCookie );
                                                }
@@ -639,7 +673,7 @@ do_syncrep2(
                                                &syncCookie_req.ctxcsn[0], &syncCookie.ctxcsn[0],
                                                &text );
                                }
-                               if ( syncCookie.octet_str && syncCookie.octet_str->bv_val &&
+                               if ( syncCookie.octet_str && !BER_BVISNULL( syncCookie.octet_str ) &&
                                        match < 0 && err == LDAP_SUCCESS )
                                {
                                        syncrepl_updateCookie( si, op, psub, &syncCookie );
@@ -655,7 +689,7 @@ do_syncrep2(
                                        if ( refreshDeletes == 0 && match < 0 &&
                                                err == LDAP_SUCCESS )
                                        {
-                                               syncrepl_del_nonpresent( op, si );
+                                               syncrepl_del_nonpresent( op, si, NULL );
                                        } else {
                                                avl_free( si->si_presentlist, avl_ber_bvfree );
                                                si->si_presentlist = NULL;
@@ -669,31 +703,44 @@ do_syncrep2(
                                rc = ldap_parse_intermediate( si->si_ld, msg,
                                        &retoid, &retdata, NULL, 0 );
                                if ( !rc && !strcmp( retoid, LDAP_SYNC_INFO ) ) {
-                                       int             si_refreshDelete = 0;
-                                       int             si_refreshPresent = 0;
                                        ber_init2( ber, retdata, LBER_USE_DER );
 
                                        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", 
+                                                       "LDAP_RES_INTERMEDIATE", 
+                                                       "NEW_COOKIE", "\n" );
                                                ber_scanf( ber, "tm", &tag, &cookie );
                                                break;
                                        case LDAP_TAG_SYNC_REFRESH_DELETE:
-                                               si_refreshDelete = 1;
+                                               Debug( LDAP_DEBUG_SYNC,
+                                                       "do_syncrep2: %s - %s%s\n", 
+                                                       "LDAP_RES_INTERMEDIATE", 
+                                                       "REFRESH_DELETE\n", "\n" );
+                                               si->si_refreshDelete = 1;
                                        case LDAP_TAG_SYNC_REFRESH_PRESENT:
-                                               si_refreshPresent = 1;
+                                               Debug( LDAP_DEBUG_SYNC,
+                                                       "do_syncrep2: %s - %s%s\n", 
+                                                       "LDAP_RES_INTERMEDIATE", 
+                                                       si_tag == LDAP_TAG_SYNC_REFRESH_PRESENT ?
+                                                       "REFRESH_PRESENT" : "REFRESH_DELETE",
+                                                       "\n" );
+                                               si->si_refreshDelete = 1;
+                                               si->si_refreshPresent = 1;
                                                ber_scanf( ber, "t{" /*"}"*/, &tag );
                                                if ( ber_peek_tag( ber, &len ) == LDAP_TAG_SYNC_COOKIE )
                                                {
                                                        ber_scanf( ber, "m", &cookie );
-                                                       if ( cookie.bv_val ) {
+                                                       if ( !BER_BVISNULL( &cookie ) ) {
                                                                struct berval tmp_bv;
                                                                ber_dupbv( &tmp_bv, &cookie );
                                                                ber_bvarray_add( &syncCookie.octet_str,
                                                                        &tmp_bv);
                                                        }
                                                        if ( syncCookie.octet_str &&
-                                                               syncCookie.octet_str[0].bv_val )
+                                                               !BER_BVISNULL( &syncCookie.octet_str[0] ) )
                                                        {
                                                                slap_parse_sync_cookie( &syncCookie );
                                                        }
@@ -706,19 +753,24 @@ do_syncrep2(
                                                ber_scanf( ber, /*"{"*/ "}" );
                                                break;
                                        case LDAP_TAG_SYNC_ID_SET:
+                                               Debug( LDAP_DEBUG_SYNC,
+                                                       "do_syncrep2: %s - %s%s\n", 
+                                                       "LDAP_RES_INTERMEDIATE", 
+                                                       "SYNC_ID_SET",
+                                                       "\n" );
                                                ber_scanf( ber, "t{" /*"}"*/, &tag );
                                                if ( ber_peek_tag( ber, &len ) ==
                                                        LDAP_TAG_SYNC_COOKIE )
                                                {
                                                        ber_scanf( ber, "m", &cookie );
-                                                       if ( cookie.bv_val ) {
+                                                       if ( !BER_BVISNULL( &cookie ) ) {
                                                                struct berval tmp_bv;
                                                                ber_dupbv( &tmp_bv, &cookie );
                                                                ber_bvarray_add( &syncCookie.octet_str,
                                                                        &tmp_bv );
                                                        }
                                                        if ( syncCookie.octet_str &&
-                                                               syncCookie.octet_str[0].bv_val )
+                                                                       !BER_BVISNULL( &syncCookie.octet_str[0] ) )
                                                        {
                                                                slap_parse_sync_cookie( &syncCookie );
                                                        }
@@ -730,19 +782,24 @@ do_syncrep2(
                                                }
                                                ber_scanf( ber, "[W]", &syncUUIDs );
                                                ber_scanf( ber, /*"{"*/ "}" );
-                                               for ( i = 0; syncUUIDs[i].bv_val; 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,
-                                                               (caddr_t) syncuuid_bv,
-                                                               syncuuid_cmp, avl_dup_error );
+                                               if ( refreshDeletes ) {
+                                                       syncrepl_del_nonpresent( op, si, syncUUIDs );
+                                                       ber_bvarray_free_x( syncUUIDs, op->o_tmpmemctx );
+                                               } else {
+                                                       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,
+                                                                       (caddr_t) syncuuid_bv,
+                                                                       syncuuid_cmp, avl_dup_error );
+                                                       }
+                                                       slap_sl_free( syncUUIDs, op->o_tmpmemctx );
                                                }
-                                               slap_sl_free( syncUUIDs, op->o_tmpmemctx );
                                                break;
                                        default:
-                                       Debug( LDAP_DEBUG_ANY,
-                                               "do_syncrep2 : unknown syncinfo tag (%ld)\n",
+                                               Debug( LDAP_DEBUG_ANY,
+                                                       "do_syncrep2 : unknown syncinfo tag (%ld)\n",
                                                (long) si_tag, 0, 0 );
                                                ldap_memfree( retoid );
                                                ber_bvfree( retdata );
@@ -761,15 +818,15 @@ do_syncrep2(
                                                        &syncCookie.ctxcsn[0], &text );
                                        }
 
-                                       if ( syncCookie.ctxcsn && syncCookie.ctxcsn[0].bv_val &&
+                                       if ( syncCookie.ctxcsn && !BER_BVISNULL( &syncCookie.ctxcsn[0] ) &&
                                                match < 0 )
                                        {
                                                syncrepl_updateCookie( si, op, psub, &syncCookie);
                                        }
 
-                                       if ( si_refreshPresent == 1 ) {
+                                       if ( si->si_refreshPresent == 1 ) {
                                                if ( match < 0 ) {
-                                                       syncrepl_del_nonpresent( op, si );
+                                                       syncrepl_del_nonpresent( op, si, NULL );
                                                }
                                        } 
 
@@ -835,7 +892,8 @@ do_syncrepl(
        struct re_s* rtask = arg;
        syncinfo_t *si = ( syncinfo_t * ) rtask->arg;
        Connection conn = {0};
-       Operation op = {0};
+       char opbuf[OPERATION_BUFFER_SIZE];
+       Operation *op;
        int rc = LDAP_SUCCESS;
        int first = 0;
        int dostop = 0;
@@ -864,35 +922,31 @@ do_syncrepl(
                return NULL;
        }
 
-       connection_fake_init( &conn, &op, ctx );
+       op = (Operation *)opbuf;
+       connection_fake_init( &conn, op, ctx );
 
        /* use global malloc for now */
-       op.o_tmpmemctx = NULL;
-       op.o_tmpmfuncs = &ch_mfuncs;
-
-       op.o_dn = si->si_updatedn;
-       op.o_ndn = si->si_updatedn;
-       op.o_managedsait = 1;
-       op.o_bd = be = si->si_be;
+       op->o_tmpmemctx = NULL;
+       op->o_tmpmfuncs = &ch_mfuncs;
 
-       op.o_sync_state.ctxcsn = NULL;
-       op.o_sync_state.sid = -1;
-       op.o_sync_state.octet_str = NULL;
-       op.o_sync_slog_size = -1;
-       LDAP_STAILQ_FIRST( &op.o_sync_slog_list ) = NULL;
-       op.o_sync_slog_list.stqh_last = &LDAP_STAILQ_FIRST(&op.o_sync_slog_list);
+       op->o_dn = si->si_updatedn;
+       op->o_ndn = si->si_updatedn;
+       op->o_managedsait = SLAP_CONTROL_NONCRITICAL;
+       op->o_bd = be = si->si_be;
 
        /* Establish session, do search */
        if ( !si->si_ld ) {
                first = 1;
-               rc = do_syncrep1( &op, si );
+               si->si_refreshDelete = 0;
+               si->si_refreshPresent = 0;
+               rc = do_syncrep1( op, si );
        }
 
        /* Process results */
        if ( rc == LDAP_SUCCESS ) {
                ldap_get_option( si->si_ld, LDAP_OPT_DESC, &s );
 
-               rc = do_syncrep2( &op, si );
+               rc = do_syncrep2( op, si );
 
                if ( abs(si->si_type) == LDAP_SYNC_REFRESH_AND_PERSIST ) {
                        /* If we succeeded, enable the connection for further listening.
@@ -918,10 +972,10 @@ do_syncrepl(
         * 3) for Refresh and Success, reschedule to run
         * 4) for Persist and Success, reschedule to defer
         */
-       ldap_pvt_thread_mutex_lock( &syncrepl_rq.rq_mutex );
+       ldap_pvt_thread_mutex_lock( &slapd_rq.rq_mutex );
 
-       if ( ldap_pvt_runqueue_isrunning( &syncrepl_rq, rtask )) {
-               ldap_pvt_runqueue_stoptask( &syncrepl_rq, rtask );
+       if ( ldap_pvt_runqueue_isrunning( &slapd_rq, rtask )) {
+               ldap_pvt_runqueue_stoptask( &slapd_rq, rtask );
        }
 
        if ( dostop ) {
@@ -933,7 +987,7 @@ do_syncrepl(
                        defer = 0;
                }
                rtask->interval.tv_sec = si->si_interval;
-               ldap_pvt_runqueue_resched( &syncrepl_rq, rtask, defer );
+               ldap_pvt_runqueue_resched( &slapd_rq, rtask, defer );
                if ( si->si_retrynum ) {
                        for ( i = 0; si->si_retrynum_init[i] != -2; i++ ) {
                                si->si_retrynum[i] = si->si_retrynum_init[i];
@@ -947,19 +1001,19 @@ do_syncrepl(
                }
 
                if ( !si->si_retrynum || si->si_retrynum[i] == -2 ) {
-                       ldap_pvt_runqueue_remove( &syncrepl_rq, rtask );
+                       ldap_pvt_runqueue_remove( &slapd_rq, rtask );
                        LDAP_STAILQ_REMOVE( &be->be_syncinfo, si, syncinfo_s, si_next );
                        syncinfo_free( si );
                } else if ( si->si_retrynum[i] >= -1 ) {
                        if ( si->si_retrynum[i] > 0 )
                                si->si_retrynum[i]--;
                        rtask->interval.tv_sec = si->si_retryinterval[i];
-                       ldap_pvt_runqueue_resched( &syncrepl_rq, rtask, 0 );
+                       ldap_pvt_runqueue_resched( &slapd_rq, rtask, 0 );
                        slap_wake_listener();
                }
        }
        
-       ldap_pvt_thread_mutex_unlock( &syncrepl_rq.rq_mutex );
+       ldap_pvt_thread_mutex_unlock( &slapd_rq.rq_mutex );
 
        return NULL;
 }
@@ -1028,7 +1082,7 @@ syncrepl_message_to_entry(
 
        while ( ber_remaining( ber ) ) {
                if ( (ber_scanf( ber, "{mW}", &tmp.sml_type, &tmp.sml_values ) ==
-                       LBER_ERROR ) || ( tmp.sml_type.bv_val == NULL ))
+                       LBER_ERROR ) || BER_BVISNULL( &tmp.sml_type ) )
                {
                        break;
                }
@@ -1104,15 +1158,44 @@ done:
        return rc;
 }
 
+/* During a refresh, we may get an LDAP_SYNC_ADD for an already existing
+ * entry if a previous refresh was interrupted before sending us a new
+ * context state. We try to compare the new entry to the existing entry
+ * and ignore the new entry if they are the same.
+ *
+ * Also, we may get an update where the entryDN has changed, due to
+ * a ModDn on the provider. We detect this as well, so we can issue
+ * the corresponding operation locally.
+ *
+ * In the case of a modify, we get a list of all the attributes
+ * in the original entry. Rather than deleting the entry and re-adding it,
+ * we issue a Modify request that deletes all the attributes and adds all
+ * 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.
+ */
+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;
+} dninfo;
+
 int
 syncrepl_entry(
        syncinfo_t* si,
        Operation *op,
        Entry* entry,
-       Modifications* modlist,
+       Modifications** modlist,
        int syncstate,
        struct berval* syncUUID,
-       struct sync_cookie* syncCookie_req )
+       struct sync_cookie* syncCookie_req,
+       struct berval* syncCSN )
 {
        Backend *be = op->o_bd;
        slap_callback   cb = { NULL };
@@ -1128,7 +1211,6 @@ syncrepl_entry(
        AttributeAssertion ava = {0};
        int rc = LDAP_SUCCESS;
        int ret = LDAP_SUCCESS;
-       const char *text;
 
        struct berval pdn = BER_BVNULL;
        struct berval org_req_dn = BER_BVNULL;
@@ -1136,11 +1218,42 @@ syncrepl_entry(
        struct berval org_dn = BER_BVNULL;
        struct berval org_ndn = BER_BVNULL;
        int     org_managedsait;
+       dninfo dni = {0};
+       int     retry = 1;
+
+       switch( syncstate ) {
+       case LDAP_SYNC_PRESENT:
+               Debug( LDAP_DEBUG_SYNC, "%s: %s\n",
+                                       "syncrepl_entry",
+                                       "LDAP_RES_SEARCH_ENTRY(LDAP_SYNC_PRESENT)", 0 );
+               break;
+       case LDAP_SYNC_ADD:
+               Debug( LDAP_DEBUG_SYNC, "%s: %s\n",
+                                       "syncrepl_entry",
+                                       "LDAP_RES_SEARCH_ENTRY(LDAP_SYNC_ADD)", 0 );
+               break;
+       case LDAP_SYNC_DELETE:
+               Debug( LDAP_DEBUG_SYNC, "%s: %s\n",
+                                       "syncrepl_entry",
+                                       "LDAP_RES_SEARCH_ENTRY(LDAP_SYNC_DELETE)", 0 );
+               break;
+       case LDAP_SYNC_MODIFY:
+               Debug( LDAP_DEBUG_SYNC, "%s: %s\n",
+                                       "syncrepl_entry",
+                                       "LDAP_RES_SEARCH_ENTRY(LDAP_SYNC_MODIFY)", 0 );
+               break;
+       default:
+               Debug( LDAP_DEBUG_ANY, "%s: %s\n",
+                                       "syncrepl_entry",
+                                       "LDAP_RES_SEARCH_ENTRY(UNKNOWN syncstate)", 0 );
+       }
 
        if (( syncstate == LDAP_SYNC_PRESENT || syncstate == LDAP_SYNC_ADD )) {
-               syncuuid_bv = ber_dupbv( NULL, syncUUID );
-               avl_insert( &si->si_presentlist, (caddr_t) syncuuid_bv,
-                       syncuuid_cmp, avl_dup_error );
+               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 ( syncstate == LDAP_SYNC_PRESENT ) {
@@ -1154,22 +1267,22 @@ syncrepl_entry(
        f.f_choice = LDAP_FILTER_EQUALITY;
        f.f_ava = &ava;
        ava.aa_desc = slap_schema.si_ad_entryUUID;
-       slap_uuidstr_from_normalized( &syncUUID_strrep, syncUUID, op->o_tmpmemctx );
+       (void)slap_uuidstr_from_normalized( &syncUUID_strrep, syncUUID, op->o_tmpmemctx );
        ava.aa_value = *syncUUID;
        op->ors_filter = &f;
 
-       op->ors_filterstr.bv_len = (sizeof("entryUUID=")-1) + syncUUID->bv_len;
+       op->ors_filterstr.bv_len = STRLENOF( "entryUUID=" ) + syncUUID->bv_len;
        op->ors_filterstr.bv_val = (char *) slap_sl_malloc(
                op->ors_filterstr.bv_len + 1, op->o_tmpmemctx ); 
-       AC_MEMCPY( op->ors_filterstr.bv_val, "entryUUID=", sizeof("entryUUID=")-1 );
-       AC_MEMCPY( &op->ors_filterstr.bv_val[sizeof("entryUUID=")-1],
+       AC_MEMCPY( op->ors_filterstr.bv_val, "entryUUID=", STRLENOF( "entryUUID=" ) );
+       AC_MEMCPY( &op->ors_filterstr.bv_val[STRLENOF( "entryUUID=" )],
                syncUUID->bv_val, syncUUID->bv_len );
        op->ors_filterstr.bv_val[op->ors_filterstr.bv_len] = '\0';
 
        op->o_tag = LDAP_REQ_SEARCH;
        op->ors_scope = LDAP_SCOPE_SUBTREE;
 
-       /* get syncrepl cookie of shadow replica from subentry */
+       /* get the entry for this UUID */
        op->o_req_dn = si->si_base;
        op->o_req_ndn = si->si_base;
 
@@ -1177,159 +1290,264 @@ syncrepl_entry(
        op->ors_tlimit = SLAP_NO_LIMIT;
        op->ors_slimit = 1;
 
-       op->ors_attrs = slap_anlist_no_attrs;
-       op->ors_attrsonly = 1;
+       op->ors_attrs = slap_anlist_all_attributes;
+       op->ors_attrsonly = 0;
 
        /* set callback function */
        op->o_callback = &cb;
        cb.sc_response = dn_callback;
-       cb.sc_private = si;
-
-       si->si_syncUUID_ndn.bv_val = NULL;
+       cb.sc_private = &dni;
+       dni.new_entry = entry;
 
        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 );
        }
 
-       if ( op->ors_filterstr.bv_val ) {
+       if ( !BER_BVISNULL( &op->ors_filterstr ) ) {
                slap_sl_free( op->ors_filterstr.bv_val, op->o_tmpmemctx );
        }
 
        cb.sc_response = null_callback;
        cb.sc_private = si;
 
-       if ( rs_search.sr_err == LDAP_SUCCESS && si->si_syncUUID_ndn.bv_val ) {
-               char *subseq_ptr;
-
-               if ( syncstate != LDAP_SYNC_DELETE ) {
-                       op->o_no_psearch = 1;
-               }
-
-               ber_dupbv( &op->o_sync_csn, syncCookie_req->ctxcsn );
-               if ( op->o_sync_csn.bv_val ) {
-                       subseq_ptr = strstr( op->o_sync_csn.bv_val, "#0000" );
-                       subseq_ptr += 4;
-                       *subseq_ptr = '1';
-               }
-               
-               op->o_req_dn = si->si_syncUUID_ndn;
-               op->o_req_ndn = si->si_syncUUID_ndn;
-               op->o_tag = LDAP_REQ_DELETE;
-               rc = be->be_delete( op, &rs_delete );
-
-               org_req_dn = op->o_req_dn;
-               org_req_ndn = op->o_req_ndn;
-               org_dn = op->o_dn;
-               org_ndn = op->o_ndn;
-               org_managedsait = get_manageDSAit( op );
-               op->o_dn = op->o_bd->be_rootdn;
-               op->o_ndn = op->o_bd->be_rootndn;
-               op->o_managedsait = 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 )) {
-                               slap_callback cb = { NULL };
-                               cb.sc_response = slap_null_cb;
-                               dnParent( &op->o_req_ndn, &pdn );
-                               op->o_req_dn = pdn;
-                               op->o_req_ndn = pdn;
-                               op->o_callback = &cb;
-                               op->o_bd->be_delete( op, &rs_delete );
-                       } else {
-                               break;
-                   }
-               }
+       if ( entry && entry->e_name.bv_val ) {
+               Debug( LDAP_DEBUG_SYNC,
+                               "syncrepl_entry: %s\n",
+                               entry->e_name.bv_val, 0, 0 );
+       } else {
+               Debug( LDAP_DEBUG_SYNC,
+                               "syncrepl_entry: %s\n",
+                               dni.dn.bv_val ? dni.dn.bv_val : "(null)", 0, 0 );
+       }
 
-               op->o_managedsait = org_managedsait;
-               op->o_dn = org_dn;
-               op->o_ndn = org_ndn;
-               op->o_req_dn = org_req_dn;
-               op->o_req_ndn = org_req_ndn;
-               op->o_delete_glue_parent = 0;
+       org_req_dn = op->o_req_dn;
+       org_req_ndn = op->o_req_ndn;
+       org_dn = op->o_dn;
+       org_ndn = op->o_ndn;
+       org_managedsait = get_manageDSAit( op );
+       op->o_dn = op->o_bd->be_rootdn;
+       op->o_ndn = op->o_bd->be_rootndn;
+       op->o_managedsait = SLAP_CONTROL_NONCRITICAL;
 
-               op->o_no_psearch = 0;
+       if ( syncstate != LDAP_SYNC_DELETE ) {
+               attr_delete( &entry->e_attrs, slap_schema.si_ad_entryUUID );
+               attr_merge_one( entry, slap_schema.si_ad_entryUUID,
+                       &syncUUID_strrep, syncUUID );
        }
 
        switch ( syncstate ) {
        case LDAP_SYNC_ADD:
        case LDAP_SYNC_MODIFY:
-               if ( rs_search.sr_err == LDAP_SUCCESS ||
-                        rs_search.sr_err == LDAP_REFERRAL ||
-                        rs_search.sr_err == LDAP_NO_SUCH_OBJECT ||
-                        rs_search.sr_err == LDAP_NOT_ALLOWED_ON_NONLEAF )
-               {
-                       attr_delete( &entry->e_attrs, slap_schema.si_ad_entryUUID );
-                       attr_merge_one( entry, slap_schema.si_ad_entryUUID,
-                               &syncUUID_strrep, syncUUID );
+retry_add:;
+               if ( BER_BVISNULL( &dni.dn )) {
 
-                       op->o_tag = LDAP_REQ_ADD;
-                       op->ora_e = entry;
                        op->o_req_dn = entry->e_name;
                        op->o_req_ndn = entry->e_nname;
+                       op->o_tag = LDAP_REQ_ADD;
+                       op->ora_e = entry;
 
                        rc = be->be_add( op, &rs_add );
+                       Debug( LDAP_DEBUG_SYNC,
+                                       "syncrepl_entry: %s (%d)\n", 
+                                       "be_add", rc, 0 );
+                       switch ( rs_add.sr_err ) {
+                       case LDAP_SUCCESS:
+                               be_entry_release_w( op, entry );
+                               ret = 0;
+                               break;
 
-                       if ( rs_add.sr_err != LDAP_SUCCESS ) {
-                               if ( rs_add.sr_err == LDAP_ALREADY_EXISTS &&
-                                        rs_search.sr_err != LDAP_NO_SUCH_OBJECT ) {
-                                       Modifications *mod;
-                                       Modifications *modtail = modlist;
-
-                                       assert( modlist );
+                       case LDAP_REFERRAL:
+                       /* we assume that LDAP_NO_SUCH_OBJECT is returned 
+                        * only if the suffix entry is not present */
+                       case LDAP_NO_SUCH_OBJECT:
+                               syncrepl_add_glue( op, entry );
+                               ret = 0;
+                               break;
 
-                                       for ( mod = modlist; mod != NULL; mod = mod->sml_next ) {
-                                               modtail = mod;
-                                       }
+                       /* if an entry was added via syncrepl_add_glue(),
+                        * it likely has no entryUUID, so the previous
+                        * be_search() doesn't find it.  In this case,
+                        * give syncrepl a chance to modify it. */
+                       case LDAP_ALREADY_EXISTS:
+                               if ( retry ) {
+                                       Operation       op2 = *op;
+                                       SlapReply       rs2 = { 0 };
+                                       slap_callback   cb2 = { 0 };
+
+                                       op2.o_tag = LDAP_REQ_SEARCH;
+                                       ber_dupbv_x( &op2.o_req_dn,
+                                               &entry->e_name,
+                                               op2.o_tmpmemctx );
+                                       ber_dupbv_x( &op2.o_req_ndn,
+                                               &entry->e_nname,
+                                               op2.o_tmpmemctx );
+                                       op2.ors_scope = LDAP_SCOPE_BASE;
+                                       op2.ors_attrs = slap_anlist_all_attributes;
+                                       op2.ors_attrsonly = 0;
+                                       op2.ors_limit = NULL;
+                                       op2.ors_slimit = 1;
+                                       op2.ors_tlimit = SLAP_NO_LIMIT;
+
+                                       f.f_choice = LDAP_FILTER_EQUALITY;
+                                       f.f_ava = &ava;
+                                       ava.aa_desc = slap_schema.si_ad_objectClass;
+                                       ber_dupbv_x( &ava.aa_value,
+                                               &slap_schema.si_oc_glue->soc_cname,
+                                               op2.o_tmpmemctx );
+                                       op2.ors_filter = &f;
+                                       filter2bv_x( &op2, op2.ors_filter,
+                                                       &op2.ors_filterstr );
+
+                                       op2.o_callback = &cb2;
+                                       cb2.sc_response = dn_callback;
+                                       cb2.sc_private = &dni;
+
+                                       be->be_search( &op2, &rs2 );
+
+                                       op2.o_tmpfree( op2.o_req_dn.bv_val,
+                                               op2.o_tmpmemctx );
+                                       op2.o_tmpfree( op2.o_req_ndn.bv_val,
+                                               op2.o_tmpmemctx );
+                                       op2.o_tmpfree( ava.aa_value.bv_val,
+                                               op2.o_tmpmemctx );
+                                       op2.o_tmpfree( op2.ors_filterstr.bv_val,
+                                               op2.o_tmpmemctx );
+
+                                       retry = 0;
+                                       goto retry_add;
+                               }
+                               /* FALLTHRU */
 
-                                       mod = (Modifications *)ch_calloc(1, sizeof(Modifications));
-                                       ber_dupbv( &uuid_bv, syncUUID );
-                                       mod->sml_op = LDAP_MOD_REPLACE;
-                                       mod->sml_desc = slap_schema.si_ad_entryUUID;
-                                       mod->sml_type = mod->sml_desc->ad_cname;
-                                       ber_bvarray_add( &mod->sml_values, &uuid_bv );
+                       default:
+                               Debug( LDAP_DEBUG_ANY,
+                                       "syncrepl_entry : be_add failed (%d)\n",
+                                       rs_add.sr_err, 0, 0 );
+                               ret = 1;
+                               break;
+                       }
+                       goto done;
+               }
+               /* FALLTHRU */
+               op->o_req_dn = dni.dn;
+               op->o_req_ndn = dni.ndn;
+               if ( dni.renamed ) {
+                       struct berval noldp, newp, nnewp;
+
+                       op->o_tag = LDAP_REQ_MODRDN;
+                       dnRdn( &entry->e_name, &op->orr_newrdn );
+                       dnRdn( &entry->e_nname, &op->orr_nnewrdn );
+
+                       dnParent( &dni.ndn, &noldp );
+                       dnParent( &entry->e_nname, &nnewp );
+                       if ( !dn_match( &noldp, &newp )) {
+                               dnParent( &entry->e_name, &newp );
+                               op->orr_newSup = &newp;
+                               op->orr_nnewSup = &nnewp;
+                       }
+                       op->orr_deleteoldrdn = 0;
+                       rc = be->be_modrdn( op, &rs_modify );
+                       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 {
+                               ret = 1;
+                               goto done;
+                       }
+               }
+               if ( dni.wasChanged ) {
+                       Modifications *mod, *modhead = NULL;
+                       Modifications *modtail = NULL;
+                       int i;
+
+                       op->o_tag = LDAP_REQ_MODIFY;
+
+                       assert( *modlist );
+
+                       /* 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_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;
+                       }
+
+                       /* Append passed in list to ours */
+                       if ( modtail ) {
+                               modtail->sml_next = *modlist;
+                               *modlist = modhead;
+                       } else {
+                               mod = *modlist;
+                       }
+
+                       /* Find end of this list */
+                       for ( ; mod != NULL; mod = mod->sml_next ) {
+                               modtail = mod;
+                       }
+
+                       mod = (Modifications *)ch_calloc(1, sizeof(Modifications));
+                       mod->sml_op = LDAP_MOD_REPLACE;
+                       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;
                                        
-                                       op->o_tag = LDAP_REQ_MODIFY;
-                                       op->orm_modlist = modlist;
-                                       op->o_req_dn = entry->e_name;
-                                       op->o_req_ndn = entry->e_nname;
+                       op->o_tag = LDAP_REQ_MODIFY;
+                       op->orm_modlist = *modlist;
+
+                       rc = be->be_modify( op, &rs_modify );
+                       Debug( LDAP_DEBUG_SYNC,
+                                       "syncrepl_entry: %s (%d)\n", 
+                                       "be_modify", 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 );
+                       }
+               }
+               ret = 1;
+               goto done;
+       case LDAP_SYNC_DELETE :
+               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 );
+                       Debug( LDAP_DEBUG_SYNC,
+                                       "syncrepl_entry: %s (%d)\n", 
+                                       "be_delete", rc, 0 );
 
-                                       rc = be->be_modify( op, &rs_modify );
-                                       if ( rs_modify.sr_err != LDAP_SUCCESS ) {
-                                               Debug( LDAP_DEBUG_ANY,
-                                                       "syncrepl_entry : be_modify failed (%d)\n",
-                                                       rs_modify.sr_err, 0, 0 );
-                                       }
-                                       ret = 1;
-                                       goto done;
-                               } else if ( rs_modify.sr_err == LDAP_REFERRAL ||
-                                                       rs_modify.sr_err == LDAP_NO_SUCH_OBJECT ) {
-                                       syncrepl_add_glue( op, entry );
-                                       ret = 0;
-                                       goto done;
+                       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 )) {
+                                       slap_callback cb = { NULL };
+                                       cb.sc_response = slap_null_cb;
+                                       dnParent( &op->o_req_ndn, &pdn );
+                                       op->o_req_dn = pdn;
+                                       op->o_req_ndn = pdn;
+                                       op->o_callback = &cb;
+                                       op->o_bd->be_delete( op, &rs_delete );
                                } else {
-                                       Debug( LDAP_DEBUG_ANY,
-                                               "syncrepl_entry : be_add failed (%d)\n",
-                                               rs_add.sr_err, 0, 0 );
-                                       ret = 1;
-                                       goto done;
+                                       break;
                                }
-                       } else {
-                               be_entry_release_w( op, entry );
-                               ret = 0;
-                               goto done;
                        }
-               } else {
-                       Debug( LDAP_DEBUG_ANY,
-                               "syncrepl_entry : be_search failed (%d)\n",
-                               rs_search.sr_err, 0, 0 );
-                       ret = 1;
-                       goto done;
                }
-
-       case LDAP_SYNC_DELETE :
-               /* Already deleted */
                ret = 0;
                goto done;
 
@@ -1341,13 +1559,18 @@ syncrepl_entry(
        }
 
 done :
-
-       if ( syncUUID_strrep.bv_val ) {
+       if ( !BER_BVISNULL( &syncUUID_strrep ) ) {
                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 );
        }
-       if ( si->si_syncUUID_ndn.bv_val ) {
-               ch_free( si->si_syncUUID_ndn.bv_val );
-               si->si_syncUUID_ndn.bv_val = NULL;
+       if ( !BER_BVISNULL( &dni.dn ) ) {
+               op->o_tmpfree( dni.dn.bv_val, op->o_tmpmemctx );
        }
        return ret;
 }
@@ -1358,10 +1581,13 @@ static struct berval gcbva[] = {
        BER_BVNULL
 };
 
+#define NP_DELETE_ONE  2
+
 static void
 syncrepl_del_nonpresent(
        Operation *op,
-       syncinfo_t *si )
+       syncinfo_t *si,
+       BerVarray uuids )
 {
        Backend* be = op->o_bd;
        slap_callback   cb = { NULL };
@@ -1374,8 +1600,7 @@ syncrepl_del_nonpresent(
        Modifications *mlnext;
        Modifications *mod;
        Modifications *modlist = NULL;
-       Modifications **modtail = &modlist;
-       Attribute       *attr;
+       Modifications **modtail;
        AttributeName   an[2];
 
        struct berval pdn = BER_BVNULL;
@@ -1397,29 +1622,52 @@ syncrepl_del_nonpresent(
        op->ors_deref = LDAP_DEREF_NEVER;
        op->o_time = slap_get_time();
        op->ors_tlimit = SLAP_NO_LIMIT;
-       op->ors_slimit = SLAP_NO_LIMIT;
 
-       memset( &an[0], 0, 2 * sizeof( AttributeName ) );
-       an[0].an_name = slap_schema.si_ad_entryUUID->ad_cname;
-       an[0].an_desc = slap_schema.si_ad_entryUUID;
-       op->ors_attrs = an;
 
-       op->ors_attrsonly = 0;
-       op->ors_filter = str2filter_x( op, si->si_filterstr.bv_val );
-       op->ors_filterstr = si->si_filterstr;
+       if ( uuids ) {
+               Filter uf;
+               AttributeAssertion eq;
+               int i;
 
-       op->o_nocaching = 1;
-       op->o_managedsait = 0;
+               op->ors_attrsonly = 1;
+               op->ors_attrs = slap_anlist_no_attrs;
+               op->ors_limit = NULL;
+               op->ors_filter = &uf;
+               op->o_managedsait = SLAP_CONTROL_NONCRITICAL;
 
-       if ( limits_check( op, &rs_search ) == 0 ) {
-               rc = be->be_search( op, &rs_search );
+               uf.f_ava = &eq;
+               uf.f_av_desc = slap_schema.si_ad_entryUUID;
+               uf.f_next = NULL;
+               uf.f_choice = LDAP_FILTER_EQUALITY;
+               si->si_refreshDelete |= NP_DELETE_ONE;
+
+               for (i=0; uuids[i].bv_val; i++) {
+                       op->ors_slimit = 1;
+                       uf.f_av_value = uuids[i];
+                       rc = be->be_search( op, &rs_search );
+               }
+               si->si_refreshDelete ^= NP_DELETE_ONE;
+       } else {
+               memset( &an[0], 0, 2 * sizeof( AttributeName ) );
+               an[0].an_name = slap_schema.si_ad_entryUUID->ad_cname;
+               an[0].an_desc = slap_schema.si_ad_entryUUID;
+               op->ors_attrs = an;
+               op->ors_slimit = SLAP_NO_LIMIT;
+               op->ors_attrsonly = 0;
+               op->ors_filter = str2filter_x( op, si->si_filterstr.bv_val );
+               op->ors_filterstr = si->si_filterstr;
+               op->o_nocaching = 1;
+               op->o_managedsait = SLAP_CONTROL_NONE;
+
+               if ( limits_check( op, &rs_search ) == 0 ) {
+                       rc = be->be_search( op, &rs_search );
+               }
+               if ( op->ors_filter ) filter_free_x( op, op->ors_filter );
        }
 
-       op->o_managedsait = 1;
+       op->o_managedsait = SLAP_CONTROL_NONCRITICAL;
        op->o_nocaching = 0;
 
-       if ( op->ors_filter ) filter_free_x( op, op->ors_filter );
-
        if ( !LDAP_LIST_EMPTY( &si->si_nonpresentlist ) ) {
                np_list = LDAP_LIST_FIRST( &si->si_nonpresentlist );
                while ( np_list != NULL ) {
@@ -1435,6 +1683,7 @@ syncrepl_del_nonpresent(
                        rc = op->o_bd->be_delete( op, &rs_delete );
 
                        if ( rs_delete.sr_err == LDAP_NOT_ALLOWED_ON_NONLEAF ) {
+                               modtail = &modlist;
                                mod = (Modifications *) ch_calloc( 1, sizeof( Modifications ));
                                mod->sml_op = LDAP_MOD_REPLACE;
                                mod->sml_desc = slap_schema.si_ad_objectClass;
@@ -1469,7 +1718,7 @@ syncrepl_del_nonpresent(
                        org_managedsait = get_manageDSAit( op );
                        op->o_dn = op->o_bd->be_rootdn;
                        op->o_ndn = op->o_bd->be_rootndn;
-                       op->o_managedsait = 1;
+                       op->o_managedsait = SLAP_CONTROL_NONCRITICAL;
 
                        while ( rs_delete.sr_err == LDAP_SUCCESS &&
                                        op->o_delete_glue_parent ) {
@@ -1497,8 +1746,8 @@ syncrepl_del_nonpresent(
 
                        ber_bvfree( np_prev->npe_name );
                        ber_bvfree( np_prev->npe_nname );
-                       op->o_req_dn.bv_val = NULL;
-                       op->o_req_ndn.bv_val = NULL;
+                       BER_BVZERO( &op->o_req_dn );
+                       BER_BVZERO( &op->o_req_ndn );
                        ch_free( np_prev );
                }
        }
@@ -1532,8 +1781,8 @@ syncrepl_add_glue(
        ndn = e->e_nname;
 
        /* count RDNs in suffix */
-       if ( be->be_nsuffix[0].bv_len ) {
-               for (i=0, ptr=be->be_nsuffix[0].bv_val; ptr; ptr=strchr( ptr, ',' )) {
+       if ( !BER_BVISEMPTY( &be->be_nsuffix[0] ) ) {
+               for ( i = 0, ptr = be->be_nsuffix[0].bv_val; ptr; ptr = strchr( ptr, ',' ) ) {
                        ptr++;
                        i++;
                }
@@ -1545,7 +1794,7 @@ syncrepl_add_glue(
 
        /* Start with BE suffix */
        for ( i = 0, ptr = NULL; i < suffrdns; i++ ) {
-               comma = strrchr(dn.bv_val, ',');
+               comma = strrchr( dn.bv_val, ',' );
                if ( ptr ) *ptr = ',';
                if ( comma ) *comma = '\0';
                ptr = comma;
@@ -1730,6 +1979,8 @@ syncrepl_updateCookie(
        *modtail = mod;
        modtail = &mod->sml_next;
 
+       slap_queue_csn( op, si->si_syncCookie.ctxcsn );
+
        mlnext = mod;
 
        op->o_tag = LDAP_REQ_ADD;
@@ -1758,7 +2009,7 @@ syncrepl_updateCookie(
        ber_dupbv( &e->e_name, &slap_syncrepl_dn_bv );
        ber_dupbv( &e->e_nname, &slap_syncrepl_dn_bv );
 
-       if ( slap_syncrepl_dn_bv.bv_val ) {
+       if ( !BER_BVISNULL( &slap_syncrepl_dn_bv ) ) {
                slap_sl_free( slap_syncrepl_dn_bv.bv_val, op->o_tmpmemctx );
        }
 
@@ -1812,20 +2063,21 @@ update_cookie_retry:
                                "be_modify failed (%d)\n", rs_modify.sr_err, 0, 0 );
                }
        }
-
        if ( e != NULL ) {
                entry_free( e );
        }
 
 done :
+       slap_graduate_commit_csn( op );
+
 
-       if ( cnbva[0].bv_val ) {
+       if ( !BER_BVISNULL( &cnbva[0] ) ) {
                ch_free( cnbva[0].bv_val );
-               cnbva[0].bv_val = NULL;
+               BER_BVZERO( &cnbva[0] );
        }
-       if ( scbva[0].bv_val ) {
+       if ( !BER_BVISNULL( &scbva[0] ) ) {
                ch_free( scbva[0].bv_val );
-               scbva[0].bv_val = NULL;
+               BER_BVZERO( &scbva[0] );
        }
 
        if ( mlnext->sml_next ) {
@@ -1870,19 +2122,93 @@ dn_callback(
        Operation*      op,
        SlapReply*      rs )
 {
-       syncinfo_t *si = op->o_callback->sc_private;
+       dninfo *dni = op->o_callback->sc_private;
 
        if ( rs->sr_type == REP_SEARCH ) {
-               if ( si->si_syncUUID_ndn.bv_val != NULL ) {
+               if ( !BER_BVISNULL( &dni->dn ) ) {
                        Debug( LDAP_DEBUG_ANY,
-                               "dn_callback : consistency error - entryUUID is not unique\n", 0, 0, 0 );
+                               "dn_callback : consistency error - "
+                               "entryUUID is not unique\n", 0, 0, 0 );
                } else {
-                       ber_dupbv_x( &si->si_syncUUID_ndn, &rs->sr_entry->e_nname, NULL );
+                       ber_dupbv_x( &dni->dn, &rs->sr_entry->e_name, op->o_tmpmemctx );
+                       ber_dupbv_x( &dni->ndn, &rs->sr_entry->e_nname, op->o_tmpmemctx );
+                       /* If there is a new entry, see if it differs from the old.
+                        * We compare the non-normalized values so that cosmetic changes
+                        * in the provider are always propagated.
+                        */
+                       if ( dni->new_entry ) {
+                               Attribute *old, *new;
+                               int i;
+
+                               /* 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.
+                                */
+                               if ( !dn_match( &rs->sr_entry->e_name,
+                                               &dni->new_entry->e_name ) )
+                               {
+                                       dni->renamed = 1;
+                               }
+
+                               for ( i = 0, old = rs->sr_entry->e_attrs;
+                                               old;
+                                               i++, old = old->a_next )
+                                       ;
+
+                               dni->attrs = i;
+
+                               /* 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...
+                                */
+                               for ( old = rs->sr_entry->e_attrs, new = dni->new_entry->e_attrs;
+                                               old && new;
+                                               old = old->a_next, new = new->a_next )
+                               {
+                                       if ( old->a_desc != new->a_desc ) {
+                                               dni->wasChanged = 1;
+                                               break;
+                                       }
+                                       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;
+                                               }
+                                       }
+                                       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++;
+                                       }
+                               }
+                       }
                }
        } else if ( rs->sr_type == REP_RESULT ) {
                if ( rs->sr_err == LDAP_SIZELIMIT_EXCEEDED ) {
                        Debug( LDAP_DEBUG_ANY,
-                               "dn_callback : consistency error - entryUUID is not unique\n", 0, 0, 0 );
+                               "dn_callback : consistency error - "
+                               "entryUUID is not unique\n", 0, 0, 0 );
                }
        }
 
@@ -1905,12 +2231,14 @@ nonpresent_callback(
                si->si_presentlist = NULL;
 
        } else if ( rs->sr_type == REP_SEARCH ) {
-               a = attr_find( rs->sr_entry->e_attrs, slap_schema.si_ad_entryUUID );
+               if ( !(si->si_refreshDelete & NP_DELETE_ONE )) {
+                       a = attr_find( rs->sr_entry->e_attrs, slap_schema.si_ad_entryUUID );
 
-               if ( a == NULL ) return 0;
+                       if ( a == NULL ) return 0;
 
-               present_uuid = avl_find( si->si_presentlist, &a->a_nvals[0],
-                       syncuuid_cmp );
+                       present_uuid = avl_find( si->si_presentlist, &a->a_nvals[0],
+                               syncuuid_cmp );
+               }
 
                if ( present_uuid == NULL ) {
                        np_entry = (struct nonpresent_entry *)
@@ -1972,8 +2300,7 @@ slap_create_syncrepl_entry(
                        context_csn, NULL );
        }
 
-       bv.bv_val = "{}";
-       bv.bv_len = sizeof("{}")-1;
+       BER_BVSTR( &bv, "{}" );
        attr_merge_one( e, slap_schema.si_ad_subtreeSpecification, &bv, NULL );
 
        build_new_dn( &e->e_name, &be->be_nsuffix[0], rdn, NULL );
@@ -1999,12 +2326,17 @@ slap_uuidstr_from_normalized(
                new = uuidstr;
        } else {
                new = (struct berval *)slap_sl_malloc( sizeof(struct berval), ctx );
+               if ( new == NULL ) {
+                       return NULL;
+               }
        }
 
        new->bv_len = 36;
 
-       if (( new->bv_val = slap_sl_malloc( new->bv_len + 1, ctx )) == NULL) {
-               if ( !uuidstr ) slap_sl_free( new, ctx );
+       if ( ( new->bv_val = slap_sl_malloc( new->bv_len + 1, ctx ) ) == NULL ) {
+               if ( new != uuidstr ) {
+                       slap_sl_free( new, ctx );
+               }
                return NULL;
        }
 
@@ -2044,11 +2376,13 @@ syncuuid_cmp( const void* v_uuid1, const void* v_uuid2 )
 }
 
 static void
-avl_ber_bvfree( void *bv )
+avl_ber_bvfree( void *v_bv )
 {
-       if( bv == NULL ) return;
-       if ( ((struct berval *)bv)->bv_val != NULL ) {
-               ch_free( ((struct berval *)bv)->bv_val );
+       struct berval   *bv = (struct berval *)v_bv;
+       
+       if( v_bv == NULL ) return;
+       if ( !BER_BVISNULL( bv ) ) {
+               ch_free( bv->bv_val );
        }
        ch_free( (char *) bv );
 }
@@ -2134,9 +2468,6 @@ syncinfo_free( syncinfo_t *sie )
                ch_free( sie->si_retrynum_init );
        }
        slap_sync_cookie_free( &sie->si_syncCookie, 0 );
-       if ( sie->si_syncUUID_ndn.bv_val ) {
-               ch_free( sie->si_syncUUID_ndn.bv_val );
-       }
        if ( sie->si_presentlist ) {
            avl_free( sie->si_presentlist, avl_ber_bvfree );
        }