]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/back-meta/search.c
happy new year
[openldap] / servers / slapd / back-meta / search.c
index 30e0d9ba22d5f972efcacab5495677b8e7c3fe15..05d41ed46d87a0998ac061ea1f0119af190ba552 100644 (file)
@@ -1,7 +1,7 @@
 /* $OpenLDAP$ */
 /* This work is part of OpenLDAP Software <http://www.openldap.org/>.
  *
- * Copyright 1999-2005 The OpenLDAP Foundation.
+ * Copyright 1999-2007 The OpenLDAP Foundation.
  * Portions Copyright 2001-2003 Pierangelo Masarati.
  * Portions Copyright 1999-2003 Howard Chu.
  * All rights reserved.
@@ -28,6 +28,7 @@
 #include <ac/string.h>
 #include <ac/time.h>
 
+#include "lutil.h"
 #include "slap.h"
 #include "../back-ldap/back-ldap.h"
 #include "back-meta.h"
 #include "ldap_log.h"
 #include "../../../libraries/libldap/ldap-int.h"
 
+/* IGNORE means that target does not (no longer) participate
+ * in the search;
+ * NOTREADY means the search on that target has not been initialized yet
+ */
+#define        META_MSGID_IGNORE       (-1)
+#define        META_MSGID_NEED_BIND    (-2)
+
 static int
 meta_send_entry(
        Operation       *op,
@@ -43,46 +51,359 @@ meta_send_entry(
        int             i,
        LDAPMessage     *e );
 
-static int
-meta_back_search_start(
+typedef enum meta_search_candidate_t {
+       META_SEARCH_ERR = -1,
+       META_SEARCH_NOT_CANDIDATE,
+       META_SEARCH_CANDIDATE,
+       META_SEARCH_BINDING,
+       META_SEARCH_NEED_BIND
+} meta_search_candidate_t;
+
+/*
+ * meta_search_dobind_init()
+ *
+ * initiates bind for a candidate target of a search.
+ */
+static meta_search_candidate_t
+meta_search_dobind_init(
        Operation               *op,
        SlapReply               *rs,
-       dncookie                *dc,
-       metasingleconn_t        *msc,
+       metaconn_t              **mcp,
        int                     candidate,
-       SlapReply               *candidates
-)
+       SlapReply               *candidates )
 {
-       metainfo_t      *mi = ( metainfo_t * )op->o_bd->be_private;
-       struct berval   realbase = op->o_req_dn;
-       int             realscope = op->ors_scope;
-       ber_len_t       suffixlen = 0;
-       struct berval   mbase = BER_BVNULL; 
-       struct berval   mfilter = BER_BVNULL;
-       char            **mapped_attrs = NULL;
-       int             rc;
+       metaconn_t              *mc = *mcp;
+       metainfo_t              *mi = ( metainfo_t * )op->o_bd->be_private;
+       metatarget_t            *mt = mi->mi_targets[ candidate ];
+       metasingleconn_t        *msc = &mc->mc_conns[ candidate ];
 
-       /* should we check return values? */
-       if ( op->ors_deref != -1 ) {
-               ldap_set_option( msc->msc_ld, LDAP_OPT_DEREF,
-                               ( void * )&op->ors_deref);
+       struct berval           binddn = msc->msc_bound_ndn,
+                               cred = msc->msc_cred;
+       int                     method;
+
+       int                     rc;
+
+       meta_search_candidate_t retcode;
+
+       Debug( LDAP_DEBUG_TRACE, "%s >>> meta_search_dobind_init[%d]\n",
+               op->o_log_prefix, candidate, 0 );
+
+       /*
+        * all the targets are already bound as pseudoroot
+        */
+       if ( mc->mc_authz_target == META_BOUND_ALL ) {
+               return META_SEARCH_CANDIDATE;
        }
-       if ( op->ors_tlimit != SLAP_NO_LIMIT ) {
-               ldap_set_option( msc->msc_ld, LDAP_OPT_TIMELIMIT,
-                               ( void * )&op->ors_tlimit);
+
+       retcode = META_SEARCH_BINDING;
+       ldap_pvt_thread_mutex_lock( &mi->mi_conninfo.lai_mutex );
+       if ( LDAP_BACK_CONN_ISBOUND( msc ) || LDAP_BACK_CONN_ISANON( msc ) ) {
+               /* already bound (or anonymous) */
+
+#ifdef DEBUG_205
+               char    buf[ SLAP_TEXT_BUFLEN ] = { '\0' };
+               int     bound = 0;
+
+               if ( LDAP_BACK_CONN_ISBOUND( msc ) ) {
+                       bound = 1;
+               }
+
+               snprintf( buf, sizeof( buf ), " mc=%p ld=%p%s DN=\"%s\"",
+                       (void *)mc, (void *)msc->msc_ld,
+                       bound ? " bound" : " anonymous",
+                       bound == 0 ? "" : msc->msc_bound_ndn.bv_val );
+               Debug( LDAP_DEBUG_ANY, "### %s meta_search_dobind_init[%d]%s\n",
+                       op->o_log_prefix, candidate, buf );
+#endif /* DEBUG_205 */
+
+               retcode = META_SEARCH_CANDIDATE;
+
+       } else if ( META_BACK_CONN_CREATING( msc ) || LDAP_BACK_CONN_BINDING( msc ) ) {
+               /* another thread is binding the target for this conn; wait */
+
+#ifdef DEBUG_205
+               char    buf[ SLAP_TEXT_BUFLEN ] = { '\0' };
+
+               snprintf( buf, sizeof( buf ), " mc=%p ld=%p needbind",
+                       (void *)mc, (void *)msc->msc_ld );
+               Debug( LDAP_DEBUG_ANY, "### %s meta_search_dobind_init[%d]%s\n",
+                       op->o_log_prefix, candidate, buf );
+#endif /* DEBUG_205 */
+
+               candidates[ candidate ].sr_msgid = META_MSGID_NEED_BIND;
+               retcode = META_SEARCH_NEED_BIND;
+
+       } else {
+               /* we'll need to bind the target for this conn */
+
+#ifdef DEBUG_205
+               char buf[ SLAP_TEXT_BUFLEN ];
+
+               snprintf( buf, sizeof( buf ), " mc=%p ld=%p binding",
+                       (void *)mc, (void *)msc->msc_ld );
+               Debug( LDAP_DEBUG_ANY, "### %s meta_search_dobind_init[%d]%s\n",
+                       op->o_log_prefix, candidate, buf );
+#endif /* DEBUG_205 */
+
+               if ( msc->msc_ld == NULL ) {
+                       /* for some reason (e.g. because formerly in "binding"
+                        * state, with eventual connection expiration or invalidation)
+                        * it was not initialized as expected */
+
+                       Debug( LDAP_DEBUG_ANY, "%s meta_search_dobind_init[%d] mc=%p ld=NULL\n",
+                               op->o_log_prefix, candidate, (void *)mc );
+
+                       rc = meta_back_init_one_conn( op, rs, *mcp, candidate,
+                               LDAP_BACK_CONN_ISPRIV( *mcp ), LDAP_BACK_DONTSEND, 0 );
+                       switch ( rc ) {
+                       case LDAP_SUCCESS:
+                               assert( msc->msc_ld != NULL );
+                               break;
+
+                       case LDAP_SERVER_DOWN:
+                       case LDAP_UNAVAILABLE:
+                               ldap_pvt_thread_mutex_unlock( &mi->mi_conninfo.lai_mutex );
+                               goto down;
+       
+                       default:
+                               ldap_pvt_thread_mutex_unlock( &mi->mi_conninfo.lai_mutex );
+                               goto other;
+                       }
+               }
+
+               LDAP_BACK_CONN_BINDING_SET( msc );
+       }
+
+       ldap_pvt_thread_mutex_unlock( &mi->mi_conninfo.lai_mutex );
+
+       if ( retcode != META_SEARCH_BINDING ) {
+               return retcode;
+       }
+
+       /* NOTE: this obsoletes pseudorootdn */
+       if ( op->o_conn != NULL &&
+               !op->o_do_not_cache &&
+               ( BER_BVISNULL( &msc->msc_bound_ndn ) ||
+                       BER_BVISEMPTY( &msc->msc_bound_ndn ) ||
+                       ( mt->mt_idassert_flags & LDAP_BACK_AUTH_OVERRIDE ) ) )
+       {
+               rc = meta_back_proxy_authz_cred( mc, candidate, op, rs, LDAP_BACK_DONTSEND, &binddn, &cred, &method );
+               if ( rc != LDAP_SUCCESS ) {
+                       goto down;
+               }
+
+               /* NOTE: we copy things here, even if bind didn't succeed yet,
+                * because the connection is not shared until bind is over */
+               if ( !BER_BVISNULL( &binddn ) ) {
+                       ber_bvreplace( &msc->msc_bound_ndn, &binddn );
+                       if ( LDAP_BACK_SAVECRED( mi ) && !BER_BVISNULL( &cred ) ) {
+                               if ( !BER_BVISNULL( &msc->msc_cred ) ) {
+                                       memset( msc->msc_cred.bv_val, 0,
+                                               msc->msc_cred.bv_len );
+                               }
+                               ber_bvreplace( &msc->msc_cred, &cred );
+                       }
+               }
+
+               if ( LDAP_BACK_CONN_ISBOUND( msc ) ) {
+                       /* apparently, idassert was configured with SASL bind,
+                        * so bind occurred inside meta_back_proxy_authz_cred() */
+                       ldap_pvt_thread_mutex_lock( &mi->mi_conninfo.lai_mutex );
+                       LDAP_BACK_CONN_BINDING_CLEAR( msc );
+                       ldap_pvt_thread_mutex_unlock( &mi->mi_conninfo.lai_mutex );
+                       return META_SEARCH_CANDIDATE;
+               }
+
+               /* paranoid */
+               switch ( method ) {
+               case LDAP_AUTH_NONE:
+               case LDAP_AUTH_SIMPLE:
+                       /* do a simple bind with binddn, cred */
+                       break;
+
+               default:
+                       assert( 0 );
+                       break;
+               }
+       }
+
+       assert( msc->msc_ld != NULL );
+
+       rc = ldap_sasl_bind( msc->msc_ld, binddn.bv_val, LDAP_SASL_SIMPLE, &cred,
+                       NULL, NULL, &candidates[ candidate ].sr_msgid );
+
+#ifdef DEBUG_205
+       {
+               char buf[ SLAP_TEXT_BUFLEN ];
+
+               snprintf( buf, sizeof( buf ), "meta_search_dobind_init[%d] mc=%p ld=%p rc=%d",
+                       candidate, (void *)mc, (void *)mc->mc_conns[ candidate ].msc_ld, rc );
+               Debug( LDAP_DEBUG_ANY, "### %s %s\n",
+                       op->o_log_prefix, buf, 0 );
+       }
+#endif /* DEBUG_205 */
+
+       switch ( rc ) {
+       case LDAP_SUCCESS:
+               assert( candidates[ candidate ].sr_msgid >= 0 );
+               META_BINDING_SET( &candidates[ candidate ] );
+               return META_SEARCH_BINDING;
+
+       case LDAP_SERVER_DOWN:
+down:;
+               /* This is the worst thing that could happen:
+                * the search will wait until the retry is over. */
+               if ( meta_back_retry( op, rs, mcp, candidate, LDAP_BACK_DONTSEND ) ) {
+                       candidates[ candidate ].sr_msgid = META_MSGID_IGNORE;
+                       return META_SEARCH_CANDIDATE;
+               }
+
+               if ( *mcp == NULL ) {
+                       retcode = META_SEARCH_ERR;
+                       rs->sr_err = LDAP_UNAVAILABLE;
+                       break;
+               }
+               /* fall thru */
+
+       default:
+other:;
+               rs->sr_err = rc;
+               rc = slap_map_api2result( rs );
+
+               ldap_pvt_thread_mutex_lock( &mi->mi_conninfo.lai_mutex );
+               meta_clear_one_candidate( op, mc, candidate );
+               if ( META_BACK_ONERR_STOP( mi ) ) {
+                       LDAP_BACK_CONN_TAINTED_SET( mc );
+                       meta_back_release_conn_lock( op, mc, 0 );
+                       *mcp = NULL;
+
+                       retcode = META_SEARCH_ERR;
+
+               } else {
+                       if ( META_BACK_ONERR_REPORT( mi ) ) {
+                               candidates[ candidate ].sr_err = rc;
+                       }
+
+                       retcode = META_SEARCH_NOT_CANDIDATE;
+               }
+               candidates[ candidate ].sr_msgid = META_MSGID_IGNORE;
+               ldap_pvt_thread_mutex_unlock( &mi->mi_conninfo.lai_mutex );
+               break;
        }
-       if ( op->ors_slimit != SLAP_NO_LIMIT ) {
-               ldap_set_option( msc->msc_ld, LDAP_OPT_SIZELIMIT,
-                               ( void * )&op->ors_slimit);
+
+       return retcode;
+}
+
+static meta_search_candidate_t
+meta_search_dobind_result(
+       Operation               *op,
+       SlapReply               *rs,
+       metaconn_t              **mcp,
+       int                     candidate,
+       SlapReply               *candidates,
+       LDAPMessage             *res )
+{
+       metainfo_t              *mi = ( metainfo_t * )op->o_bd->be_private;
+       metaconn_t              *mc = *mcp;
+       metasingleconn_t        *msc = &mc->mc_conns[ candidate ];
+
+       meta_search_candidate_t retcode = META_SEARCH_NOT_CANDIDATE;
+       int                     rc;
+
+       assert( msc->msc_ld != NULL );
+
+       /* FIXME: matched? referrals? response controls? */
+       rc = ldap_parse_result( msc->msc_ld, res,
+               &candidates[ candidate ].sr_err,
+               NULL, NULL, NULL, NULL, 0 );
+       if ( rc != LDAP_SUCCESS ) {
+               candidates[ candidate ].sr_err = rc;
+
+       } else {
+               rc = slap_map_api2result( &candidates[ candidate ] );
        }
 
-       dc->target = &mi->mi_targets[ candidate ];
+       ldap_pvt_thread_mutex_lock( &mi->mi_conninfo.lai_mutex );
+       LDAP_BACK_CONN_BINDING_CLEAR( msc );
+       if ( rc != LDAP_SUCCESS ) {
+               if ( META_BACK_ONERR_STOP( mi ) ) {
+                       LDAP_BACK_CONN_TAINTED_SET( mc );
+                       meta_clear_one_candidate( op, mc, candidate );
+                       meta_back_release_conn_lock( op, mc, 0 );
+                       *mcp = NULL;
+                       retcode = META_SEARCH_ERR;
+                       rs->sr_err = rc;
+
+               } else if ( META_BACK_ONERR_REPORT( mi ) ) {
+                       candidates[ candidate ].sr_err = rc;
+               }
+
+       } else {
+               /* FIXME: check if bound as idassert authcDN! */
+               if ( BER_BVISNULL( &msc->msc_bound_ndn )
+                       || BER_BVISEMPTY( &msc->msc_bound_ndn ) )
+               {
+                       LDAP_BACK_CONN_ISANON_SET( msc );
+
+               } else {
+                       LDAP_BACK_CONN_ISBOUND_SET( msc );
+               }
+               retcode = META_SEARCH_CANDIDATE;
+       }
+
+       candidates[ candidate ].sr_msgid = META_MSGID_IGNORE;
+       META_BINDING_CLEAR( &candidates[ candidate ] );
+
+       ldap_pvt_thread_mutex_unlock( &mi->mi_conninfo.lai_mutex );
+
+       return retcode;
+}
+
+static meta_search_candidate_t
+meta_back_search_start(
+       Operation               *op,
+       SlapReply               *rs,
+       dncookie                *dc,
+       metaconn_t              **mcp,
+       int                     candidate,
+       SlapReply               *candidates )
+{
+       metainfo_t              *mi = ( metainfo_t * )op->o_bd->be_private;
+       metatarget_t            *mt = mi->mi_targets[ candidate ];
+       metasingleconn_t        *msc = &(*mcp)->mc_conns[ candidate ];
+       struct berval           realbase = op->o_req_dn;
+       int                     realscope = op->ors_scope;
+       struct berval           mbase = BER_BVNULL; 
+       struct berval           mfilter = BER_BVNULL;
+       char                    **mapped_attrs = NULL;
+       int                     rc;
+       meta_search_candidate_t retcode;
+       struct timeval          tv, *tvp = NULL;
+       int                     nretries = 1;
+       LDAPControl             **ctrls = NULL;
+
+       /* this should not happen; just in case... */
+       if ( msc->msc_ld == NULL ) {
+               Debug( LDAP_DEBUG_ANY,
+                       "%s: meta_back_search_start candidate=%d ld=NULL%s.\n",
+                       op->o_log_prefix, candidate,
+                       META_BACK_ONERR_STOP( mi ) ? "" : " (ignored)" );
+               if ( META_BACK_ONERR_STOP( mi ) ) {
+                       return META_SEARCH_ERR;
+               }
+               if ( META_BACK_ONERR_REPORT( mi ) ) {
+                       candidates[ candidate ].sr_err = LDAP_OTHER;
+               }
+               candidates[ candidate ].sr_msgid = META_MSGID_IGNORE;
+               return META_SEARCH_NOT_CANDIDATE;
+       }
+
+       Debug( LDAP_DEBUG_TRACE, "%s >>> meta_back_search_start[%d]\n", op->o_log_prefix, candidate, 0 );
 
        /*
         * modifies the base according to the scope, if required
         */
-       suffixlen = mi->mi_targets[ candidate ].mt_nsuffix.bv_len;
-       if ( suffixlen > op->o_req_ndn.bv_len ) {
+       if ( mt->mt_nsuffix.bv_len > op->o_req_ndn.bv_len ) {
                switch ( op->ors_scope ) {
                case LDAP_SCOPE_SUBTREE:
                        /*
@@ -93,41 +414,42 @@ meta_back_search_start(
                         * the requested searchBase already passed
                         * thru the candidate analyzer...
                         */
-                       if ( dnIsSuffix( &mi->mi_targets[ candidate ].mt_nsuffix,
-                                       &op->o_req_ndn ) )
-                       {
-                               realbase = mi->mi_targets[ candidate ].mt_nsuffix;
+                       if ( dnIsSuffix( &mt->mt_nsuffix, &op->o_req_ndn ) ) {
+                               realbase = mt->mt_nsuffix;
+                               if ( mt->mt_scope == LDAP_SCOPE_SUBORDINATE ) {
+                                       realscope = LDAP_SCOPE_SUBORDINATE;
+                               }
 
                        } else {
                                /*
                                 * this target is no longer candidate
                                 */
-                               return 0;
+                               retcode = META_SEARCH_NOT_CANDIDATE;
+                               goto doreturn;
                        }
                        break;
 
-#ifdef LDAP_SCOPE_SUBORDINATE
                case LDAP_SCOPE_SUBORDINATE:
-#endif /* LDAP_SCOPE_SUBORDINATE */
                case LDAP_SCOPE_ONELEVEL:
                {
-                       struct berval   rdn = mi->mi_targets[ candidate ].mt_nsuffix;
+                       struct berval   rdn = mt->mt_nsuffix;
                        rdn.bv_len -= op->o_req_ndn.bv_len + STRLENOF( "," );
                        if ( dnIsOneLevelRDN( &rdn )
-                                       && dnIsSuffix( &mi->mi_targets[ candidate ].mt_nsuffix, &op->o_req_ndn ) )
+                                       && dnIsSuffix( &mt->mt_nsuffix, &op->o_req_ndn ) )
                        {
                                /*
                                 * if there is exactly one level,
                                 * make the target suffix the new
                                 * base, and make scope "base"
                                 */
-                               realbase = mi->mi_targets[ candidate ].mt_nsuffix;
-#ifdef LDAP_SCOPE_SUBORDINATE
+                               realbase = mt->mt_nsuffix;
                                if ( op->ors_scope == LDAP_SCOPE_SUBORDINATE ) {
-                                       realscope = LDAP_SCOPE_SUBTREE;
-                               } else
-#endif /* LDAP_SCOPE_SUBORDINATE */
-                               {
+                                       if ( mt->mt_scope == LDAP_SCOPE_SUBORDINATE ) {
+                                               realscope = LDAP_SCOPE_SUBORDINATE;
+                                       } else {
+                                               realscope = LDAP_SCOPE_SUBTREE;
+                                       }
+                               } else {
                                        realscope = LDAP_SCOPE_BASE;
                                }
                                break;
@@ -138,30 +460,43 @@ meta_back_search_start(
                        /*
                         * this target is no longer candidate
                         */
-                       return 0;
+                       retcode = META_SEARCH_NOT_CANDIDATE;
+                       goto doreturn;
                }
        }
 
+       /* initiate dobind */
+       retcode = meta_search_dobind_init( op, rs, mcp, candidate, candidates );
+
+       Debug( LDAP_DEBUG_TRACE, "%s <<< meta_search_dobind_init[%d]=%d\n", op->o_log_prefix, candidate, retcode );
+
+       if ( retcode != META_SEARCH_CANDIDATE ) {
+               goto doreturn;
+       }
+
        /*
         * Rewrite the search base, if required
         */
+       dc->target = mt;
        dc->ctx = "searchBase";
        switch ( ldap_back_dn_massage( dc, &realbase, &mbase ) ) {
-       default:
+       case LDAP_SUCCESS:
                break;
 
-       case REWRITE_REGEXEC_UNWILLING:
+       case LDAP_UNWILLING_TO_PERFORM:
                rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
                rs->sr_text = "Operation not allowed";
                send_ldap_result( op, rs );
-               return -1;
+               retcode = META_SEARCH_ERR;
+               goto doreturn;
 
-       case REWRITE_REGEXEC_ERR:
+       default:
 
                /*
                 * this target is no longer candidate
                 */
-               return 0;
+               retcode = META_SEARCH_NOT_CANDIDATE;
+               goto doreturn;
        }
 
        /*
@@ -178,40 +513,83 @@ meta_back_search_start(
                /*
                 * this target is no longer candidate
                 */
-               rc = 0;
+               retcode = META_SEARCH_NOT_CANDIDATE;
                goto done;
        }
 
        /*
         * Maps required attributes
         */
-       rc = ldap_back_map_attrs( &mi->mi_targets[ candidate ].mt_rwmap.rwm_at,
+       rc = ldap_back_map_attrs( &mt->mt_rwmap.rwm_at,
                        op->ors_attrs, BACKLDAP_MAP, &mapped_attrs );
        if ( rc != LDAP_SUCCESS ) {
                /*
                 * this target is no longer candidate
                 */
-               rc = 0;
+               retcode = META_SEARCH_NOT_CANDIDATE;
+               goto done;
+       }
+
+       /* should we check return values? */
+       if ( op->ors_deref != -1 ) {
+               assert( msc->msc_ld != NULL );
+               (void)ldap_set_option( msc->msc_ld, LDAP_OPT_DEREF,
+                               ( void * )&op->ors_deref );
+       }
+
+       if ( op->ors_tlimit != SLAP_NO_LIMIT ) {
+               tv.tv_sec = op->ors_tlimit > 0 ? op->ors_tlimit : 1;
+               tv.tv_usec = 0;
+               tvp = &tv;
+       }
+
+retry:;
+       ctrls = op->o_ctrls;
+       if ( ldap_back_proxy_authz_ctrl( &msc->msc_bound_ndn,
+               mt->mt_version, &mt->mt_idassert, op, rs, &ctrls )
+               != LDAP_SUCCESS )
+       {
+               candidates[ candidate ].sr_msgid = META_MSGID_IGNORE;
+               retcode = META_SEARCH_NOT_CANDIDATE;
                goto done;
        }
 
        /*
         * Starts the search
         */
+       assert( msc->msc_ld != NULL );
        rc = ldap_search_ext( msc->msc_ld,
                        mbase.bv_val, realscope, mfilter.bv_val,
                        mapped_attrs, op->ors_attrsonly,
-                       op->o_ctrls, NULL, NULL, op->ors_slimit,
+                       ctrls, NULL, tvp, op->ors_slimit,
                        &candidates[ candidate ].sr_msgid ); 
-       if ( rc == LDAP_SUCCESS ) {
-               rc = 1;
+       switch ( rc ) {
+       case LDAP_SUCCESS:
+               retcode = META_SEARCH_CANDIDATE;
+               break;
+       
+       case LDAP_SERVER_DOWN:
+               if ( nretries && meta_back_retry( op, rs, mcp, candidate, LDAP_BACK_DONTSEND ) ) {
+                       nretries = 0;
+                       /* if the identity changed, there might be need to re-authz */
+                       (void)ldap_back_proxy_authz_ctrl_free( op, &ctrls );
+                       goto retry;
+               }
 
-       } else {
-               candidates[ candidate ].sr_msgid = -1;
-               rc = 0;
+               if ( *mcp == NULL ) {
+                       retcode = META_SEARCH_ERR;
+                       break;
+               }
+               /* fall thru */
+
+       default:
+               candidates[ candidate ].sr_msgid = META_MSGID_IGNORE;
+               retcode = META_SEARCH_NOT_CANDIDATE;
        }
 
 done:;
+       (void)ldap_back_proxy_authz_ctrl_free( op, &ctrls );
+
        if ( mapped_attrs ) {
                free( mapped_attrs );
        }
@@ -222,7 +600,10 @@ done:;
                free( mbase.bv_val );
        }
 
-       return rc;
+doreturn:;
+       Debug( LDAP_DEBUG_TRACE, "%s <<< meta_back_search_start[%d]=%d\n", op->o_log_prefix, candidate, retcode );
+
+       return retcode;
 }
 
 int
@@ -230,13 +611,16 @@ meta_back_search( Operation *op, SlapReply *rs )
 {
        metainfo_t      *mi = ( metainfo_t * )op->o_bd->be_private;
        metaconn_t      *mc;
-       struct timeval  tv = { 0, 0 };
-       time_t          stoptime;
-       LDAPMessage     *res = NULL, *e;
+       struct timeval  save_tv = { 0, 0 },
+                       tv;
+       time_t          stoptime = (time_t)-1;
        int             rc = 0, sres = LDAP_SUCCESS;
        char            *matched = NULL;
-       int             i, last = 0, ncandidates = 0,
-                       initial_candidates = 0, candidate_match = 0;
+       int             last = 0, ncandidates = 0,
+                       initial_candidates = 0, candidate_match = 0,
+                       needbind = 0;
+       ldap_back_send_t        sendok = LDAP_BACK_SENDERR;
+       long            i;
        dncookie        dc;
        int             is_ok = 0;
        void            *savepriv;
@@ -248,8 +632,9 @@ meta_back_search( Operation *op, SlapReply *rs )
         * FIXME: in case of values return filter, we might want
         * to map attrs and maybe rewrite value
         */
-       mc = meta_back_getconn( op, rs, NULL, LDAP_BACK_SENDERR );
-       if ( !mc || !meta_back_dobind( op, rs, mc, LDAP_BACK_SENDERR ) ) {
+getconn:;
+       mc = meta_back_getconn( op, rs, NULL, sendok );
+       if ( !mc ) {
                return rs->sr_err;
        }
 
@@ -260,61 +645,143 @@ meta_back_search( Operation *op, SlapReply *rs )
         * Inits searches
         */
        for ( i = 0; i < mi->mi_ntargets; i++ ) {
-               metasingleconn_t        *msc = &mc->mc_conns[ i ];
-
-               candidates[ i ].sr_msgid = -1;
-
-               if ( candidates[ i ].sr_tag != META_CANDIDATE ) {
+               /* reset sr_msgid; it is used in most loops
+                * to check if that target is still to be considered */
+               candidates[ i ].sr_msgid = META_MSGID_IGNORE;
+
+               /* a target is marked as candidate by meta_back_getconn();
+                * if for any reason (an error, it's over or so) it is
+                * no longer active, sr_msgid is set to META_MSGID_IGNORE
+                * but it remains candidate, which means it has been active
+                * at some point during the operation.  This allows to 
+                * use its response code and more to compute the final
+                * response */
+               if ( !META_IS_CANDIDATE( &candidates[ i ] ) ) {
                        continue;
                }
 
-               candidates[ i ].sr_err = LDAP_SUCCESS;
                candidates[ i ].sr_matched = NULL;
                candidates[ i ].sr_text = NULL;
                candidates[ i ].sr_ref = NULL;
                candidates[ i ].sr_ctrls = NULL;
+       }
 
-               switch ( meta_back_search_start( op, rs, &dc, msc, i, candidates ) )
+       for ( i = 0; i < mi->mi_ntargets; i++ ) {
+               if ( !META_IS_CANDIDATE( &candidates[ i ] )
+                       || candidates[ i ].sr_err != LDAP_SUCCESS )
                {
-               case 0:
+                       continue;
+               }
+
+               switch ( meta_back_search_start( op, rs, &dc, &mc, i, candidates ) )
+               {
+               case META_SEARCH_NOT_CANDIDATE:
+                       candidates[ i ].sr_msgid = META_MSGID_IGNORE;
                        break;
 
-               case 1:
+               case META_SEARCH_NEED_BIND:
+                       ++needbind;
+                       /* fallthru */
+
+               case META_SEARCH_CANDIDATE:
+               case META_SEARCH_BINDING:
+                       candidates[ i ].sr_type = REP_INTERMEDIATE;
                        ++ncandidates;
                        break;
 
-               case -1:
+               case META_SEARCH_ERR:
+                       savepriv = op->o_private;
+                       op->o_private = (void *)i;
+                       send_ldap_result( op, rs );
+                       op->o_private = savepriv;
                        rc = -1;
                        goto finish;
                }
        }
 
+       if ( ncandidates > 0 && needbind == ncandidates ) {
+               /*
+                * give up the second time...
+                *
+                * NOTE: this should not occur the second time, since a fresh
+                * connection has ben created; however, targets may also
+                * need bind because the bind timed out or so.
+                */
+               if ( sendok & LDAP_BACK_BINDING ) {
+                       Debug( LDAP_DEBUG_ANY,
+                               "%s meta_back_search: unable to initialize conn\n",
+                               op->o_log_prefix, 0, 0 );
+                       rs->sr_err = LDAP_UNAVAILABLE;
+                       rs->sr_text = "unable to initialize connection to remote targets";
+                       send_ldap_result( op, rs );
+                       rc = -1;
+                       goto finish;
+               }
+
+               /* FIXME: better create a separate connection? */
+               sendok |= LDAP_BACK_BINDING;
+
+#ifdef DEBUG_205
+               Debug( LDAP_DEBUG_ANY, "*** %s drop mc=%p create new connection\n",
+                       op->o_log_prefix, (void *)mc, 0 );
+#endif /* DEBUG_205 */
+
+               meta_back_release_conn( op, mc );
+               mc = NULL;
+
+               needbind = 0;
+               ncandidates = 0;
+
+               goto getconn;
+       }
+
        initial_candidates = ncandidates;
 
-#if 0
-       {
-               char    cnd[BUFSIZ];
-               int     i;
+       if ( LogTest( LDAP_DEBUG_TRACE ) ) {
+               char    cnd[ SLAP_TEXT_BUFLEN ];
+               int     c;
 
-               for ( i = 0; i < mi->mi_ntargets; i++ ) {
-                       if ( candidates[ i ].sr_tag == META_CANDIDATE ) {
-                               cnd[ i ] = '*';
+               for ( c = 0; c < mi->mi_ntargets; c++ ) {
+                       if ( META_IS_CANDIDATE( &candidates[ c ] ) ) {
+                               cnd[ c ] = '*';
                        } else {
-                               cnd[ i ] = ' ';
+                               cnd[ c ] = ' ';
                        }
                }
-               cnd[ i ] = '\0';
+               cnd[ c ] = '\0';
 
-               Debug( LDAP_DEBUG_ANY, "%s meta_back_search: ncandidates=%d "
+               Debug( LDAP_DEBUG_TRACE, "%s meta_back_search: ncandidates=%d "
                        "cnd=\"%s\"\n", op->o_log_prefix, ncandidates, cnd );
        }
-#endif
 
        if ( initial_candidates == 0 ) {
-               send_ldap_error( op, rs, LDAP_NO_SUCH_OBJECT, NULL );
-               /* FIXME: find a way to look up the best match */
-
+               /* NOTE: here we are not sending any matchedDN;
+                * this is intended, because if the back-meta
+                * is serving this search request, but no valid
+                * candidate could be looked up, it means that
+                * there is a hole in the mapping of the targets
+                * and thus no knowledge of any remote superior
+                * is available */
+               Debug( LDAP_DEBUG_ANY, "%s meta_back_search: "
+                       "base=\"%s\" scope=%d: "
+                       "no candidate could be selected\n",
+                       op->o_log_prefix, op->o_req_dn.bv_val,
+                       op->ors_scope );
+
+               /* FIXME: we're sending the first error we encounter;
+                * maybe we should pick the worst... */
                rc = LDAP_NO_SUCH_OBJECT;
+               for ( i = 0; i < mi->mi_ntargets; i++ ) {
+                       if ( META_IS_CANDIDATE( &candidates[ i ] )
+                               && candidates[ i ].sr_err != LDAP_SUCCESS )
+                       {
+                               rc = candidates[ i ].sr_err;
+                               break;
+                       }
+               }
+
+               send_ldap_error( op, rs, rc, NULL );
+
                goto finish;
        }
 
@@ -325,7 +792,6 @@ meta_back_search( Operation *op, SlapReply *rs )
 
        if ( op->ors_tlimit != SLAP_NO_LIMIT ) {
                stoptime = op->o_time + op->ors_tlimit;
-               tv.tv_sec = 0;
        }
 
        /*
@@ -335,12 +801,114 @@ meta_back_search( Operation *op, SlapReply *rs )
         * among the candidates
         */
        for ( rc = 0; ncandidates > 0; ) {
-               int     gotit = 0, doabandon = 0;
+               int     gotit = 0,
+                       doabandon = 0,
+                       alreadybound = ncandidates;
+
+               /* check time limit */
+               if ( op->ors_tlimit != SLAP_NO_LIMIT
+                               && slap_get_time() > stoptime )
+               {
+                       doabandon = 1;
+                       rc = rs->sr_err = LDAP_TIMELIMIT_EXCEEDED;
+                       savepriv = op->o_private;
+                       op->o_private = (void *)i;
+                       send_ldap_result( op, rs );
+                       op->o_private = savepriv;
+                       goto finish;
+               }
 
                for ( i = 0; i < mi->mi_ntargets; i++ ) {
                        metasingleconn_t        *msc = &mc->mc_conns[ i ];
+                       LDAPMessage             *res = NULL, *msg;
 
-                       if ( candidates[ i ].sr_msgid == -1 ) {
+                       /* if msgid is invalid, don't ldap_result() */
+                       if ( candidates[ i ].sr_msgid == META_MSGID_IGNORE ) {
+                               continue;
+                       }
+
+                       /* if target still needs bind, retry */
+                       if ( candidates[ i ].sr_msgid == META_MSGID_NEED_BIND ) {
+                               meta_search_candidate_t retcode;
+
+                               /* initiate dobind */
+                               retcode = meta_search_dobind_init( op, rs, &mc, i, candidates );
+
+                               Debug( LDAP_DEBUG_TRACE, "%s <<< meta_search_dobind_init[%ld]=%d\n",
+                                       op->o_log_prefix, i, retcode );
+
+                               switch ( retcode ) {
+                               case META_SEARCH_NEED_BIND:
+                                       alreadybound--;
+                                       /* fallthru */
+
+                               case META_SEARCH_BINDING:
+                                       break;
+
+                               case META_SEARCH_ERR:
+                                       if ( META_BACK_ONERR_STOP( mi ) ) {
+                                               savepriv = op->o_private;
+                                               op->o_private = (void *)i;
+                                               send_ldap_result( op, rs );
+                                               op->o_private = savepriv;
+                                               goto finish;
+                                       }
+                                       if ( META_BACK_ONERR_REPORT( mi ) ) {
+                                               candidates[ i ].sr_err = rs->sr_err;
+                                       }
+                                       /* fallthru */
+
+                               case META_SEARCH_NOT_CANDIDATE:
+                                       /*
+                                        * When no candidates are left,
+                                        * the outer cycle finishes
+                                        */
+                                       candidates[ i ].sr_msgid = META_MSGID_IGNORE;
+                                       assert( ncandidates > 0 );
+                                       --ncandidates;
+                                       break;
+
+                               case META_SEARCH_CANDIDATE:
+                                       candidates[ i ].sr_msgid = META_MSGID_IGNORE;
+                                       switch ( meta_back_search_start( op, rs, &dc, &mc, i, candidates ) )
+                                       {
+                                       case META_SEARCH_CANDIDATE:
+                                               assert( candidates[ i ].sr_msgid >= 0 );
+                                               break;
+
+                                       case META_SEARCH_ERR:
+                                               if ( META_BACK_ONERR_STOP( mi ) ) {
+                                                       savepriv = op->o_private;
+                                                       op->o_private = (void *)i;
+                                                       send_ldap_result( op, rs );
+                                                       op->o_private = savepriv;
+                                                       goto finish;
+                                               }
+                                               if ( META_BACK_ONERR_REPORT( mi ) ) {
+                                                       candidates[ i ].sr_err = rs->sr_err;
+                                               }
+                                               /* fallthru */
+
+                                       case META_SEARCH_NOT_CANDIDATE:
+                                               /* means that meta_back_search_start()
+                                                * failed but onerr == continue */
+                                               candidates[ i ].sr_msgid = META_MSGID_IGNORE;
+                                               assert( ncandidates > 0 );
+                                               --ncandidates;
+                                               break;
+
+                                       default:
+                                               /* impossible */
+                                               assert( 0 );
+                                               break;
+                                       }
+                                       break;
+
+                               default:
+                                       /* impossible */
+                                       assert( 0 );
+                                       break;
+                               }
                                continue;
                        }
 
@@ -348,6 +916,24 @@ meta_back_search( Operation *op, SlapReply *rs )
                        if ( op->o_abandon ) {
                                break;
                        }
+
+#ifdef DEBUG_205
+                       if ( msc->msc_ld == NULL ) {
+                               char    buf[ SLAP_TEXT_BUFLEN ];
+
+                               ldap_pvt_thread_mutex_lock( &mi->mi_conninfo.lai_mutex );
+                               snprintf( buf, sizeof( buf ),
+                                       "%s meta_back_search[%ld] mc=%p msgid=%d%s%s%s\n",
+                                       op->o_log_prefix, (long)i, (void *)mc,
+                                       candidates[ i ].sr_msgid,
+                                       META_IS_BINDING( &candidates[ i ] ) ? " binding" : "",
+                                       LDAP_BACK_CONN_BINDING( &mc->mc_conns[ i ] ) ? " connbinding" : "",
+                                       META_BACK_CONN_CREATING( &mc->mc_conns[ i ] ) ? " conncreating" : "" );
+                               ldap_pvt_thread_mutex_unlock( &mi->mi_conninfo.lai_mutex );
+                                       
+                               Debug( LDAP_DEBUG_ANY, "!!! %s\n", buf, 0, 0 );
+                       }
+#endif /* DEBUG_205 */
                        
                        /*
                         * FIXME: handle time limit as well?
@@ -356,195 +942,247 @@ meta_back_search( Operation *op, SlapReply *rs )
                         * get a LDAP_TIMELIMIT_EXCEEDED from
                         * one of them ...
                         */
+                       tv = save_tv;
                        rc = ldap_result( msc->msc_ld, candidates[ i ].sr_msgid,
-                                       0, &tv, &res );
-
-                       if ( rc == 0 ) {
+                                       LDAP_MSG_RECEIVED, &tv, &res );
+                       switch ( rc ) {
+                       case 0:
                                /* FIXME: res should not need to be freed */
                                assert( res == NULL );
-
-                               /* check time limit */
-                               if ( op->ors_tlimit != SLAP_NO_LIMIT
-                                               && slap_get_time() > stoptime )
-                               {
-                                       doabandon = 1;
-                                       rc = rs->sr_err = LDAP_TIMELIMIT_EXCEEDED;
-                                       savepriv = op->o_private;
-                                       op->o_private = (void *)i;
-                                       send_ldap_result( op, rs );
-                                       op->o_private = savepriv;
-                                       goto finish;
-                               }
-
                                continue;
 
-                       } else if ( rc == -1 ) {
+                       case -1:
 really_bad:;
                                /* something REALLY bad happened! */
-                               ( void )meta_clear_unused_candidates( op, -1 );
-                               rs->sr_err = LDAP_OTHER;
-                               savepriv = op->o_private;
-                               op->o_private = (void *)i;
-                               send_ldap_result( op, rs );
-                               op->o_private = savepriv;
-                               
-                               /* anything else needs be done? */
-
-                               /* FIXME: res should not need to be freed */
-                               assert( res == NULL );
+                               if ( candidates[ i ].sr_type == REP_INTERMEDIATE ) {
+                                       candidates[ i ].sr_type = REP_RESULT;
+
+                                       if ( meta_back_retry( op, rs, &mc, i, LDAP_BACK_DONTSEND ) ) {
+                                               candidates[ i ].sr_msgid = META_MSGID_IGNORE;
+                                               switch ( meta_back_search_start( op, rs, &dc, &mc, i, candidates ) )
+                                               {
+                                               case META_SEARCH_CANDIDATE:
+                                                       /* get back into business... */
+                                                       continue;
+
+                                                       /* means that failed but onerr == continue */
+                                               case META_SEARCH_NOT_CANDIDATE:
+                                                       candidates[ i ].sr_msgid = META_MSGID_IGNORE;
+                                                       --ncandidates;
+
+                                                       if ( META_BACK_ONERR_STOP( mi ) ) {
+                                                               savepriv = op->o_private;
+                                                               op->o_private = (void *)i;
+                                                               send_ldap_result( op, rs );
+                                                               op->o_private = savepriv;
+                                                               goto finish;
+                                                       }
+                                                       if ( META_BACK_ONERR_REPORT( mi ) ) {
+                                                               candidates[ i ].sr_err = rs->sr_err;
+                                                       }
+                                                       break;
 
-                               goto finish;
+                                               case META_SEARCH_BINDING:
+                                               case META_SEARCH_NEED_BIND:
+                                                       assert( 0 );
 
-                       } else if ( rc == LDAP_RES_SEARCH_ENTRY ) {
-                               if ( --op->ors_slimit == -1 ) {
-                                       ldap_msgfree( res );
-                                       res = NULL;
+                                               default:
+                                                       /* unrecoverable error */
+                                                       candidates[ i ].sr_msgid = META_MSGID_IGNORE;
+                                                       rc = rs->sr_err = LDAP_OTHER;
+                                                       goto finish;
+                                               }
+                                       }
 
-                                       rs->sr_err = LDAP_SIZELIMIT_EXCEEDED;
-                                       savepriv = op->o_private;
-                                       op->o_private = (void *)i;
-                                       send_ldap_result( op, rs );
-                                       op->o_private = savepriv;
-                                       goto finish;
+                                       if ( META_BACK_ONERR_STOP( mi ) ) {
+                                               savepriv = op->o_private;
+                                               op->o_private = (void *)i;
+                                               send_ldap_result( op, rs );
+                                               op->o_private = savepriv;
+                                               goto finish;
+                                       }
+                                       if ( META_BACK_ONERR_REPORT( mi ) ) {
+                                               candidates[ i ].sr_err = rs->sr_err;
+                                       }
                                }
 
-                               is_ok++;
+                               /*
+                                * When no candidates are left,
+                                * the outer cycle finishes
+                                */
+                               candidates[ i ].sr_msgid = META_MSGID_IGNORE;
+                               assert( ncandidates > 0 );
+                               --ncandidates;
+                               rs->sr_err = candidates[ i ].sr_err;
+                               continue;
 
-                               e = ldap_first_entry( msc->msc_ld, res );
-                               savepriv = op->o_private;
-                               op->o_private = (void *)i;
-                               meta_send_entry( op, rs, mc, i, e );
-                               op->o_private = savepriv;
+                       default:
+                               /* only touch when activity actually took place... */
+                               if ( mi->mi_idle_timeout != 0 && msc->msc_time < op->o_time ) {
+                                       msc->msc_time = op->o_time;
+                               }
+                               break;
+                       }
 
-                               ldap_msgfree( res );
-                               res = NULL;
+                       for ( msg = ldap_first_message( msc->msc_ld, res );
+                               msg != NULL;
+                               msg = ldap_next_message( msc->msc_ld, msg ) )
+                       {
+                               rc = ldap_msgtype( msg );
+                               if ( rc == LDAP_RES_SEARCH_ENTRY ) {
+                                       LDAPMessage     *e;
 
-                               gotit = 1;
+                                       if ( candidates[ i ].sr_type == REP_INTERMEDIATE ) {
+                                               /* don't retry any more... */
+                                               candidates[ i ].sr_type = REP_RESULT;
+                                       }
 
-#if 0
-                               /*
-                                * If scope is BASE, we need to jump out
-                                * as soon as one entry is found; if
-                                * the target pool is properly crafted,
-                                * this should correspond to the sole
-                                * entry that has the base DN
-                                */
-                               /* FIXME: this defeats the purpose of
-                                * doing a search with scope == base and
-                                * sizelimit = 1 to determine if a
-                                * candidate is actually unique */
-                               if ( op->ors_scope == LDAP_SCOPE_BASE
-                                               && rs->sr_nentries > 0 )
-                               {
-                                       doabandon = 1;
-                                       ncandidates = 0;
-                                       sres = LDAP_SUCCESS;
-                                       break;
-                               }
-#endif
+                                       is_ok++;
 
-                       } else if ( rc == LDAP_RES_SEARCH_REFERENCE ) {
-                               char            **references = NULL;
-                               int             cnt;
+                                       e = ldap_first_entry( msc->msc_ld, msg );
+                                       savepriv = op->o_private;
+                                       op->o_private = (void *)i;
+                                       rs->sr_err = meta_send_entry( op, rs, mc, i, e );
+
+                                       switch ( rs->sr_err ) {
+                                       case LDAP_SIZELIMIT_EXCEEDED:
+                                               savepriv = op->o_private;
+                                               op->o_private = (void *)i;
+                                               send_ldap_result( op, rs );
+                                               op->o_private = savepriv;
+                                               rs->sr_err = LDAP_SUCCESS;
+                                               ldap_msgfree( res );
+                                               res = NULL;
+                                               goto finish;
 
-                               is_ok++;
+                                       case LDAP_UNAVAILABLE:
+                                               rs->sr_err = LDAP_OTHER;
+                                               ldap_msgfree( res );
+                                               res = NULL;
+                                               goto finish;
+                                       }
+                                       op->o_private = savepriv;
 
-                               rc = ldap_parse_reference( msc->msc_ld, res,
-                                               &references, &rs->sr_ctrls, 1 );
-                               res = NULL;
+                                       /* don't wait any longer... */
+                                       gotit = 1;
+                                       save_tv.tv_sec = 0;
+                                       save_tv.tv_usec = 0;
 
-                               if ( rc != LDAP_SUCCESS ) {
-                                       continue;
-                               }
+                               } else if ( rc == LDAP_RES_SEARCH_REFERENCE ) {
+                                       char            **references = NULL;
+                                       int             cnt;
 
-                               if ( references == NULL ) {
-                                       continue;
-                               }
+                                       if ( candidates[ i ].sr_type == REP_INTERMEDIATE ) {
+                                               /* don't retry any more... */
+                                               candidates[ i ].sr_type = REP_RESULT;
+                                       }
+       
+                                       is_ok++;
+       
+                                       rc = ldap_parse_reference( msc->msc_ld, msg,
+                                                       &references, &rs->sr_ctrls, 0 );
+       
+                                       if ( rc != LDAP_SUCCESS ) {
+                                               continue;
+                                       }
+       
+                                       if ( references == NULL ) {
+                                               continue;
+                                       }
 
 #ifdef ENABLE_REWRITE
-                               dc.ctx = "referralDN";
+                                       dc.ctx = "referralDN";
 #else /* ! ENABLE_REWRITE */
-                               dc.tofrom = 0;
-                               dc.normalized = 0;
+                                       dc.tofrom = 0;
+                                       dc.normalized = 0;
 #endif /* ! ENABLE_REWRITE */
 
-                               /* FIXME: merge all and return at the end */
-
-                               for ( cnt = 0; references[ cnt ]; cnt++ )
-                                       ;
-
-                               rs->sr_ref = ch_calloc( sizeof( struct berval ), cnt + 1 );
-
-                               for ( cnt = 0; references[ cnt ]; cnt++ ) {
-                                       ber_str2bv( references[ cnt ], 0, 1, &rs->sr_ref[ cnt ] );
-                               }
-                               BER_BVZERO( &rs->sr_ref[ cnt ] );
-
-                               ( void )ldap_back_referral_result_rewrite( &dc, rs->sr_ref );
+                                       /* FIXME: merge all and return at the end */
+       
+                                       for ( cnt = 0; references[ cnt ]; cnt++ )
+                                               ;
+       
+                                       rs->sr_ref = ch_calloc( sizeof( struct berval ), cnt + 1 );
+       
+                                       for ( cnt = 0; references[ cnt ]; cnt++ ) {
+                                               ber_str2bv( references[ cnt ], 0, 1, &rs->sr_ref[ cnt ] );
+                                       }
+                                       BER_BVZERO( &rs->sr_ref[ cnt ] );
+       
+                                       ( void )ldap_back_referral_result_rewrite( &dc, rs->sr_ref );
 
-                               if ( rs->sr_ref != NULL && !BER_BVISNULL( &rs->sr_ref[ 0 ] ) ) {
-                                       /* ignore return value by now */
-                                       savepriv = op->o_private;
-                                       op->o_private = (void *)i;
-                                       ( void )send_search_reference( op, rs );
-                                       op->o_private = savepriv;
+                                       if ( rs->sr_ref != NULL && !BER_BVISNULL( &rs->sr_ref[ 0 ] ) ) {
+                                               /* ignore return value by now */
+                                               savepriv = op->o_private;
+                                               op->o_private = (void *)i;
+                                               ( void )send_search_reference( op, rs );
+                                               op->o_private = savepriv;
+       
+                                               ber_bvarray_free( rs->sr_ref );
+                                               rs->sr_ref = NULL;
+                                       }
 
-                                       ber_bvarray_free( rs->sr_ref );
-                                       rs->sr_ref = NULL;
-                               }
+                                       /* cleanup */
+                                       if ( references ) {
+                                               ber_memvfree( (void **)references );
+                                       }
 
-                               /* cleanup */
-                               if ( references ) {
-                                       ldap_value_free( references );
-                               }
+                                       if ( rs->sr_ctrls ) {
+                                               ldap_controls_free( rs->sr_ctrls );
+                                               rs->sr_ctrls = NULL;
+                                       }
 
-                               if ( rs->sr_ctrls ) {
-                                       ldap_controls_free( rs->sr_ctrls );
-                                       rs->sr_ctrls = NULL;
-                               }
+                               } else if ( rc == LDAP_RES_SEARCH_RESULT ) {
+                                       char            buf[ SLAP_TEXT_BUFLEN ];
+                                       char            **references = NULL;
 
-                       } else if ( rc == LDAP_RES_SEARCH_RESULT ) {
-                               char            buf[ SLAP_TEXT_BUFLEN ];
-                               char            **references = NULL;
-
-                               if ( ldap_parse_result( msc->msc_ld,
-                                                       res,
-                                                       &candidates[ i ].sr_err,
-                                                       (char **)&candidates[ i ].sr_matched,
-                                                       NULL /* (char **)&candidates[ i ].sr_text */ ,
-                                                       &references,
-                                                       &candidates[ i ].sr_ctrls, 1 ) != LDAP_SUCCESS )
-                               {
-                                       res = NULL;
-                                       ldap_get_option( msc->msc_ld,
+                                       if ( candidates[ i ].sr_type == REP_INTERMEDIATE ) {
+                                               /* don't retry any more... */
+                                               candidates[ i ].sr_type = REP_RESULT;
+                                       }
+       
+                                       /* NOTE: ignores response controls
+                                        * (and intermediate response controls
+                                        * as well, except for those with search
+                                        * references); this may not be correct,
+                                        * but if they're not ignored then
+                                        * back-meta would need to merge them
+                                        * consistently (think of pagedResults...)
+                                        */
+                                       /* FIXME: response controls? */
+                                       rs->sr_err = ldap_parse_result( msc->msc_ld,
+                                               msg,
+                                               &candidates[ i ].sr_err,
+                                               (char **)&candidates[ i ].sr_matched,
+                                               NULL /* (char **)&candidates[ i ].sr_text */ ,
+                                               &references,
+                                               NULL /* &candidates[ i ].sr_ctrls (unused) */ ,
+                                               0 );
+                                       if ( rs->sr_err != LDAP_SUCCESS ) {
+                                               ldap_get_option( msc->msc_ld,
                                                        LDAP_OPT_ERROR_NUMBER,
                                                        &rs->sr_err );
-                                       sres = slap_map_api2result( rs );
-                                       goto really_bad;
-                               }
-
-                               rs->sr_err = candidates[ i ].sr_err;
-                               sres = slap_map_api2result( rs );
-                               res = NULL;
-
-                               /* massage matchedDN if need be */
-                               if ( candidates[ i ].sr_matched != NULL ) {
-                                       if ( candidates[ i ].sr_matched[ 0 ] == '\0' ) {
-                                               ldap_memfree( (char *)candidates[ i ].sr_matched );
-                                               candidates[ i ].sr_matched = NULL;
+                                               sres = slap_map_api2result( rs );
+                                               candidates[ i ].sr_type = REP_RESULT;
+                                               ldap_msgfree( res );
+                                               res = NULL;
+                                               goto really_bad;
+                                       }
 
-                                       } else {
+                                       /* massage matchedDN if need be */
+                                       if ( candidates[ i ].sr_matched != NULL ) {
                                                struct berval   match, mmatch;
 
                                                ber_str2bv( candidates[ i ].sr_matched,
                                                        0, 0, &match );
+                                               candidates[ i ].sr_matched = NULL;
 
                                                dc.ctx = "matchedDN";
-                                               dc.target = &mi->mi_targets[ i ];
-
+                                               dc.target = mi->mi_targets[ i ];
                                                if ( !ldap_back_dn_massage( &dc, &match, &mmatch ) ) {
                                                        if ( mmatch.bv_val == match.bv_val ) {
-                                                               candidates[ i ].sr_matched = ch_strdup( mmatch.bv_val );
+                                                               candidates[ i ].sr_matched
+                                                                       = ch_strdup( mmatch.bv_val );
 
                                                        } else {
                                                                candidates[ i ].sr_matched = mmatch.bv_val;
@@ -554,129 +1192,268 @@ really_bad:;
                                                } 
                                                ldap_memfree( match.bv_val );
                                        }
-                               }
-
-                               /* just get rid of the error message, if any */
-                               if ( candidates[ i ].sr_text && candidates[ i ].sr_text[ 0 ] == '\0' )
-                               {
-                                       ldap_memfree( (char *)candidates[ i ].sr_text );
-                                       candidates[ i ].sr_text = NULL;
-                               }
-
-                               /* add references to array */
-                               if ( references ) {
-                                       BerVarray       sr_ref;
-                                       int             cnt;
-
-                                       for ( cnt = 0; references[ cnt ]; cnt++ )
-                                               ;
 
-                                       sr_ref = ch_calloc( sizeof( struct berval ), cnt + 1 );
+                                       /* add references to array */
+                                       if ( references ) {
+                                               BerVarray       sr_ref;
+                                               int             cnt;
+       
+                                               for ( cnt = 0; references[ cnt ]; cnt++ )
+                                                       ;
+       
+                                               sr_ref = ch_calloc( sizeof( struct berval ), cnt + 1 );
+       
+                                               for ( cnt = 0; references[ cnt ]; cnt++ ) {
+                                                       ber_str2bv( references[ cnt ], 0, 1, &sr_ref[ cnt ] );
+                                               }
+                                               BER_BVZERO( &sr_ref[ cnt ] );
+       
+                                               ( void )ldap_back_referral_result_rewrite( &dc, sr_ref );
+                                       
+                                               /* cleanup */
+                                               ber_memvfree( (void **)references );
+       
+                                               if ( rs->sr_v2ref == NULL ) {
+                                                       rs->sr_v2ref = sr_ref;
 
-                                       for ( cnt = 0; references[ cnt ]; cnt++ ) {
-                                               ber_str2bv( references[ cnt ], 0, 1, &sr_ref[ cnt ] );
+                                               } else {
+                                                       for ( cnt = 0; !BER_BVISNULL( &sr_ref[ cnt ] ); cnt++ ) {
+                                                               ber_bvarray_add( &rs->sr_v2ref, &sr_ref[ cnt ] );
+                                                       }
+                                                       ber_memfree( sr_ref );
+                                               }
                                        }
-                                       BER_BVZERO( &sr_ref[ cnt ] );
-
-                                       ( void )ldap_back_referral_result_rewrite( &dc, sr_ref );
-                               
-                                       /* cleanup */
-                                       ldap_value_free( references );
-
-                                       if ( rs->sr_v2ref == NULL ) {
-                                               rs->sr_v2ref = sr_ref;
-
-                                       } else {
-                                               for ( cnt = 0; !BER_BVISNULL( &sr_ref[ cnt ] ); cnt++ ) {
-                                                       ber_bvarray_add( &rs->sr_v2ref, &sr_ref[ cnt ] );
+       
+                                       rs->sr_err = candidates[ i ].sr_err;
+                                       sres = slap_map_api2result( rs );
+       
+                                       if ( LogTest( LDAP_DEBUG_TRACE | LDAP_DEBUG_ANY ) ) {
+                                               snprintf( buf, sizeof( buf ),
+                                                       "%s meta_back_search[%ld] "
+                                                       "match=\"%s\" err=%ld",
+                                                       op->o_log_prefix, i,
+                                                       candidates[ i ].sr_matched ? candidates[ i ].sr_matched : "",
+                                                       (long) candidates[ i ].sr_err );
+                                               if ( candidates[ i ].sr_err == LDAP_SUCCESS ) {
+                                                       Debug( LDAP_DEBUG_TRACE, "%s.\n", buf, 0, 0 );
+       
+                                               } else {
+                                                       Debug( LDAP_DEBUG_ANY, "%s (%s).\n",
+                                                               buf, ldap_err2string( candidates[ i ].sr_err ), 0 );
                                                }
-                                               ber_memfree( sr_ref );
                                        }
-                               }
-
-                               rs->sr_err = candidates[ i ].sr_err;
-                               sres = slap_map_api2result( rs );
-
-                               snprintf( buf, sizeof( buf ),
-                                       "%s meta_back_search[%d] "
-                                       "match=\"%s\" err=%d\n",
-                                       op->o_log_prefix, i,
-                                       candidates[ i ].sr_matched ? candidates[ i ].sr_matched : "",
-                                       candidates[ i ].sr_err );
-                               Debug( LDAP_DEBUG_ANY, "%s", buf, 0, 0 );
-
-                               switch ( sres ) {
-                               case LDAP_NO_SUCH_OBJECT:
-                                       /* is_ok is touched any time a valid
-                                        * (even intermediate) result is
-                                        * returned; as a consequence, if
-                                        * a candidate returns noSuchObject
-                                        * it is ignored and the candidate
-                                        * is simply demoted. */
-                                       if ( is_ok ) {
-                                               sres = LDAP_SUCCESS;
+       
+                                       switch ( sres ) {
+                                       case LDAP_NO_SUCH_OBJECT:
+                                               /* is_ok is touched any time a valid
+                                                * (even intermediate) result is
+                                                * returned; as a consequence, if
+                                                * a candidate returns noSuchObject
+                                                * it is ignored and the candidate
+                                                * is simply demoted. */
+                                               if ( is_ok ) {
+                                                       sres = LDAP_SUCCESS;
+                                               }
+                                               break;
+       
+                                       case LDAP_SUCCESS:
+                                       case LDAP_REFERRAL:
+                                               is_ok++;
+                                               break;
+       
+                                       case LDAP_SIZELIMIT_EXCEEDED:
+                                               /* if a target returned sizelimitExceeded
+                                                * and the entry count is equal to the
+                                                * proxy's limit, the target would have
+                                                * returned more, and the error must be
+                                                * propagated to the client; otherwise,
+                                                * the target enforced a limit lower
+                                                * than what requested by the proxy;
+                                                * ignore it */
+                                               if ( rs->sr_nentries == op->ors_slimit
+                                                       || META_BACK_ONERR_STOP( mi ) )
+                                               {
+                                                       savepriv = op->o_private;
+                                                       op->o_private = (void *)i;
+                                                       send_ldap_result( op, rs );
+                                                       op->o_private = savepriv;
+                                                       ldap_msgfree( res );
+                                                       res = NULL;
+                                                       goto finish;
+                                               }
+                                               if ( META_BACK_ONERR_REPORT( mi ) ) {
+                                                       candidates[ i ].sr_err = rs->sr_err;
+                                               }
+                                               break;
+       
+                                       default:
+                                               if ( META_BACK_ONERR_STOP( mi ) ) {
+                                                       savepriv = op->o_private;
+                                                       op->o_private = (void *)i;
+                                                       send_ldap_result( op, rs );
+                                                       op->o_private = savepriv;
+                                                       ldap_msgfree( res );
+                                                       res = NULL;
+                                                       goto finish;
+                                               }
+                                               if ( META_BACK_ONERR_REPORT( mi ) ) {
+                                                       candidates[ i ].sr_err = rs->sr_err;
+                                               }
+                                               break;
                                        }
-                                       break;
-
-                               case LDAP_SUCCESS:
-                               case LDAP_REFERRAL:
-                                       is_ok++;
-                                       break;
-
-                               default:
-                                       if ( META_BACK_ONERR_STOP( mi ) ) {
-                                               savepriv = op->o_private;
-                                               op->o_private = (void *)i;
-                                               send_ldap_result( op, rs );
-                                               op->o_private = savepriv;
-                                               goto finish;
+       
+                                       last = i;
+                                       rc = 0;
+       
+                                       /*
+                                        * When no candidates are left,
+                                        * the outer cycle finishes
+                                        */
+                                       candidates[ i ].sr_msgid = META_MSGID_IGNORE;
+                                       --ncandidates;
+       
+                               } else if ( rc == LDAP_RES_BIND ) {
+                                       meta_search_candidate_t retcode;
+       
+                                       retcode = meta_search_dobind_result( op, rs, &mc, i, candidates, msg );
+                                       if ( retcode == META_SEARCH_CANDIDATE ) {
+                                               candidates[ i ].sr_msgid = META_MSGID_IGNORE;
+                                               retcode = meta_back_search_start( op, rs, &dc, &mc, i, candidates );
                                        }
-                                       break;
+       
+                                       switch ( retcode ) {
+                                       case META_SEARCH_CANDIDATE:
+                                               break;
+       
+                                               /* means that failed but onerr == continue */
+                                       case META_SEARCH_NOT_CANDIDATE:
+                                       case META_SEARCH_ERR:
+                                               candidates[ i ].sr_msgid = META_MSGID_IGNORE;
+                                               --ncandidates;
+       
+                                               if ( META_BACK_ONERR_STOP( mi ) ) {
+                                                       savepriv = op->o_private;
+                                                       op->o_private = (void *)i;
+                                                       send_ldap_result( op, rs );
+                                                       op->o_private = savepriv;
+                                                       ldap_msgfree( res );
+                                                       res = NULL;
+                                                       goto finish;
+                                               }
+                                               if ( META_BACK_ONERR_REPORT( mi ) ) {
+                                                       candidates[ i ].sr_err = rs->sr_err;
+                                               }
+                                               break;
+       
+                                       default:
+                                               assert( 0 );
+                                               break;
+                                       }
+       
+                               } else {
+                                       assert( 0 );
+                                       ldap_msgfree( res );
+                                       res = NULL;
+                                       goto really_bad;
                                }
-
-                               last = i;
-                               rc = 0;
-
-                               /*
-                                * When no candidates are left,
-                                * the outer cycle finishes
-                                */
-                               candidates[ i ].sr_msgid = -1;
-                               --ncandidates;
-
-                       } else {
-                               assert( 0 );
-                               goto really_bad;
                        }
+
+                       ldap_msgfree( res );
+                       res = NULL;
                }
 
                /* check for abandon */
                if ( op->o_abandon || doabandon ) {
                        for ( i = 0; i < mi->mi_ntargets; i++ ) {
-                               metasingleconn_t        *msc = &mc->mc_conns[ i ];
+                               if ( candidates[ i ].sr_msgid >= 0 ) {
+                                       if ( META_IS_BINDING( &candidates[ i ] ) ) {
+                                               ldap_pvt_thread_mutex_lock( &mi->mi_conninfo.lai_mutex );
+                                               if ( LDAP_BACK_CONN_BINDING( &mc->mc_conns[ i ] ) ) {
+                                                       /* if still binding, destroy */
+
+#ifdef DEBUG_205
+                                                       char buf[ SLAP_TEXT_BUFLEN ];
+
+                                                       snprintf( buf, sizeof( buf), "%s meta_back_search(abandon) "
+                                                               "ldap_unbind_ext[%ld] mc=%p ld=%p",
+                                                               op->o_log_prefix, i, (void *)mc,
+                                                               (void *)mc->mc_conns[i].msc_ld );
+
+                                                       Debug( LDAP_DEBUG_ANY, "### %s\n", buf, 0, 0 );
+#endif /* DEBUG_205 */
 
-                               if ( candidates[ i ].sr_msgid != -1 ) {
-                                       ldap_abandon_ext( msc->msc_ld,
-                                               candidates[ i ].sr_msgid,
-                                               NULL, NULL );
-                                       candidates[ i ].sr_msgid = -1;
+                                                       meta_clear_one_candidate( op, mc, i );
+                                               }
+                                               ldap_pvt_thread_mutex_unlock( &mi->mi_conninfo.lai_mutex );
+                                               META_BINDING_CLEAR( &candidates[ i ] );
+                                               
+                                       } else {
+                                               (void)meta_back_cancel( mc, op, rs,
+                                                       candidates[ i ].sr_msgid, i,
+                                                       LDAP_BACK_DONTSEND );
+                                       }
+
+                                       candidates[ i ].sr_msgid = META_MSGID_IGNORE;
+                                       assert( ncandidates > 0 );
+                                       --ncandidates;
                                }
                        }
 
                        if ( op->o_abandon ) {
                                rc = SLAPD_ABANDON;
-                               goto finish;
+                               /* let send_ldap_result play cleanup handlers (ITS#4645) */
+                               break;
                        }
                }
 
-               if ( gotit == 0 ) {
-                       tv.tv_sec = 0;
-                        tv.tv_usec = 100000;   /* 0.1 s */
-                        ldap_pvt_thread_yield();
+               /* if no entry was found during this loop,
+                * set a minimal timeout */
+               if ( ncandidates > 0 && gotit == 0 ) {
+                       if ( save_tv.tv_sec == 0 && save_tv.tv_usec == 0 ) {
+                               save_tv.tv_usec = LDAP_BACK_RESULT_UTIMEOUT/initial_candidates;
 
-               } else {
-                       tv.tv_sec = 0;
-                       tv.tv_usec = 0;
+                               /* arbitrarily limit to something between 1 and 2 minutes */
+                       } else if ( ( stoptime == -1 && save_tv.tv_sec < 60 )
+                               || save_tv.tv_sec < ( stoptime - slap_get_time() ) / ( 2 * ncandidates ) )
+                       {
+                               /* double the timeout */
+                               lutil_timermul( &save_tv, 2, &save_tv );
+                       }
+
+#if 0
+                       if ( LogTest( LDAP_DEBUG_TRACE ) ) {
+                               char    buf[ SLAP_TEXT_BUFLEN ];
+
+                               snprintf( buf, sizeof( buf ), "%s %ld.%06ld %d/%d mc=%p",
+                                       op->o_log_prefix, save_tv.tv_sec, save_tv.tv_usec,
+                                       ncandidates, initial_candidates, mc );
+                               Debug( LDAP_DEBUG_TRACE, "### %s\n", buf, 0, 0 );
+                               for ( i = 0; i < mi->mi_ntargets; i++ ) {
+                                       if ( candidates[ i ].sr_msgid == META_MSGID_IGNORE ) {
+                                               continue;
+                                       }
+                       
+                                       snprintf( buf, sizeof( buf ), "[%ld] ld=%p%s%s\n",
+                                               i,
+                                               mc->mc_conns[ i ].msc_ld,
+                                               ( candidates[ i ].sr_msgid == META_MSGID_NEED_BIND ) ?  " needbind" : "",
+                                               META_IS_BINDING( &candidates[ i ] ) ? " binding" : "" );
+                                       Debug( LDAP_DEBUG_TRACE, "###    %s\n", buf, 0, 0 );
+                               }
+                       }
+#endif
+
+                       if ( alreadybound == 0 ) {
+#if 0
+                               Debug( LDAP_DEBUG_TRACE, "### %s select(%ld.%06ld)\n",
+                                       op->o_log_prefix, save_tv.tv_sec, save_tv.tv_usec );
+#endif
+                               tv = save_tv;
+                               (void)select( 0, NULL, NULL, NULL, &tv );
+
+                       } else {
+                               ldap_pvt_thread_yield();
+                       }
                }
        }
 
@@ -684,7 +1461,13 @@ really_bad:;
                /*
                 * FIXME: need a better strategy to handle errors
                 */
-               rc = meta_back_op_result( mc, op, rs, META_TARGET_NONE );
+               if ( mc ) {
+                       rc = meta_back_op_result( mc, op, rs, META_TARGET_NONE,
+                               -1, stoptime != -1 ? (stoptime - slap_get_time()) : 0,
+                               LDAP_BACK_SENDERR );
+               } else {
+                       rc = rs->sr_err;
+               }
                goto finish;
        }
 
@@ -693,27 +1476,75 @@ really_bad:;
         * 
         * FIXME: only the last one gets caught!
         */
-       if ( candidate_match > 0 && rs->sr_nentries > 0 ) {
+       savepriv = op->o_private;
+       op->o_private = (void *)(long)mi->mi_ntargets;
+       if ( candidate_match > 0 ) {
+               struct berval   pmatched = BER_BVNULL;
+
                /* we use the first one */
                for ( i = 0; i < mi->mi_ntargets; i++ ) {
-                       if ( candidates[ i ].sr_tag == META_CANDIDATE
-                                       && candidates[ i ].sr_matched )
+                       if ( META_IS_CANDIDATE( &candidates[ i ] )
+                                       && candidates[ i ].sr_matched != NULL )
                        {
-                               matched = (char *)candidates[ i ].sr_matched;
-                               candidates[ i ].sr_matched = NULL;
-                               break;
+                               struct berval   bv, pbv;
+                               int             rc;
+
+                               /* if we got success, and this target
+                                * returned noSuchObject, and its suffix
+                                * is a superior of the searchBase,
+                                * ignore the matchedDN */
+                               if ( sres == LDAP_SUCCESS
+                                       && candidates[ i ].sr_err == LDAP_NO_SUCH_OBJECT
+                                       && op->o_req_ndn.bv_len > mi->mi_targets[ i ]->mt_nsuffix.bv_len )
+                               {
+                                       free( (char *)candidates[ i ].sr_matched );
+                                       candidates[ i ].sr_matched = NULL;
+                                       continue;
+                               }
+
+                               ber_str2bv( candidates[ i ].sr_matched, 0, 0, &bv );
+                               rc = dnPretty( NULL, &bv, &pbv, op->o_tmpmemctx );
+
+                               if ( rc == LDAP_SUCCESS ) {
+
+                                       /* NOTE: if they all are superiors
+                                        * of the baseDN, the shorter is also 
+                                        * superior of the longer... */
+                                       if ( pbv.bv_len > pmatched.bv_len ) {
+                                               if ( !BER_BVISNULL( &pmatched ) ) {
+                                                       op->o_tmpfree( pmatched.bv_val, op->o_tmpmemctx );
+                                               }
+                                               pmatched = pbv;
+                                               op->o_private = (void *)i;
+
+                                       } else {
+                                               op->o_tmpfree( pbv.bv_val, op->o_tmpmemctx );
+                                       }
+                               }
+
+                               if ( candidates[ i ].sr_matched != NULL ) {
+                                       free( (char *)candidates[ i ].sr_matched );
+                                       candidates[ i ].sr_matched = NULL;
+                               }
                        }
                }
+
+               if ( !BER_BVISNULL( &pmatched ) ) {
+                       matched = pmatched.bv_val;
+               }
+
+       } else if ( sres == LDAP_NO_SUCH_OBJECT ) {
+               matched = op->o_bd->be_suffix[ 0 ].bv_val;
        }
 
 #if 0
        {
-               char    buf[BUFSIZ];
-               char    cnd[BUFSIZ];
+               char    buf[ SLAP_TEXT_BUFLEN ];
+               char    cnd[ SLAP_TEXT_BUFLEN ];
                int     i;
 
                for ( i = 0; i < mi->mi_ntargets; i++ ) {
-                       if ( candidates[ i ].sr_tag == META_CANDIDATE ) {
+                       if ( META_IS_CANDIDATE( &candidates[ i ] ) ) {
                                cnd[ i ] = '*';
                        } else {
                                cnd[ i ] = ' ';
@@ -731,27 +1562,46 @@ really_bad:;
        /*
         * In case we returned at least one entry, we return LDAP_SUCCESS
         * otherwise, the latter error code we got
-        *
-        * FIXME: we should handle error codes and return the more 
-        * important/reasonable
         */
 
-       if ( sres == LDAP_SUCCESS && rs->sr_v2ref ) {
-               sres = LDAP_REFERRAL;
+       if ( sres == LDAP_SUCCESS ) {
+               if ( rs->sr_v2ref ) {
+                       sres = LDAP_REFERRAL;
+               }
+
+               if ( META_BACK_ONERR_REPORT( mi ) ) {
+                       /*
+                        * Report errors, if any
+                        *
+                        * FIXME: we should handle error codes and return the more 
+                        * important/reasonable
+                        */
+                       for ( i = 0; i < mi->mi_ntargets; i++ ) {
+                               if ( !META_IS_CANDIDATE( &candidates[ i ] ) ) {
+                                       continue;
+                               }
+
+                               if ( candidates[ i ].sr_err != LDAP_SUCCESS
+                                       && candidates[ i ].sr_err != LDAP_NO_SUCH_OBJECT )
+                               {
+                                       sres = candidates[ i ].sr_err;
+                                       break;
+                               }
+                       }
+               }
        }
+
        rs->sr_err = sres;
        rs->sr_matched = matched;
        rs->sr_ref = ( sres == LDAP_REFERRAL ? rs->sr_v2ref : NULL );
-       savepriv = op->o_private;
-       op->o_private = (void *)mi->mi_ntargets;
        send_ldap_result( op, rs );
        op->o_private = savepriv;
        rs->sr_matched = NULL;
        rs->sr_ref = NULL;
 
 finish:;
-       if ( matched ) {
-               free( matched );
+       if ( matched && matched != op->o_bd->be_suffix[ 0 ].bv_val ) {
+               op->o_tmpfree( matched, op->o_tmpmemctx );
        }
 
        if ( rs->sr_v2ref ) {
@@ -759,10 +1609,29 @@ finish:;
        }
 
        for ( i = 0; i < mi->mi_ntargets; i++ ) {
-               if ( candidates[ i ].sr_tag != META_CANDIDATE ) {
+               if ( !META_IS_CANDIDATE( &candidates[ i ] ) ) {
                        continue;
                }
 
+               if ( mc && META_IS_BINDING( &candidates[ i ] ) ) {
+                       ldap_pvt_thread_mutex_lock( &mi->mi_conninfo.lai_mutex );
+                       if ( LDAP_BACK_CONN_BINDING( &mc->mc_conns[ i ] ) ) {
+                               assert( candidates[ i ].sr_msgid >= 0 );
+                               assert( mc->mc_conns[ i ].msc_ld != NULL );
+
+#ifdef DEBUG_205
+                               Debug( LDAP_DEBUG_ANY, "### %s meta_back_search(cleanup) "
+                                       "ldap_unbind_ext[%ld] ld=%p\n",
+                                       op->o_log_prefix, i, (void *)mc->mc_conns[i].msc_ld );
+#endif /* DEBUG_205 */
+
+                               /* if still binding, destroy */
+                               meta_clear_one_candidate( op, mc, i );
+                       }
+                       ldap_pvt_thread_mutex_unlock( &mi->mi_conninfo.lai_mutex );
+                       META_BINDING_CLEAR( &candidates[ i ] );
+               }
+
                if ( candidates[ i ].sr_matched ) {
                        free( (char *)candidates[ i ].sr_matched );
                        candidates[ i ].sr_matched = NULL;
@@ -782,11 +1651,30 @@ finish:;
                        ldap_controls_free( candidates[ i ].sr_ctrls );
                        candidates[ i ].sr_ctrls = NULL;
                }
+
+               if ( META_BACK_TGT_QUARANTINE( mi->mi_targets[ i ] ) ) {
+                       meta_back_quarantine( op, &candidates[ i ], i );
+               }
+
+               /* only in case of timelimit exceeded, if the timelimit exceeded because
+                * one contacted target never responded, invalidate the connection
+                * NOTE: should we quarantine the target as well?  right now, the connection
+                * is invalidated; the next time it will be recreated and the target
+                * will be quarantined if it cannot be contacted */
+               if ( mi->mi_idle_timeout != 0
+                       && rs->sr_err == LDAP_TIMELIMIT_EXCEEDED
+                       && op->o_time > mc->mc_conns[ i ].msc_time )
+               {
+                       /* don't let anyone else use this expired connection */
+                       LDAP_BACK_CONN_TAINTED_SET( mc );
+               }
        }
 
-       meta_back_release_conn( op, mc );
+       if ( mc ) {
+               meta_back_release_conn( op, mc );
+       }
 
-       return rc;
+       return rs->sr_err;
 }
 
 static int
@@ -802,9 +1690,11 @@ meta_send_entry(
        Entry                   ent = { 0 };
        BerElement              ber = *e->lm_ber;
        Attribute               *attr, **attrp;
-       struct berval           *bv, bdn;
+       struct berval           bdn,
+                               dn = BER_BVNULL;
        const char              *text;
        dncookie                dc;
+       int                     rc;
 
        if ( ber_scanf( &ber, "{m{", &bdn ) == LBER_ERROR ) {
                return LDAP_DECODING_ERROR;
@@ -813,12 +1703,12 @@ meta_send_entry(
        /*
         * Rewrite the dn of the result, if needed
         */
-       dc.target = &mi->mi_targets[ target ];
+       dc.target = mi->mi_targets[ target ];
        dc.conn = op->o_conn;
        dc.rs = rs;
        dc.ctx = "searchResult";
 
-       rs->sr_err = ldap_back_dn_massage( &dc, &bdn, &ent.e_name );
+       rs->sr_err = ldap_back_dn_massage( &dc, &bdn, &dn );
        if ( rs->sr_err != LDAP_SUCCESS) {
                return rs->sr_err;
        }
@@ -830,9 +1720,14 @@ meta_send_entry(
         * 
         * FIXME: should we log anything, or delegate to dnNormalize?
         */
-       if ( dnNormalize( 0, NULL, NULL, &ent.e_name, &ent.e_nname,
-               op->o_tmpmemctx ) != LDAP_SUCCESS )
-       {
+       rc = dnPrettyNormal( NULL, &dn, &ent.e_name, &ent.e_nname,
+               op->o_tmpmemctx );
+       if ( dn.bv_val != bdn.bv_val ) {
+               free( dn.bv_val );
+       }
+       BER_BVZERO( &dn );
+
+       if ( rc != LDAP_SUCCESS ) {
                return LDAP_INVALID_DN_SYNTAX;
        }
 
@@ -848,24 +1743,24 @@ meta_send_entry(
 
        dc.ctx = "searchAttrDN";
        while ( ber_scanf( &ber, "{m", &a ) != LBER_ERROR ) {
-               int             last = 0;
+               int                             last = 0;
+               slap_syntax_validate_func       *validate;
+               slap_syntax_transform_func      *pretty;
 
-               ldap_back_map( &mi->mi_targets[ target ].mt_rwmap.rwm_at, 
+               ldap_back_map( &mi->mi_targets[ target ]->mt_rwmap.rwm_at, 
                                &a, &mapped, BACKLDAP_REMAP );
                if ( BER_BVISNULL( &mapped ) || mapped.bv_val[0] == '\0' ) {
+                       ( void )ber_scanf( &ber, "x" /* [W] */ );
                        continue;
                }
-               attr = ( Attribute * )ch_malloc( sizeof( Attribute ) );
+               attr = attr_alloc( NULL );
                if ( attr == NULL ) {
                        continue;
                }
-               attr->a_flags = 0;
-               attr->a_next = 0;
-               attr->a_desc = NULL;
                if ( slap_bv2ad( &mapped, &attr->a_desc, &text )
                                != LDAP_SUCCESS) {
-                       if ( slap_bv2undef_ad( &mapped, &attr->a_desc, &text ) 
-                                       != LDAP_SUCCESS )
+                       if ( slap_bv2undef_ad( &mapped, &attr->a_desc, &text,
+                               SLAP_AD_PROXIED ) != LDAP_SUCCESS )
                        {
                                char    buf[ SLAP_TEXT_BUFLEN ];
 
@@ -876,13 +1771,15 @@ meta_send_entry(
                                        mapped.bv_val, text );
 
                                Debug( LDAP_DEBUG_ANY, "%s", buf, 0, 0 );
-                               ch_free( attr );
+                               attr_free( attr );
                                continue;
                        }
                }
 
                /* no subschemaSubentry */
-               if ( attr->a_desc == slap_schema.si_ad_subschemaSubentry ) {
+               if ( attr->a_desc == slap_schema.si_ad_subschemaSubentry
+                       || attr->a_desc == slap_schema.si_ad_entryDN )
+               {
 
                        /* 
                         * We eat target's subschemaSubentry because
@@ -890,10 +1787,14 @@ meta_send_entry(
                         * to resolve to the appropriate backend;
                         * later, the local subschemaSubentry is
                         * added.
+                        *
+                        * We also eat entryDN because the frontend
+                        * will reattach it without checking if already
+                        * present...
                         */
                        ( void )ber_scanf( &ber, "x" /* [W] */ );
 
-                       ch_free(attr);
+                       attr_free(attr);
                        continue;
                }
 
@@ -902,13 +1803,26 @@ meta_send_entry(
                {
                        attr->a_vals = (struct berval *)&slap_dummy_bv;
 
-               } else if ( attr->a_desc == slap_schema.si_ad_objectClass
+               } else {
+                       for ( last = 0; !BER_BVISNULL( &attr->a_vals[ last ] ); ++last )
+                               ;
+               }
+
+               validate = attr->a_desc->ad_type->sat_syntax->ssyn_validate;
+               pretty = attr->a_desc->ad_type->sat_syntax->ssyn_pretty;
+
+               if ( !validate && !pretty ) {
+                       attr_free( attr );
+                       goto next_attr;
+               }
+
+               if ( attr->a_desc == slap_schema.si_ad_objectClass
                                || attr->a_desc == slap_schema.si_ad_structuralObjectClass )
                {
-                       for ( last = 0; !BER_BVISNULL( &attr->a_vals[ last ] ); ++last );
+                       struct berval   *bv;
 
                        for ( bv = attr->a_vals; !BER_BVISNULL( bv ); bv++ ) {
-                               ldap_back_map( &mi->mi_targets[ target ].mt_rwmap.rwm_oc,
+                               ldap_back_map( &mi->mi_targets[ target ]->mt_rwmap.rwm_oc,
                                                bv, &mapped, BACKLDAP_REMAP );
                                if ( BER_BVISNULL( &mapped ) || mapped.bv_val[0] == '\0') {
                                        free( bv->bv_val );
@@ -936,17 +1850,59 @@ meta_send_entry(
                 * ACLs to the target directory server, and letting
                 * everything pass thru the ldap backend.
                 */
-               } else if ( attr->a_desc->ad_type->sat_syntax ==
+               } else {
+                       int     i;
+
+                       if ( attr->a_desc->ad_type->sat_syntax ==
                                slap_schema.si_syn_distinguishedName )
-               {
-                       ldap_dnattr_result_rewrite( &dc, attr->a_vals );
+                       {
+                               ldap_dnattr_result_rewrite( &dc, attr->a_vals );
+
+                       } else if ( attr->a_desc == slap_schema.si_ad_ref ) {
+                               ldap_back_referral_result_rewrite( &dc, attr->a_vals );
+
+                       }
+
+                       for ( i = 0; i < last; i++ ) {
+                               struct berval   pval;
+                               int             rc;
+
+                               if ( pretty ) {
+                                       rc = pretty( attr->a_desc->ad_type->sat_syntax,
+                                               &attr->a_vals[i], &pval, NULL );
+
+                               } else {
+                                       rc = validate( attr->a_desc->ad_type->sat_syntax,
+                                               &attr->a_vals[i] );
+                               }
+
+                               if ( rc ) {
+                                       LBER_FREE( attr->a_vals[i].bv_val );
+                                       if ( --last == i ) {
+                                               BER_BVZERO( &attr->a_vals[ i ] );
+                                               break;
+                                       }
+                                       attr->a_vals[i] = attr->a_vals[last];
+                                       BER_BVZERO( &attr->a_vals[last] );
+                                       i--;
+                                       continue;
+                               }
+
+                               if ( pretty ) {
+                                       LBER_FREE( attr->a_vals[i].bv_val );
+                                       attr->a_vals[i] = pval;
+                               }
+                       }
 
-               } else if ( attr->a_desc == slap_schema.si_ad_ref ) {
-                       ldap_back_referral_result_rewrite( &dc, attr->a_vals );
+                       if ( last == 0 && attr->a_vals != &slap_dummy_bv ) {
+                               attr_free( attr );
+                               goto next_attr;
+                       }
                }
 
                if ( last && attr->a_desc->ad_type->sat_equality &&
-                       attr->a_desc->ad_type->sat_equality->smr_normalize ) {
+                       attr->a_desc->ad_type->sat_equality->smr_normalize )
+               {
                        int i;
 
                        attr->a_nvals = ch_malloc( ( last + 1 ) * sizeof( struct berval ) );
@@ -966,18 +1922,23 @@ meta_send_entry(
 
                *attrp = attr;
                attrp = &attr->a_next;
+next_attr:;
        }
        rs->sr_entry = &ent;
        rs->sr_attrs = op->ors_attrs;
        rs->sr_flags = 0;
-       send_search_entry( op, rs );
+       rs->sr_err = LDAP_SUCCESS;
+       rc = send_search_entry( op, rs );
+       switch ( rc ) {
+       case LDAP_UNAVAILABLE:
+               rc = LDAP_OTHER;
+               break;
+       }
        rs->sr_entry = NULL;
        rs->sr_attrs = NULL;
        
        if ( !BER_BVISNULL( &ent.e_name ) ) {
-               if ( ent.e_name.bv_val != bdn.bv_val ) {
-                       free( ent.e_name.bv_val );
-               }
+               free( ent.e_name.bv_val );
                BER_BVZERO( &ent.e_name );
        }
        if ( !BER_BVISNULL( &ent.e_nname ) ) {
@@ -986,6 +1947,6 @@ meta_send_entry(
        }
        entry_clean( &ent );
 
-       return LDAP_SUCCESS;
+       return rc;
 }