]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/syncrepl.c
Add SLAP_MR_ORDERED_INDEX - support for inequality indexing. Currently
[openldap] / servers / slapd / syncrepl.c
index 9885f2b9dc705b97211ddce220dc37e5103dd674..f0fac417fb7b4c9a8184b9221fac843911ca7da4 100644 (file)
@@ -23,7 +23,6 @@
 #include <ac/string.h>
 #include <ac/socket.h>
 
-#include "ldap_pvt.h"
 #include "lutil.h"
 #include "slap.h"
 #include "lutil_ldap.h"
@@ -52,8 +51,9 @@ struct runqueue_s syncrepl_rq;
 void
 init_syncrepl(syncinfo_t *si)
 {
-       int i, j, k, n;
-       char **tmp;
+       int i, j, k, l, n;
+       char **attrs, **exattrs;
+       ObjectClass *oc;
 
        if ( !sync_descs[0] ) {
                sync_descs[0] = slap_schema.si_ad_objectClass;
@@ -62,51 +62,157 @@ init_syncrepl(syncinfo_t *si)
                sync_descs[3] = NULL;
        }
 
-       for ( n = 0; si->si_attrs[ n ] != NULL; n++ ) /* empty */;
-
-       if ( n ) {
-               /* Delete Attributes */
-               for ( i = 0; sync_descs[i] != NULL; i++ ) {
-                       for ( j = 0; si->si_attrs[j] != NULL; j++ ) {
-                               if ( strcmp( si->si_attrs[j], sync_descs[i]->ad_cname.bv_val )
-                                       == 0 )
-                               {
-                                       ch_free( si->si_attrs[j] );
-                                       for ( k = j; si->si_attrs[k] != NULL; k++ ) {
-                                               si->si_attrs[k] = si->si_attrs[k+1];
+       if ( si->si_allattrs && si->si_allopattrs )
+               attrs = NULL;
+       else
+               attrs = anlist2attrs( si->si_anlist );
+
+       if ( attrs ) {
+               if ( si->si_allattrs ) {
+                       i = 0;
+                       while ( attrs[i] ) {
+                               if ( !is_at_operational( at_find( attrs[i] ))) {
+                                       for ( j = i; attrs[j] != NULL; j++ ) {
+                                               if ( j == i )
+                                                       ch_free( attrs[i] );
+                                               attrs[j] = attrs[j+1];
                                        }
+                               } else {
+                                       i++;
                                }
                        }
+                       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] ) {
+                               if ( is_at_operational( at_find( attrs[i] ))) {
+                                       for ( j = i; attrs[j] != NULL; j++ ) {
+                                               if ( j == i )
+                                                       ch_free( attrs[i] );
+                                               attrs[j] = attrs[j+1];
+                                       }
+                               } else {
+                                       i++;
+                               }
+                       }
+                       attrs = ( char ** ) ch_realloc( attrs, (i + 2)*sizeof( char * ) );
+                       attrs[i] = ch_strdup("+");
+                       attrs[i + 1] = NULL;
                }
-               for ( n = 0; si->si_attrs[ n ] != NULL; n++ ) /* empty */;
-               tmp = ( char ** ) ch_realloc( si->si_attrs, (n + 4)*sizeof( char * ));
-               if ( tmp == NULL ) {
-#ifdef NEW_LOGGING
-                       LDAP_LOG( OPERATION, ERR, "out of memory\n", 0,0,0 );
-#else
-                       Debug( LDAP_DEBUG_ANY, "out of memory\n", 0,0,0 );
-#endif
+
+               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 ( n = 0; attrs[ n ] != NULL; n++ ) /* empty */;
+                       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 */
+                       for ( i = 0; sync_descs[ i ] != NULL; i++ ) {
+                               attrs[ n++ ] = ch_strdup ( sync_descs[i]->ad_cname.bv_val );
+                       }
+                       attrs[ n ] = NULL;
                }
 
        } else {
-               tmp = ( char ** ) ch_realloc( si->si_attrs, 5 * sizeof( char * ));
-               if ( tmp == NULL ) {
-#ifdef NEW_LOGGING
-                       LDAP_LOG( OPERATION, ERR, "out of memory\n", 0,0,0 );
-#else
-                       Debug( LDAP_DEBUG_ANY, "out of memory\n", 0,0,0 );
-#endif
+               for ( n = 0; sync_descs[ n ] != NULL; n++ ) ;
+
+               attrs = ( char ** ) ch_malloc((
+                                       (si->si_allattrs   ? 1 : ( si->si_allopattrs ? 0 : 1)) +
+                                       (si->si_allopattrs ? 1 : ( si->si_allattrs   ? n : 1))
+                                       + 1 ) * sizeof( char * ) );
+               if ( attrs == NULL ) {
+                       Debug( LDAP_DEBUG_ANY, "out of memory\n", 0, 0, 0 );
+               }
+               
+               i = 0;
+               if ( si->si_allattrs ) {
+                       attrs[i++] = ch_strdup( "*" );
+               }
+
+               if ( si->si_allopattrs ) {
+                       attrs[i++] = ch_strdup( "+" );
+               } else {
+                       for ( j = 0; sync_descs[ j ] != NULL; j++ ) {
+                               attrs[ i++ ] = ch_strdup ( sync_descs[j]->ad_cname.bv_val );
+                       }
+               }
+               attrs[i] = NULL;
+
+               if ( !si->si_allattrs && !si->si_allopattrs ) {
+                       attrs[0] = ch_strdup( "*" );
+                       attrs[1] = ch_strdup( "+" );
+                       attrs[2] = NULL;
                }
-               tmp[ n++ ] = ch_strdup( "*" );
        }
        
-       si->si_attrs = tmp;
+       si->si_attrs = attrs;
+
+       exattrs = anlist2attrs( si->si_exanlist );
+
+       if ( exattrs ) {
+               for ( n = 0; exattrs[n] != NULL; n++ ) ;
 
-       /* Add Attributes */
-       for ( i = 0; sync_descs[ i ] != NULL; i++ ) {
-               si->si_attrs[ n++ ] = ch_strdup ( sync_descs[i]->ad_cname.bv_val );
-               si->si_attrs[ n ] = NULL;
+               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 )) {
+                                       for ( k = j; exattrs[k] != NULL; k++ ) {
+                                               if ( k == j )
+                                                       ch_free( exattrs[k] );
+                                               exattrs[k] = exattrs[k+1];
+                                       }
+                               } else {
+                                       j++;
+                               }
+                       }
+               }
+
+               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 ) {
+                                       k = 0;
+                                       while ( oc->soc_required[k] ) {
+                                               if ( !strcmp( exattrs[i],
+                                                        oc->soc_required[k]->sat_cname.bv_val )) {
+                                                       for ( l = i; exattrs[l]; l++ ) {
+                                                               if ( l == i )
+                                                                       ch_free( exattrs[i] );
+                                                               exattrs[l] = exattrs[l+1];
+                                                       }
+                                               } else {
+                                                       k++;
+                                               }
+                                       }
+                               }
+                       }
+               }
+
+               for ( i = 0; exattrs[i] != NULL; i++ ) ;
+
+               if ( i != n )
+                       exattrs = (char **) ch_realloc( exattrs, (i + 1)*sizeof(char *));
        }
+
+       si->si_exattrs = exattrs;       
 }
 
 static int
@@ -126,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),
@@ -155,12 +261,12 @@ ldap_sync_search(
                ctrls[1] = NULL;
        }
 
-       timeout.tv_sec = si->si_tlimit > 0 ? si->si_tlimit : 1;
+       timeout.tv_sec = si->si_tlimit;
        timeout.tv_usec = 0;
 
        rc = ldap_search_ext( si->si_ld, si->si_base.bv_val, si->si_scope,
                si->si_filterstr.bv_val, si->si_attrs, si->si_attrsonly,
-               ctrls, NULL, si->si_tlimit < 0 ? NULL : &timeout,
+               ctrls, NULL, si->si_tlimit > 0 ? &timeout : NULL,
                si->si_slimit, &msgid );
        ber_free_buf( ber );
        return rc;
@@ -179,21 +285,18 @@ do_syncrep1(
        struct sync_cookie      *sc = NULL;
        struct sync_cookie      syncCookie = { NULL, -1, 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_provideruri );
        if ( rc != LDAP_SUCCESS ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, ERR,
-                       "do_syncrep1: ldap_initialize failed (%s)\n",
-                       si->si_provideruri, 0, 0 );
-#else
                Debug( LDAP_DEBUG_ANY,
                        "do_syncrep1: ldap_initialize failed (%s)\n",
                        si->si_provideruri, 0, 0 );
-#endif
                return rc;
        }
 
@@ -205,17 +308,10 @@ do_syncrep1(
        if ( si->si_tls ) {
                rc = ldap_start_tls_s( si->si_ld, NULL, NULL );
                if( rc != LDAP_SUCCESS ) {
-#ifdef NEW_LOGGING
-                       LDAP_LOG ( OPERATION, ERR, "do_syncrep1: "
-                               "%s: ldap_start_tls failed (%d)\n",
-                               si->si_tls == SYNCINFO_TLS_CRITICAL ? "Error" : "Warning",
-                               rc, 0 );
-#else
                        Debug( LDAP_DEBUG_ANY,
                                "%s: ldap_start_tls failed (%d)\n",
                                si->si_tls == SYNCINFO_TLS_CRITICAL ? "Error" : "Warning",
                                rc, 0 );
-#endif
                        if( si->si_tls == SYNCINFO_TLS_CRITICAL ) goto done;
                }
        }
@@ -229,15 +325,9 @@ do_syncrep1(
                                LDAP_OPT_X_SASL_SECPROPS, si->si_secprops);
 
                        if( rc != LDAP_OPT_SUCCESS ) {
-#ifdef NEW_LOGGING
-                               LDAP_LOG ( OPERATION, ERR, "do_bind: Error: "
-                                       "ldap_set_option(%s,SECPROPS,\"%s\") failed!\n",
-                                       si->si_provideruri, si->si_secprops, 0 );
-#else
                                Debug( LDAP_DEBUG_ANY, "Error: ldap_set_option "
                                        "(%s,SECPROPS,\"%s\") failed!\n",
                                        si->si_provideruri, si->si_secprops, 0 );
-#endif
                                goto done;
                        }
                }
@@ -260,15 +350,9 @@ do_syncrep1(
                 *      2) on err policy : exit, retry, backoff ...
                 */
                if ( rc != LDAP_SUCCESS ) {
-#ifdef NEW_LOGGING
-                       LDAP_LOG ( OPERATION, ERR, "do_syncrep1: "
-                               "ldap_sasl_interactive_bind_s failed (%d)\n",
-                               rc, 0, 0 );
-#else
                        Debug( LDAP_DEBUG_ANY, "do_syncrep1: "
                                "ldap_sasl_interactive_bind_s failed (%d)\n",
                                rc, 0, 0 );
-#endif
 
                        /* FIXME (see above comment) */
                        /* if Kerberos credentials cache is not active, retry */
@@ -292,21 +376,31 @@ do_syncrep1(
                rc = ldap_bind_s( si->si_ld,
                        si->si_binddn, si->si_passwd, si->si_bindmethod );
                if ( rc != LDAP_SUCCESS ) {
-#ifdef NEW_LOGGING
-                       LDAP_LOG ( OPERATION, ERR, "do_syncrep1: "
-                               "ldap_bind_s failed (%d)\n", rc, 0, 0 );
-#else
                        Debug( LDAP_DEBUG_ANY, "do_syncrep1: "
                                "ldap_bind_s failed (%d)\n", rc, 0, 0 );
-#endif
                        goto done;
                }
        }
 
+       /* Set SSF to strongest of TLS, SASL SSFs */
+       op->o_sasl_ssf = 0;
+       op->o_tls_ssf = 0;
+       op->o_transport_ssf = 0;
+#ifdef HAVE_TLS
+       if ( ldap_get_option( si->si_ld, LDAP_OPT_X_TLS_SSL_CTX, &ssl )
+               == LDAP_SUCCESS && ssl != NULL )
+       {
+               op->o_tls_ssf = ldap_pvt_tls_get_strength( ssl );
+       }
+#endif /* HAVE_TLS */
+       ldap_get_option( si->si_ld, LDAP_OPT_X_SASL_SSF, &op->o_sasl_ssf );
+       op->o_ssf = ( op->o_sasl_ssf > op->o_tls_ssf )
+               ?  op->o_sasl_ssf : op->o_tls_ssf;
+
        /* 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;
@@ -328,12 +422,12 @@ do_syncrep1(
 
                /* read stored cookie if it exists */
                backend_attribute( op, NULL, &op->o_req_ndn,
-                       slap_schema.si_ad_syncreplCookie, &cookie );
+                       slap_schema.si_ad_syncreplCookie, &cookie, ACL_READ );
 
                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 );
@@ -383,7 +477,7 @@ do_syncrep1(
                        struct berval cookie_bv;
                        /* try to read stored cookie */
                        backend_attribute( op, NULL, &op->o_req_ndn,
-                               slap_schema.si_ad_syncreplCookie, &cookie );
+                               slap_schema.si_ad_syncreplCookie, &cookie, ACL_READ );
                        if ( cookie ) {
                                ber_dupbv( &cookie_bv, &cookie[0] );
                                ber_bvarray_add( &si->si_syncCookie.octet_str, &cookie_bv );
@@ -396,13 +490,8 @@ do_syncrep1(
        rc = ldap_sync_search( si, op->o_tmpmemctx );
 
        if( rc != LDAP_SUCCESS ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG ( OPERATION, ERR, "do_syncrep1: "
-                       "ldap_search_ext: %s (%d)\n", ldap_err2string( rc ), rc, 0 );
-#else
                Debug( LDAP_DEBUG_ANY, "do_syncrep1: "
                        "ldap_search_ext: %s (%d)\n", ldap_err2string( rc ), rc, 0 );
-#endif
        }
 
 done:
@@ -413,6 +502,8 @@ done:
                }
        }
 
+       slap_sl_free( op->o_req_ndn.bv_val, op->o_tmpmemctx );
+
        return rc;
 }
 
@@ -468,11 +559,7 @@ do_syncrep2(
        ber_init2( ber, NULL, LBER_USE_DER );
        ber_set_option( ber, LBER_OPT_BER_MEMCTX, &op->o_tmpmemctx );
 
-#ifdef NEW_LOGGING
-       LDAP_LOG ( OPERATION, DETAIL1, "do_syncrep2\n", 0, 0, 0 );
-#else
        Debug( LDAP_DEBUG_TRACE, "=>do_syncrep2\n", 0, 0, 0 );
-#endif
 
        psub = &si->si_be->be_nsuffix[0];
 
@@ -500,6 +587,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;
                                }
@@ -508,21 +598,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 );
                                        if ( syncCookie.octet_str &&
-                                               syncCookie.octet_str[0].bv_val )
+                                               !BER_BVISNULL( &syncCookie.octet_str[0] ) )
                                        {
                                                syncrepl_updateCookie( si, op, psub, &syncCookie );
                                        }
@@ -533,18 +625,13 @@ do_syncrep2(
                                }
                                if ( rc_efree && entry ) {
                                        entry_free( entry );
-                                       entry = NULL;
                                }
+                               entry = NULL;
                                break;
 
                        case LDAP_RES_SEARCH_REFERENCE:
-#ifdef NEW_LOGGING
-                               LDAP_LOG( OPERATION, ERR,
-                                       "do_syncrep2 : reference received\n", 0, 0, 0 );
-#else
                                Debug( LDAP_DEBUG_ANY,
                                        "do_syncrep2 : reference received\n", 0, 0, 0 );
-#endif
                                break;
 
                        case LDAP_RES_SEARCH_RESULT:
@@ -557,13 +644,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 );
                                                }
@@ -585,7 +672,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 );
@@ -632,14 +719,14 @@ do_syncrep2(
                                                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 );
                                                        }
@@ -657,14 +744,14 @@ do_syncrep2(
                                                        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 );
                                                        }
@@ -676,25 +763,20 @@ do_syncrep2(
                                                }
                                                ber_scanf( ber, "[W]", &syncUUIDs );
                                                ber_scanf( ber, /*"{"*/ "}" );
-                                               for ( i = 0; syncUUIDs[i].bv_val; 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,
                                                                (caddr_t) syncuuid_bv,
                                                                syncuuid_cmp, avl_dup_error );
                                                }
-                                               ber_memfree_x( syncUUIDs, op->o_tmpmemctx );
+                                               slap_sl_free( syncUUIDs, op->o_tmpmemctx );
                                                break;
                                        default:
-#ifdef NEW_LOGGING
-                                       LDAP_LOG( OPERATION, ERR,
-                                               "do_syncrep2 : unknown syncinfo tag (%ld)\n",
-                                               (long) si_tag, 0, 0 );
-#else
                                        Debug( LDAP_DEBUG_ANY,
                                                "do_syncrep2 : unknown syncinfo tag (%ld)\n",
                                                (long) si_tag, 0, 0 );
-#endif
                                                ldap_memfree( retoid );
                                                ber_bvfree( retdata );
                                                continue;
@@ -712,7 +794,7 @@ 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);
@@ -729,15 +811,9 @@ do_syncrep2(
                                        break;
 
                                } else {
-#ifdef NEW_LOGGING
-                                       LDAP_LOG( OPERATION, ERR, "do_syncrep2 :"
-                                               " unknown intermediate "
-                                               "response\n", 0, 0, 0 );
-#else
                                        Debug( LDAP_DEBUG_ANY, "do_syncrep2 : "
                                                "unknown intermediate response (%d)\n",
                                                rc, 0, 0 );
-#endif
                                        ldap_memfree( retoid );
                                        ber_bvfree( retdata );
                                        break;
@@ -745,13 +821,8 @@ do_syncrep2(
                                break;
 
                        default:
-#ifdef NEW_LOGGING
-                               LDAP_LOG( OPERATION, ERR, "do_syncrep2 : "
-                                       "unknown message\n", 0, 0, 0 );
-#else
                                Debug( LDAP_DEBUG_ANY, "do_syncrep2 : "
                                        "unknown message\n", 0, 0, 0 );
-#endif
                                break;
 
                        }
@@ -771,13 +842,8 @@ do_syncrep2(
                ldap_get_option( si->si_ld, LDAP_OPT_ERROR_NUMBER, &rc );
                errstr = ldap_err2string( rc );
                
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, ERR,
-                       "do_syncrep2 : %s\n", errstr, 0, 0 );
-#else
                Debug( LDAP_DEBUG_ANY,
                        "do_syncrep2 : %s\n", errstr, 0, 0 );
-#endif
        }
 
 done:
@@ -807,12 +873,10 @@ do_syncrepl(
        int first = 0;
        int dostop = 0;
        ber_socket_t s;
+       int i, defer = 1;
+       Backend *be;
 
-#ifdef NEW_LOGGING
-       LDAP_LOG ( OPERATION, DETAIL1, "do_syncrepl\n", 0, 0, 0 );
-#else
        Debug( LDAP_DEBUG_TRACE, "=>do_syncrepl\n", 0, 0, 0 );
-#endif
 
        if ( si == NULL )
                return NULL;
@@ -835,10 +899,14 @@ do_syncrepl(
 
        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 = si->si_be;
+       op.o_bd = be = si->si_be;
 
        op.o_sync_state.ctxcsn = NULL;
        op.o_sync_state.sid = -1;
@@ -865,11 +933,10 @@ do_syncrepl(
                         */
                        if ( rc == LDAP_SUCCESS ) {
                                if ( first ) {
-                                       rc = connection_client_setup( s, do_syncrepl,
-                                               arg );
+                                       rc = connection_client_setup( s, do_syncrepl, arg );
                                } else {
                                        connection_client_enable( s );
-                               }
+                               } 
                        } else if ( !first ) {
                                dostop = 1;
                        }
@@ -885,6 +952,7 @@ do_syncrepl(
         * 4) for Persist and Success, reschedule to defer
         */
        ldap_pvt_thread_mutex_lock( &syncrepl_rq.rq_mutex );
+
        if ( ldap_pvt_runqueue_isrunning( &syncrepl_rq, rtask )) {
                ldap_pvt_runqueue_stoptask( &syncrepl_rq, rtask );
        }
@@ -893,17 +961,37 @@ do_syncrepl(
                connection_client_stop( s );
        }
 
-       if ( rc && rc != LDAP_SERVER_DOWN ) {
-               ldap_pvt_runqueue_remove( &syncrepl_rq, rtask );
+       if ( rc == LDAP_SUCCESS ) {
+               if ( si->si_type == LDAP_SYNC_REFRESH_ONLY ) {
+                       defer = 0;
+               }
+               rtask->interval.tv_sec = si->si_interval;
+               ldap_pvt_runqueue_resched( &syncrepl_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];
+                       }
+                       si->si_retrynum[i] = -2;
+               }
        } else {
-               if ( rc == LDAP_SERVER_DOWN ||
-                       si->si_type == LDAP_SYNC_REFRESH_ONLY ) {
-                       rc = 0;
-               } else {
-                       rc = 1;
+               for ( i = 0; si->si_retrynum && si->si_retrynum[i] <= 0; i++ ) {
+                       if ( si->si_retrynum[i] == -1  || si->si_retrynum[i] == -2 )
+                               break;
+               }
+
+               if ( !si->si_retrynum || si->si_retrynum[i] == -2 ) {
+                       ldap_pvt_runqueue_remove( &syncrepl_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 );
+                       slap_wake_listener();
                }
-               ldap_pvt_runqueue_resched( &syncrepl_rq, rtask, rc );
        }
+       
        ldap_pvt_thread_mutex_unlock( &syncrepl_rq.rq_mutex );
 
        return NULL;
@@ -935,13 +1023,8 @@ syncrepl_message_to_entry(
        *modlist = NULL;
 
        if ( ldap_msgtype( msg ) != LDAP_RES_SEARCH_ENTRY ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, ERR,
-                       "Message type should be entry (%d)", ldap_msgtype( msg ), 0, 0 );
-#else
                Debug( LDAP_DEBUG_ANY,
                        "Message type should be entry (%d)", ldap_msgtype( msg ), 0, 0 );
-#endif
                return -1;
        }
 
@@ -950,23 +1033,20 @@ syncrepl_message_to_entry(
        rc = ldap_get_dn_ber( si->si_ld, msg, &ber, &bdn );
 
        if ( rc != LDAP_SUCCESS ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, ERR,
-                       "syncrepl_message_to_entry : dn get failed (%d)", rc, 0, 0 );
-#else
                Debug( LDAP_DEBUG_ANY,
                        "syncrepl_message_to_entry : dn get failed (%d)", rc, 0, 0 );
-#endif
                return rc;
        }
 
        dnPrettyNormal( NULL, &bdn, &dn, &ndn, op->o_tmpmemctx );
        ber_dupbv( &op->o_req_dn, &dn );
        ber_dupbv( &op->o_req_ndn, &ndn );
-       sl_free( ndn.bv_val, op->o_tmpmemctx );
-       sl_free( dn.bv_val, op->o_tmpmemctx );
+       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;
        }
 
@@ -981,7 +1061,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;
                }
@@ -1000,44 +1080,58 @@ syncrepl_message_to_entry(
        }
 
        if ( *modlist == NULL ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, ERR,
-                       "syncrepl_message_to_entry: no attributes\n", 0, 0, 0 );
-#else
                Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_entry: no attributes\n",
                        0, 0, 0 );
-#endif
+               rc = -1;
+               goto done;
        }
 
        rc = slap_mods_check( *modlist, 1, &text, txtbuf, textlen, NULL );
 
        if ( rc != LDAP_SUCCESS ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, ERR,
-                       "syncrepl_message_to_entry: mods check (%s)\n", text, 0, 0 );
-#else
                Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_entry: mods check (%s)\n",
                        text, 0, 0 );
-#endif
                goto done;
        }
+
+       /* Strip out dynamically generated attrs */
+       for ( modtail = modlist; *modtail ; ) {
+               mod = *modtail;
+               if ( mod->sml_desc->ad_type->sat_flags & SLAP_AT_DYNAMIC ) {
+                       *modtail = mod->sml_next;
+                       slap_mod_free( &mod->sml_mod, 0 );
+                       ch_free( mod );
+               } else {
+                       modtail = &mod->sml_next;
+               }
+       }
+
+       /* Strip out attrs in exattrs list */
+       for ( modtail = modlist; *modtail ; ) {
+               mod = *modtail;
+               if ( ldap_charray_inlist( si->si_exattrs,
+                                       mod->sml_desc->ad_type->sat_cname.bv_val )) {
+                       *modtail = mod->sml_next;
+                       slap_mod_free( &mod->sml_mod, 0 );
+                       ch_free( mod );
+               } else {
+                       modtail = &mod->sml_next;
+               }
+       }
        
        rc = slap_mods2entry( *modlist, &e, 1, 1, &text, txtbuf, textlen);
        if( rc != LDAP_SUCCESS ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, ERR,
-                       "syncrepl_message_to_entry: mods2entry (%s)\n", text, 0, 0 );
-#else
                Debug( LDAP_DEBUG_ANY, "syncrepl_message_to_entry: mods2entry (%s)\n",
                        text, 0, 0 );
-#endif
        }
 
 done:
        ber_free ( ber, 0 );
        if ( rc != LDAP_SUCCESS ) {
-               entry_free( e );
-               e = NULL;
+               if ( e ) {
+                       entry_free( e );
+                       *entry = e = NULL;
+               }
        }
 
        return rc;
@@ -1047,7 +1141,7 @@ int
 syncrepl_entry(
        syncinfo_t* si,
        Operation *op,
-       Entry* e,
+       Entry* entry,
        Modifications* modlist,
        int syncstate,
        struct berval* syncUUID,
@@ -1059,7 +1153,10 @@ syncrepl_entry(
        struct berval   syncUUID_strrep = BER_BVNULL;
        struct berval   uuid_bv = BER_BVNULL;
 
-       SlapReply       rs = {REP_RESULT};
+       SlapReply       rs_search = {REP_RESULT};
+       SlapReply       rs_delete = {REP_RESULT};
+       SlapReply       rs_add = {REP_RESULT};
+       SlapReply       rs_modify = {REP_RESULT};
        Filter f = {0};
        AttributeAssertion ava = {0};
        int rc = LDAP_SUCCESS;
@@ -1080,21 +1177,25 @@ syncrepl_entry(
        }
 
        if ( syncstate == LDAP_SYNC_PRESENT ) {
-               return e ? 1 : 0;
+               return 0;
+       } else if ( syncstate != LDAP_SYNC_DELETE ) {
+               if ( entry == NULL ) {
+                       return 0;
+               }
        }
 
        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_val = (char *) sl_malloc(
+       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';
 
@@ -1105,25 +1206,33 @@ syncrepl_entry(
        op->o_req_dn = si->si_base;
        op->o_req_ndn = si->si_base;
 
+       op->o_time = slap_get_time();
+       op->ors_tlimit = SLAP_NO_LIMIT;
+       op->ors_slimit = 1;
+
+       op->ors_attrs = slap_anlist_no_attrs;
+       op->ors_attrsonly = 1;
+
        /* set callback function */
        op->o_callback = &cb;
        cb.sc_response = dn_callback;
        cb.sc_private = si;
 
-       si->si_syncUUID_ndn.bv_val = NULL;
+       BER_BVZERO( &si->si_syncUUID_ndn );
 
-       if ( limits_check( op, &rs ) == 0 ) {
-               rc = be->be_search( op, &rs );
+       if ( limits_check( op, &rs_search ) == 0 ) {
+               rc = be->be_search( op, &rs_search );
        }
 
-       if ( op->ors_filterstr.bv_val ) {
-               sl_free( op->ors_filterstr.bv_val, op->o_tmpmemctx );
+       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 ( rc == LDAP_SUCCESS && si->si_syncUUID_ndn.bv_val ) {
+       if ( rs_search.sr_err == LDAP_SUCCESS && !BER_BVISNULL( &si->si_syncUUID_ndn ) )
+       {
                char *subseq_ptr;
 
                if ( syncstate != LDAP_SYNC_DELETE ) {
@@ -1131,7 +1240,7 @@ syncrepl_entry(
                }
 
                ber_dupbv( &op->o_sync_csn, syncCookie_req->ctxcsn );
-               if ( op->o_sync_csn.bv_val ) {
+               if ( !BER_BVISNULL( &op->o_sync_csn ) ) {
                        subseq_ptr = strstr( op->o_sync_csn.bv_val, "#0000" );
                        subseq_ptr += 4;
                        *subseq_ptr = '1';
@@ -1140,7 +1249,7 @@ syncrepl_entry(
                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 );
+               rc = be->be_delete( op, &rs_delete );
 
                org_req_dn = op->o_req_dn;
                org_req_ndn = op->o_req_ndn;
@@ -1151,7 +1260,7 @@ syncrepl_entry(
                op->o_ndn = op->o_bd->be_rootndn;
                op->o_managedsait = 1;
 
-               while ( rs.sr_err == LDAP_SUCCESS && op->o_delete_glue_parent ) {
+               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 };
@@ -1160,7 +1269,7 @@ syncrepl_entry(
                                op->o_req_dn = pdn;
                                op->o_req_ndn = pdn;
                                op->o_callback = &cb;
-                               op->o_bd->be_delete( op, &rs );
+                               op->o_bd->be_delete( op, &rs_delete );
                        } else {
                                break;
                    }
@@ -1179,23 +1288,25 @@ syncrepl_entry(
        switch ( syncstate ) {
        case LDAP_SYNC_ADD:
        case LDAP_SYNC_MODIFY:
-               if ( rc == LDAP_SUCCESS ||
-                        rc == LDAP_REFERRAL ||
-                        rc == LDAP_NO_SUCH_OBJECT ||
-                        rc == LDAP_NOT_ALLOWED_ON_NONLEAF )
+               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( &e->e_attrs, slap_schema.si_ad_entryUUID );
-                       attr_merge_one( e, slap_schema.si_ad_entryUUID,
-                               syncUUID, &ava.aa_value );
+                       attr_delete( &entry->e_attrs, slap_schema.si_ad_entryUUID );
+                       attr_merge_one( entry, slap_schema.si_ad_entryUUID,
+                               &syncUUID_strrep, syncUUID );
 
                        op->o_tag = LDAP_REQ_ADD;
-                       op->ora_e = e;
-                       op->o_req_dn = e->e_name;
-                       op->o_req_ndn = e->e_nname;
-                       rc = be->be_add( op, &rs );
+                       op->ora_e = entry;
+                       op->o_req_dn = entry->e_name;
+                       op->o_req_ndn = entry->e_nname;
+
+                       rc = be->be_add( op, &rs_add );
 
-                       if ( rc != LDAP_SUCCESS ) {
-                               if ( rc == LDAP_ALREADY_EXISTS ) {
+                       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;
 
@@ -1215,53 +1326,38 @@ syncrepl_entry(
                                        
                                        op->o_tag = LDAP_REQ_MODIFY;
                                        op->orm_modlist = modlist;
-                                       op->o_req_dn = e->e_name;
-                                       op->o_req_ndn = e->e_nname;
+                                       op->o_req_dn = entry->e_name;
+                                       op->o_req_ndn = entry->e_nname;
 
-                                       rc = be->be_modify( op, &rs );
-                                       if ( rc != LDAP_SUCCESS ) {
-#ifdef NEW_LOGGING
-                                               LDAP_LOG( OPERATION, ERR,
-                                                       "syncrepl_entry : be_modify failed (%d)\n",
-                                                       rc, 0, 0 );
-#else
+                                       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",
-                                                       rc, 0, 0 );
-#endif
+                                                       rs_modify.sr_err, 0, 0 );
                                        }
                                        ret = 1;
                                        goto done;
-                               } else if ( rc == LDAP_REFERRAL || rc == LDAP_NO_SUCH_OBJECT ) {
-                                       syncrepl_add_glue( op, e );
+                               } 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;
                                } else {
-#ifdef NEW_LOGGING
-                                       LDAP_LOG( OPERATION, ERR,
-                                               "syncrepl_entry : be_add failed (%d)\n",
-                                               rc, 0, 0 );
-#else
                                        Debug( LDAP_DEBUG_ANY,
                                                "syncrepl_entry : be_add failed (%d)\n",
-                                               rc, 0, 0 );
-#endif
+                                               rs_add.sr_err, 0, 0 );
                                        ret = 1;
                                        goto done;
                                }
                        } else {
-                               be_entry_release_w( op, e );
+                               be_entry_release_w( op, entry );
                                ret = 0;
                                goto done;
                        }
                } else {
-#ifdef NEW_LOGGING
-                       LDAP_LOG( OPERATION, ERR,
-                               "syncrepl_entry : be_search failed (%d)\n", rc, 0, 0 );
-#else
                        Debug( LDAP_DEBUG_ANY,
-                               "syncrepl_entry : be_search failed (%d)\n", rc, 0, 0 );
-#endif
+                               "syncrepl_entry : be_search failed (%d)\n",
+                               rs_search.sr_err, 0, 0 );
                        ret = 1;
                        goto done;
                }
@@ -1272,24 +1368,21 @@ syncrepl_entry(
                goto done;
 
        default :
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, ERR,
-                       "syncrepl_entry : unknown syncstate\n", 0, 0, 0 );
-#else
                Debug( LDAP_DEBUG_ANY,
                        "syncrepl_entry : unknown syncstate\n", 0, 0, 0 );
-#endif
                ret = 1;
                goto done;
        }
 
 done :
 
-       if ( syncUUID_strrep.bv_val ) {
-               ber_memfree_x( syncUUID_strrep.bv_val, op->o_tmpmemctx );
+       if ( !BER_BVISNULL( &syncUUID_strrep ) ) {
+               slap_sl_free( syncUUID_strrep.bv_val, op->o_tmpmemctx );
+               BER_BVZERO( &syncUUID_strrep );
        }
-       if ( si->si_syncUUID_ndn.bv_val ) {
-               ber_memfree_x( si->si_syncUUID_ndn.bv_val, op->o_tmpmemctx );
+       if ( !BER_BVISNULL( &si->si_syncUUID_ndn ) ) {
+               ch_free( si->si_syncUUID_ndn.bv_val );
+               BER_BVZERO( &si->si_syncUUID_ndn );
        }
        return ret;
 }
@@ -1307,7 +1400,9 @@ syncrepl_del_nonpresent(
 {
        Backend* be = op->o_bd;
        slap_callback   cb = { NULL };
-       SlapReply       rs = {REP_RESULT};
+       SlapReply       rs_search = {REP_RESULT};
+       SlapReply       rs_delete = {REP_RESULT};
+       SlapReply       rs_modify = {REP_RESULT};
        struct nonpresent_entry *np_list, *np_prev;
        int rc;
        Modifications *ml;
@@ -1316,6 +1411,7 @@ syncrepl_del_nonpresent(
        Modifications *modlist = NULL;
        Modifications **modtail = &modlist;
        Attribute       *attr;
+       AttributeName   an[2];
 
        struct berval pdn = BER_BVNULL;
        struct berval org_req_dn = BER_BVNULL;
@@ -1334,18 +1430,24 @@ syncrepl_del_nonpresent(
        op->o_tag = LDAP_REQ_SEARCH;
        op->ors_scope = si->si_scope;
        op->ors_deref = LDAP_DEREF_NEVER;
-       op->ors_slimit = 0;
-       op->ors_tlimit = 0;
+       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_attrs = NULL;
        op->ors_filter = str2filter_x( op, si->si_filterstr.bv_val );
        op->ors_filterstr = si->si_filterstr;
 
        op->o_nocaching = 1;
        op->o_managedsait = 0;
 
-       if ( limits_check( op, &rs ) == 0 ) {
-               be->be_search( op, &rs );
+       if ( limits_check( op, &rs_search ) == 0 ) {
+               rc = be->be_search( op, &rs_search );
        }
 
        op->o_managedsait = 1;
@@ -1365,9 +1467,9 @@ syncrepl_del_nonpresent(
                        cb.sc_private = si;
                        op->o_req_dn = *np_prev->npe_name;
                        op->o_req_ndn = *np_prev->npe_nname;
-                       rc = op->o_bd->be_delete( op, &rs );
+                       rc = op->o_bd->be_delete( op, &rs_delete );
 
-                       if ( rc == LDAP_NOT_ALLOWED_ON_NONLEAF ) {
+                       if ( rs_delete.sr_err == LDAP_NOT_ALLOWED_ON_NONLEAF ) {
                                mod = (Modifications *) ch_calloc( 1, sizeof( Modifications ));
                                mod->sml_op = LDAP_MOD_REPLACE;
                                mod->sml_desc = slap_schema.si_ad_objectClass;
@@ -1387,7 +1489,7 @@ syncrepl_del_nonpresent(
                                op->o_tag = LDAP_REQ_MODIFY;
                                op->orm_modlist = modlist;
 
-                               rc = be->be_modify( op, &rs );
+                               rc = be->be_modify( op, &rs_modify );
 
                                for ( ml = modlist; ml != NULL; ml = mlnext ) {
                                        mlnext = ml->sml_next;
@@ -1404,7 +1506,7 @@ syncrepl_del_nonpresent(
                        op->o_ndn = op->o_bd->be_rootndn;
                        op->o_managedsait = 1;
 
-                       while ( rs.sr_err == LDAP_SUCCESS &&
+                       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 )) {
@@ -1415,7 +1517,7 @@ syncrepl_del_nonpresent(
                                        op->o_req_ndn = pdn;
                                        op->o_callback = &cb;
                                        /* give it a root privil ? */
-                                       op->o_bd->be_delete( op, &rs );
+                                       op->o_bd->be_delete( op, &rs_delete );
                                } else {
                                        break;
                            }
@@ -1430,8 +1532,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 );
                }
        }
@@ -1453,7 +1555,7 @@ syncrepl_add_glue(
        struct berval dn = {0, NULL};
        struct berval ndn = {0, NULL};
        Entry   *glue;
-       SlapReply       rs = {REP_RESULT};
+       SlapReply       rs_add = {REP_RESULT};
        char    *ptr, *comma;
 
        op->o_tag = LDAP_REQ_ADD;
@@ -1465,8 +1567,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++;
                }
@@ -1478,7 +1580,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;
@@ -1529,8 +1631,8 @@ syncrepl_add_glue(
                op->o_req_dn = glue->e_name;
                op->o_req_ndn = glue->e_nname;
                op->ora_e = glue;
-               rc = be->be_add ( op, &rs );
-               if ( rc == LDAP_SUCCESS ) {
+               rc = be->be_add ( op, &rs_add );
+               if ( rs_add.sr_err == LDAP_SUCCESS ) {
                        be_entry_release_w( op, glue );
                } else {
                /* incl. ALREADY EXIST */
@@ -1557,8 +1659,8 @@ syncrepl_add_glue(
        op->o_req_dn = e->e_name;
        op->o_req_ndn = e->e_nname;
        op->ora_e = e;
-       rc = be->be_add ( op, &rs );
-       if ( rc == LDAP_SUCCESS ) {
+       rc = be->be_add ( op, &rs_add );
+       if ( rs_add.sr_err == LDAP_SUCCESS ) {
                be_entry_release_w( op, e );
        } else {
                entry_free( e );
@@ -1615,7 +1717,8 @@ syncrepl_updateCookie(
        struct berval slap_syncrepl_cn_bv = BER_BVNULL;
        
        slap_callback cb = { NULL };
-       SlapReply       rs = {REP_RESULT};
+       SlapReply       rs_add = {REP_RESULT};
+       SlapReply       rs_modify = {REP_RESULT};
 
        slap_sync_cookie_free( &si->si_syncCookie, 0 );
        slap_dup_sync_cookie( &si->si_syncCookie, syncCookie );
@@ -1641,21 +1744,24 @@ syncrepl_updateCookie(
        *modtail = mod;
        modtail = &mod->sml_next;
 
-       if ( scbva[0].bv_val ) ch_free( scbva[0].bv_val );
-       ber_dupbv( &scbva[0], &si->si_syncCookie.octet_str[0] );
        mod = (Modifications *) ch_calloc( 1, sizeof( Modifications ));
        mod->sml_op = LDAP_MOD_REPLACE;
-       mod->sml_desc = slap_schema.si_ad_syncreplCookie;
+       mod->sml_desc = slap_schema.si_ad_subtreeSpecification;
        mod->sml_type = mod->sml_desc->ad_cname;
-       mod->sml_values = scbva;
+       mod->sml_values = ssbva;
        *modtail = mod;
        modtail = &mod->sml_next;
 
+       /* Keep this last, so we can avoid touching the previous
+        * attributes unnecessarily.
+        */
+       if ( scbva[0].bv_val ) ch_free( scbva[0].bv_val );
+       ber_dupbv( &scbva[0], &si->si_syncCookie.octet_str[0] );
        mod = (Modifications *) ch_calloc( 1, sizeof( Modifications ));
        mod->sml_op = LDAP_MOD_REPLACE;
-       mod->sml_desc = slap_schema.si_ad_subtreeSpecification;
+       mod->sml_desc = slap_schema.si_ad_syncreplCookie;
        mod->sml_type = mod->sml_desc->ad_cname;
-       mod->sml_values = ssbva;
+       mod->sml_values = scbva;
        *modtail = mod;
        modtail = &mod->sml_next;
 
@@ -1663,20 +1769,15 @@ syncrepl_updateCookie(
 
        op->o_tag = LDAP_REQ_ADD;
        rc = slap_mods_opattrs( op, modlist, modtail,
-                &text, txtbuf, textlen );
+                &text, txtbuf, textlen, 0 );
 
        for ( ml = modlist; ml != NULL; ml = ml->sml_next ) {
                ml->sml_op = LDAP_MOD_REPLACE;
        }
 
        if( rc != LDAP_SUCCESS ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, ERR,
-                       "syncrepl_updateCookie: mods opattrs (%s)\n", text, 0, 0 );
-#else
                Debug( LDAP_DEBUG_ANY, "syncrepl_updateCookie: mods opattrs (%s)\n",
                         text, 0, 0 );
-#endif
        }
 
        e = ( Entry * ) ch_calloc( 1, sizeof( Entry ));
@@ -1692,8 +1793,8 @@ 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 ) {
-               sl_free( slap_syncrepl_dn_bv.bv_val, op->o_tmpmemctx );
+       if ( !BER_BVISNULL( &slap_syncrepl_dn_bv ) ) {
+               slap_sl_free( slap_syncrepl_dn_bv.bv_val, op->o_tmpmemctx );
        }
 
        e->e_attrs = NULL;
@@ -1701,13 +1802,8 @@ syncrepl_updateCookie(
        rc = slap_mods2entry( modlist, &e, 1, 1, &text, txtbuf, textlen );
 
        if( rc != LDAP_SUCCESS ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, ERR,
-                       "syncrepl_updateCookie: mods2entry (%s)\n", text, 0, 0 );
-#else
                Debug( LDAP_DEBUG_ANY, "syncrepl_updateCookie: mods2entry (%s)\n",
                         text, 0, 0 );
-#endif
        }
 
        cb.sc_response = null_callback;
@@ -1720,52 +1816,35 @@ syncrepl_updateCookie(
        /* update persistent cookie */
 update_cookie_retry:
        op->o_tag = LDAP_REQ_MODIFY;
-       op->orm_modlist = modlist;
-       rc = be->be_modify( op, &rs );
+       /* Just modify the cookie value, not the entire entry */
+       op->orm_modlist = mod;
+       rc = be->be_modify( op, &rs_modify );
 
-       if ( rc != LDAP_SUCCESS ) {
-               if ( rc == LDAP_REFERRAL ||
-                        rc == LDAP_NO_SUCH_OBJECT ) {
+       if ( rs_modify.sr_err != LDAP_SUCCESS ) {
+               if ( rs_modify.sr_err == LDAP_REFERRAL ||
+                        rs_modify.sr_err == LDAP_NO_SUCH_OBJECT ) {
                        op->o_tag = LDAP_REQ_ADD;
                        op->ora_e = e;
-                       rc = be->be_add( op, &rs );
-                       if ( rc != LDAP_SUCCESS ) {
-                               if ( rc == LDAP_ALREADY_EXISTS ) {
+                       rc = be->be_add( op, &rs_add );
+                       if ( rs_add.sr_err != LDAP_SUCCESS ) {
+                               if ( rs_add.sr_err == LDAP_ALREADY_EXISTS ) {
                                        goto update_cookie_retry;
-                               } else if ( rc == LDAP_REFERRAL ||
-                                                       rc == LDAP_NO_SUCH_OBJECT ) {
-#ifdef NEW_LOGGING
-                                       LDAP_LOG( OPERATION, ERR,
-                                               "cookie will be non-persistent\n",
-                                               0, 0, 0 );
-#else
+                               } else if ( rs_add.sr_err == LDAP_REFERRAL ||
+                                                       rs_add.sr_err == LDAP_NO_SUCH_OBJECT ) {
                                        Debug( LDAP_DEBUG_ANY,
                                                "cookie will be non-persistent\n",
                                                0, 0, 0 );
-#endif
                                } else {
-#ifdef NEW_LOGGING
-                                       LDAP_LOG( OPERATION, ERR,
-                                               "be_add failed (%d)\n",
-                                               rc, 0, 0 );
-#else
                                        Debug( LDAP_DEBUG_ANY,
-                                               "be_add failed (%d)\n",
-                                               rc, 0, 0 );
-#endif
+                                               "be_add failed (%d)\n", rs_add.sr_err, 0, 0 );
                                }
                        } else {
                                be_entry_release_w( op, e );
                                goto done;
                        }
                } else {
-#ifdef NEW_LOGGING
-                       LDAP_LOG( OPERATION, ERR,
-                               "be_modify failed (%d)\n", rc, 0, 0 );
-#else
                        Debug( LDAP_DEBUG_ANY,
-                               "be_modify failed (%d)\n", rc, 0, 0 );
-#endif
+                               "be_modify failed (%d)\n", rs_modify.sr_err, 0, 0 );
                }
        }
 
@@ -1775,13 +1854,13 @@ update_cookie_retry:
 
 done :
 
-       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 ) {
@@ -1829,16 +1908,18 @@ dn_callback(
        syncinfo_t *si = op->o_callback->sc_private;
 
        if ( rs->sr_type == REP_SEARCH ) {
-               if ( si->si_syncUUID_ndn.bv_val != NULL ) {
-#ifdef NEW_LOGGING
-                       LDAP_LOG( OPERATION, ERR,
-                               "dn_callback : multiple entries match dn\n", 0, 0, 0 );
-#else
+               if ( !BER_BVISNULL( &si->si_syncUUID_ndn ) ) {
                        Debug( LDAP_DEBUG_ANY,
-                               "dn_callback : multiple entries match dn\n", 0, 0, 0 );
-#endif
+                               "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, op->o_tmpmemctx );
+                       ber_dupbv_x( &si->si_syncUUID_ndn, &rs->sr_entry->e_nname, NULL );
+               }
+       } 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 );
                }
        }
 
@@ -1896,15 +1977,9 @@ null_callback(
                rs->sr_err != LDAP_NO_SUCH_OBJECT &&
                rs->sr_err != LDAP_NOT_ALLOWED_ON_NONLEAF )
        {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, ERR,
-                       "null_callback : error code 0x%x\n",
-                       rs->sr_err, 0, 0 );
-#else
                Debug( LDAP_DEBUG_ANY,
                        "null_callback : error code 0x%x\n",
                        rs->sr_err, 0, 0 );
-#endif
        }
        return LDAP_SUCCESS;
 }
@@ -1934,8 +2009,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 );
@@ -1960,13 +2034,18 @@ slap_uuidstr_from_normalized(
        if ( uuidstr ) {
                new = uuidstr;
        } else {
-               new = (struct berval *)sl_malloc( sizeof(struct berval), ctx );
+               new = (struct berval *)slap_sl_malloc( sizeof(struct berval), ctx );
+               if ( new == NULL ) {
+                       return NULL;
+               }
        }
 
        new->bv_len = 36;
 
-       if (( new->bv_val = sl_malloc( new->bv_len + 1, ctx )) == NULL) {
-               if ( !uuidstr ) 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;
        }
 
@@ -2006,11 +2085,124 @@ 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 );
 }
+
+void
+syncinfo_free( syncinfo_t *sie )
+{
+       if ( sie->si_provideruri ) {
+               ch_free( sie->si_provideruri );
+       }
+       if ( sie->si_provideruri_bv ) {
+               ber_bvarray_free( sie->si_provideruri_bv );
+       }
+       if ( sie->si_updatedn.bv_val ) {
+               ch_free( sie->si_updatedn.bv_val );
+       }
+       if ( sie->si_binddn ) {
+               ch_free( sie->si_binddn );
+       }
+       if ( sie->si_passwd ) {
+               ch_free( sie->si_passwd );
+       }
+       if ( sie->si_saslmech ) {
+               ch_free( sie->si_saslmech );
+       }
+       if ( sie->si_secprops ) {
+               ch_free( sie->si_secprops );
+       }
+       if ( sie->si_realm ) {
+               ch_free( sie->si_realm );
+       }
+       if ( sie->si_authcId ) {
+               ch_free( sie->si_authcId );
+       }
+       if ( sie->si_authzId ) {
+               ch_free( sie->si_authzId );
+       }
+       if ( sie->si_filterstr.bv_val ) {
+               ch_free( sie->si_filterstr.bv_val );
+       }
+       if ( sie->si_base.bv_val ) {
+               ch_free( sie->si_base.bv_val );
+       }
+       if ( sie->si_attrs ) {
+               int i = 0;
+               while ( sie->si_attrs[i] != NULL ) {
+                       ch_free( sie->si_attrs[i] );
+                       i++;
+               }
+               ch_free( sie->si_attrs );
+       }
+       if ( sie->si_exattrs ) {
+               int i = 0;
+               while ( sie->si_exattrs[i] != NULL ) {
+                       ch_free( sie->si_exattrs[i] );
+                       i++;
+               }
+               ch_free( sie->si_exattrs );
+       }
+       if ( sie->si_anlist ) {
+               int i = 0;
+               while ( sie->si_anlist[i].an_name.bv_val != NULL ) {
+                       ch_free( sie->si_anlist[i].an_name.bv_val );
+                       i++;
+               }
+               ch_free( sie->si_anlist );
+       }
+       if ( sie->si_exanlist ) {
+               int i = 0;
+               while ( sie->si_exanlist[i].an_name.bv_val != NULL ) {
+                       ch_free( sie->si_exanlist[i].an_name.bv_val );
+                       i++;
+               }
+               ch_free( sie->si_exanlist );
+       }
+       if ( sie->si_retryinterval ) {
+               ch_free( sie->si_retryinterval );
+       }
+       if ( sie->si_retrynum ) {
+               ch_free( sie->si_retrynum );
+       }
+       if ( sie->si_retrynum_init ) {
+               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 );
+       }
+       if ( sie->si_ld ) {
+               ldap_ld_free( sie->si_ld, 1, NULL, NULL );
+       }
+       while ( !LDAP_LIST_EMPTY( &sie->si_nonpresentlist )) {
+               struct nonpresent_entry* npe;
+               npe = LDAP_LIST_FIRST( &sie->si_nonpresentlist );
+               LDAP_LIST_REMOVE( npe, npe_link );
+               if ( npe->npe_name ) {
+                       if ( npe->npe_name->bv_val ) {
+                               ch_free( npe->npe_name->bv_val );
+                       }
+                       ch_free( npe->npe_name );
+               }
+               if ( npe->npe_nname ) {
+                       if ( npe->npe_nname->bv_val ) {
+                               ch_free( npe->npe_nname->bv_val );
+                       }
+                       ch_free( npe->npe_nname );
+               }
+               ch_free( npe );
+       }
+       ch_free( sie );
+}