]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/back-ldap/bind.c
Remove unused label "error_return"
[openldap] / servers / slapd / back-ldap / bind.c
index 3d8092d7842c06ebdaba328986e266d970878ec5..09b23924d5f11d5313ac508e30c8350be7804559 100644 (file)
@@ -2,7 +2,7 @@
 /* $OpenLDAP$ */
 /* This work is part of OpenLDAP Software <http://www.openldap.org/>.
  *
- * Copyright 1999-2004 The OpenLDAP Foundation.
+ * Copyright 1999-2005 The OpenLDAP Foundation.
  * Portions Copyright 2000-2003 Pierangelo Masarati.
  * Portions Copyright 1999-2003 Howard Chu.
  * All rights reserved.
@@ -16,7 +16,7 @@
  * <http://www.OpenLDAP.org/license.html>.
  */
 /* ACKNOWLEDGEMENTS:
- * This work was initially developed by the Howard Chu for inclusion
+ * This work was initially developed by Howard Chu for inclusion
  * in OpenLDAP Software and subsequently enhanced by Pierangelo
  * Masarati.
  */
@@ -25,6 +25,7 @@
 
 #include <stdio.h>
 
+#include <ac/errno.h>
 #include <ac/socket.h>
 #include <ac/string.h>
 
 
 static LDAP_REBIND_PROC        ldap_back_rebind;
 
+static int
+ldap_back_proxy_authz_bind( struct ldapconn *lc, Operation *op, SlapReply *rs );
+
+static int
+ldap_back_prepare_conn( struct ldapconn **lcp, Operation *op, SlapReply *rs, ldap_back_send_t sendok );
+
 int
-ldap_back_bind(
-    Operation          *op,
-    SlapReply          *rs )
+ldap_back_bind( Operation *op, SlapReply *rs )
 {
        struct ldapinfo *li = (struct ldapinfo *) op->o_bd->be_private;
        struct ldapconn *lc;
 
-       struct berval mdn = BER_BVNULL;
        int rc = 0;
        ber_int_t msgid;
-       dncookie dc;
 
-       lc = ldap_back_getconn(op, rs);
+       lc = ldap_back_getconn( op, rs, LDAP_BACK_SENDERR );
        if ( !lc ) {
-               return( -1 );
+               return rs->sr_err;
        }
 
-       /*
-        * Rewrite the bind dn if needed
-        */
-       dc.rwmap = &li->rwmap;
-#ifdef ENABLE_REWRITE
-       dc.conn = op->o_conn;
-       dc.rs = rs;
-       dc.ctx = "bindDN";
-#else
-       dc.tofrom = 1;
-       dc.normalized = 0;
-#endif
-       if ( ldap_back_dn_massage( &dc, &op->o_req_dn, &mdn ) ) {
-               send_ldap_result( op, rs );
-               return -1;
+       if ( !BER_BVISNULL( &lc->lc_bound_ndn ) ) {
+               ch_free( lc->lc_bound_ndn.bv_val );
+               BER_BVZERO( &lc->lc_bound_ndn );
        }
+       lc->lc_bound = 0;
 
-       if ( !BER_BVISNULL( &lc->bound_dn ) ) {
-               ch_free( lc->bound_dn.bv_val );
-               BER_BVZERO( &lc->bound_dn );
-       }
-       lc->bound = 0;
        /* method is always LDAP_AUTH_SIMPLE if we got here */
-       rs->sr_err = ldap_sasl_bind(lc->ld, mdn.bv_val, LDAP_SASL_SIMPLE,
-               &op->oq_bind.rb_cred, op->o_ctrls, NULL, &msgid);
-       rc = ldap_back_op_result( lc, op, rs, msgid, 1 );
-       if (rc == LDAP_SUCCESS) {
-               lc->bound = 1;
-               if ( mdn.bv_val != op->o_req_dn.bv_val ) {
-                       lc->bound_dn = mdn;
-               } else {
-                       ber_dupbv( &lc->bound_dn, &op->o_req_dn );
+       rs->sr_err = ldap_sasl_bind( lc->lc_ld, op->o_req_dn.bv_val,
+                       LDAP_SASL_SIMPLE,
+                       &op->orb_cred, op->o_ctrls, NULL, &msgid );
+       rc = ldap_back_op_result( lc, op, rs, msgid, LDAP_BACK_SENDERR );
+
+       if ( rc == LDAP_SUCCESS ) {
+               /* If defined, proxyAuthz will be used also when
+                * back-ldap is the authorizing backend; for this
+                * purpose, a successful bind is followed by a
+                * bind with the configured identity assertion */
+               /* NOTE: use with care */
+               if ( li->idassert_flags & LDAP_BACK_AUTH_OVERRIDE ) {
+                       ldap_back_proxy_authz_bind( lc, op, rs );
+                       if ( lc->lc_bound == 0 ) {
+                               rc = 1;
+                               goto done;
+                       }
                }
-               BER_BVZERO( &mdn );
 
-               if ( li->savecred ) {
-                       if ( !BER_BVISNULL( &lc->cred ) ) {
-                               memset( lc->cred.bv_val, 0, lc->cred.bv_len );
-                               ch_free( lc->cred.bv_val );
+               lc->lc_bound = 1;
+               ber_dupbv( &lc->lc_bound_ndn, &op->o_req_ndn );
+
+               if ( LDAP_BACK_SAVECRED( li ) ) {
+                       if ( !BER_BVISNULL( &lc->lc_cred ) ) {
+                               memset( lc->lc_cred.bv_val, 0,
+                                               lc->lc_cred.bv_len );
                        }
-                       ber_dupbv( &lc->cred, &op->oq_bind.rb_cred );
-                       ldap_set_rebind_proc( lc->ld, ldap_back_rebind, lc );
+                       ber_bvreplace( &lc->lc_cred, &op->orb_cred );
+                       ldap_set_rebind_proc( lc->lc_ld, ldap_back_rebind, lc );
                }
        }
+done:;
 
        /* must re-insert if local DN changed as result of bind */
-       if ( lc->bound && !bvmatch(&op->o_req_ndn, &lc->local_dn ) ) {
-               int lerr;
+       if ( lc->lc_bound && !dn_match( &op->o_req_ndn, &lc->lc_local_ndn ) ) {
+               int             lerr;
+
+               /* wait for all other ops to release the connection */
+retry_lock:;
+               switch ( ldap_pvt_thread_mutex_trylock( &li->conn_mutex ) ) {
+               case LDAP_PVT_THREAD_EBUSY:
+               default:
+                       ldap_pvt_thread_yield();
+                       goto retry_lock;
+
+               case 0:
+                       if ( lc->lc_refcnt > 1 ) {
+                               ldap_pvt_thread_mutex_unlock( &li->conn_mutex );
+                               ldap_pvt_thread_yield();
+                               goto retry_lock;
+                       }
+                       break;
+               }
 
-               ldap_pvt_thread_mutex_lock( &li->conn_mutex );
+               assert( lc->lc_refcnt == 1 );
                lc = avl_delete( &li->conntree, (caddr_t)lc,
                                ldap_back_conn_cmp );
-               if ( !BER_BVISNULL( &lc->local_dn ) )
-                       ch_free( lc->local_dn.bv_val );
-               ber_dupbv( &lc->local_dn, &op->o_req_ndn );
+               assert( lc != NULL );
+
+               ber_bvreplace( &lc->lc_local_ndn, &op->o_req_ndn );
                lerr = avl_insert( &li->conntree, (caddr_t)lc,
                        ldap_back_conn_cmp, ldap_back_conn_dup );
                ldap_pvt_thread_mutex_unlock( &li->conn_mutex );
                if ( lerr == -1 ) {
+                       /* we can do this because lc_refcnt == 1 */
                        ldap_back_conn_free( lc );
+                       lc = NULL;
                }
        }
 
-       if ( !BER_BVISNULL( &mdn ) && mdn.bv_val != op->o_req_dn.bv_val ) {
-               free( mdn.bv_val );
+       if ( lc != NULL ) {
+               ldap_back_release_conn( op, rs, lc );
        }
 
        return( rc );
@@ -133,23 +150,22 @@ ldap_back_bind(
  * used by avl stuff
  */
 int
-ldap_back_conn_cmp(
-       const void *c1,
-       const void *c2
-       )
+ldap_back_conn_cmp( const void *c1, const void *c2 )
 {
        const struct ldapconn *lc1 = (const struct ldapconn *)c1;
        const struct ldapconn *lc2 = (const struct ldapconn *)c2;
        int rc;
-       
+
        /* If local DNs don't match, it is definitely not a match */
-       if ( ( rc = ber_bvcmp( &lc1->local_dn, &lc2->local_dn ) ) )
+       rc = ber_bvcmp( &lc1->lc_local_ndn, &lc2->lc_local_ndn );
+       if ( rc ) {
                return rc;
+       }
 
        /* For shared sessions, conn is NULL. Only explicitly
         * bound sessions will have non-NULL conn.
         */
-       return SLAP_PTRCMP(lc1->conn, lc2->conn);
+       return SLAP_PTRCMP( lc1->lc_conn, lc2->lc_conn );
 }
 
 /*
@@ -159,51 +175,56 @@ ldap_back_conn_cmp(
  * used by avl stuff
  */
 int
-ldap_back_conn_dup(
-       void *c1,
-       void *c2
-       )
+ldap_back_conn_dup( void *c1, void *c2 )
 {
        struct ldapconn *lc1 = (struct ldapconn *)c1;
        struct ldapconn *lc2 = (struct ldapconn *)c2;
 
        /* Cannot have more than one shared session with same DN */
-       if ( dn_match( &lc1->local_dn, &lc2->local_dn ) &&
-                lc1->conn == lc2->conn ) return -1;
+       if ( dn_match( &lc1->lc_local_ndn, &lc2->lc_local_ndn ) &&
+                               lc1->lc_conn == lc2->lc_conn )
+       {
+               return -1;
+       }
                
        return 0;
 }
 
 #if PRINT_CONNTREE > 0
-static void ravl_print( Avlnode *root, int depth )
+static void
+ravl_print( Avlnode *root, int depth )
 {
        int     i;
        struct ldapconn *lc;
        
-       if ( root == 0 )
+       if ( root == 0 ) {
                return;
+       }
        
        ravl_print( root->avl_right, depth+1 );
        
-       for ( i = 0; i < depth; i++ )
+       for ( i = 0; i < depth; i++ ) {
                printf( "   " );
+       }
 
        lc = root->avl_data;
        printf( "lc(%lx) local(%s) conn(%lx) %d\n",
-                       lc, lc->local_dn.bv_val, lc->conn, root->avl_bf );
+                       lc, lc->lc_local_ndn.bv_val, lc->lc_conn, root->avl_bf );
        
        ravl_print( root->avl_left, depth+1 );
 }
 
-static void myprint( Avlnode *root )
+static void
+myprint( Avlnode *root )
 {
        printf( "********\n" );
        
-       if ( root == 0 )
+       if ( root == 0 ) {
                printf( "\tNULL\n" );
 
-       else
+       } else {
                ravl_print( root, 0 );
+       }
        
        printf( "********\n" );
 }
@@ -215,131 +236,331 @@ ldap_back_freeconn( Operation *op, struct ldapconn *lc )
        struct ldapinfo *li = (struct ldapinfo *) op->o_bd->be_private;
 
        ldap_pvt_thread_mutex_lock( &li->conn_mutex );
-       lc = avl_delete( &li->conntree, (caddr_t)lc,
-                       ldap_back_conn_cmp );
-       ldap_back_conn_free( (void *)lc );
+
+       assert( lc->lc_refcnt > 0 );
+       if ( --lc->lc_refcnt == 0 ) {
+               lc = avl_delete( &li->conntree, (caddr_t)lc,
+                               ldap_back_conn_cmp );
+               assert( lc != NULL );
+
+               ldap_back_conn_free( (void *)lc );
+       }
+
        ldap_pvt_thread_mutex_unlock( &li->conn_mutex );
 
        return 0;
 }
 
+#ifdef HAVE_TLS
+static int
+ldap_back_start_tls(
+       LDAP            *ld,
+       int             protocol,
+       int             *is_tls,
+       const char      *url,
+       unsigned        flags,
+       const char      **text )
+{
+       int             rc = LDAP_SUCCESS;
+       struct ldapinfo li;
+
+       /* this is ridicolous... */
+       li.flags = flags;
+
+       /* start TLS ("tls-[try-]{start,propagate}" statements) */
+       if ( ( LDAP_BACK_USE_TLS( &li ) || ( *is_tls && LDAP_BACK_PROPAGATE_TLS( &li ) ) )
+                               && !ldap_is_ldaps_url( url ) )
+       {
+#ifdef SLAP_STARTTLS_ASYNCHRONOUS
+               /*
+                * use asynchronous StartTLS
+                * in case, chase referral (not implemented yet)
+                */
+               int             msgid;
+
+               if ( protocol == 0 ) {
+                       ldap_get_option( ld, LDAP_OPT_PROTOCOL_VERSION,
+                                       (void *)&protocol );
+               }
+
+               if ( protocol < LDAP_VERSION3 ) {
+                       protocol = LDAP_VERSION3;
+                       /* Set LDAP version */
+                       ldap_set_option( ld, LDAP_OPT_PROTOCOL_VERSION,
+                                       (const void *)&protocol );
+               }
+
+               rc = ldap_start_tls( ld, NULL, NULL, &msgid );
+               if ( rc == LDAP_SUCCESS ) {
+                       LDAPMessage     *res = NULL;
+                       int             retries = 1;
+                       struct timeval  tv = { 0, 0 };
+
+retry:;
+                       rc = ldap_result( ld, msgid, LDAP_MSG_ALL, &tv, &res );
+                       if ( rc < 0 ) {
+                               rc = LDAP_OTHER;
+
+                       } else if ( rc == 0 ) {
+                               if ( retries ) {
+                                       retries--;
+                                       tv.tv_sec = 0;
+                                       tv.tv_usec = 100000;
+                                       goto retry;
+                               }
+                               rc = LDAP_OTHER;
+
+                       } else if ( rc == LDAP_RES_EXTENDED ) {
+                               struct berval   *data = NULL;
+
+                               rc = ldap_parse_extended_result( ld, res,
+                                               NULL, &data, 0 );
+                               if ( rc == LDAP_SUCCESS ) {
+                                       rc = ldap_result2error( ld, res, 1 );
+                                       res = NULL;
+                                       
+                                       /* FIXME: in case a referral 
+                                        * is returned, should we try
+                                        * using it instead of the 
+                                        * configured URI? */
+                                       if ( rc == LDAP_SUCCESS ) {
+                                               rc = ldap_install_tls( ld );
+
+                                       } else if ( rc == LDAP_REFERRAL ) {
+                                               rc = LDAP_OTHER;
+                                               *text = "unwilling to chase referral returned by Start TLS exop";
+                                       }
+
+                                       if ( data ) {
+                                               if ( data->bv_val ) {
+                                                       ber_memfree( data->bv_val );
+                                               }
+                                               ber_memfree( data );
+                                       }
+                               }
+
+                       } else {
+                               rc = LDAP_OTHER;
+                       }
+
+                       if ( res != NULL ) {
+                               ldap_msgfree( res );
+                       }
+               }
+#else /* ! SLAP_STARTTLS_ASYNCHRONOUS */
+               /*
+                * use synchronous StartTLS
+                */
+               rc = ldap_start_tls_s( ld, NULL, NULL );
+#endif /* ! SLAP_STARTTLS_ASYNCHRONOUS */
+
+               /* if StartTLS is requested, only attempt it if the URL
+                * is not "ldaps://"; this may occur not only in case
+                * of misconfiguration, but also when used in the chain 
+                * overlay, where the "uri" can be parsed out of a referral */
+               switch ( rc ) {
+               case LDAP_SUCCESS:
+                       *is_tls = 1;
+                       break;
+
+               case LDAP_SERVER_DOWN:
+                       break;
+
+               default:
+                       if ( LDAP_BACK_TLS_CRITICAL( &li ) ) {
+                               *text = "could not start TLS";
+                               break;
+                       }
+
+                       /* in case Start TLS is not critical */
+                       *is_tls = 0;
+                       rc = LDAP_SUCCESS;
+                       break;
+               }
+
+       } else {
+               *is_tls = 0;
+       }
+
+       return rc;
+}
+#endif /* HAVE_TLS */
+
+static int
+ldap_back_prepare_conn( struct ldapconn **lcp, Operation *op, SlapReply *rs, ldap_back_send_t sendok )
+{
+       struct ldapinfo *li = (struct ldapinfo *)op->o_bd->be_private;
+       int             vers = op->o_protocol;
+       LDAP            *ld = NULL;
+#ifdef HAVE_TLS
+       int             is_tls = op->o_conn->c_is_tls;
+#endif /* HAVE_TLS */
+
+       assert( lcp != NULL );
+
+       rs->sr_err = ldap_initialize( &ld, li->url );
+       if ( rs->sr_err != LDAP_SUCCESS ) {
+               goto error_return;
+       }
+
+       /* Set LDAP version. This will always succeed: If the client
+        * bound with a particular version, then so can we.
+        */
+       ldap_set_option( ld, LDAP_OPT_PROTOCOL_VERSION, (const void *)&vers );
+
+       /* automatically chase referrals ("[dont-]chase-referrals" statement) */
+       if ( LDAP_BACK_CHASE_REFERRALS( li ) ) {
+               ldap_set_option( ld, LDAP_OPT_REFERRALS, LDAP_OPT_ON );
+       }
+
+#ifdef HAVE_TLS
+       rs->sr_err = ldap_back_start_tls( ld,
+                       op->o_protocol, &is_tls,
+                       li->url, li->flags, &rs->sr_text );
+       if ( rs->sr_err != LDAP_SUCCESS ) {
+               ldap_unbind_ext( ld, NULL, NULL );
+               goto error_return;
+       }
+#endif /* HAVE_TLS */
+
+       if ( *lcp == NULL ) {
+               *lcp = (struct ldapconn *)ch_calloc( 1, sizeof( struct ldapconn ) );
+               (*lcp)->lc_flags= li->flags;
+       }
+       (*lcp)->lc_ld = ld;
+       (*lcp)->lc_refcnt = 1;
+#ifdef HAVE_TLS
+       (*lcp)->lc_is_tls = is_tls;
+#endif /* HAVE_TLS */
+
+error_return:;
+       if ( rs->sr_err != LDAP_SUCCESS ) {
+               rs->sr_err = slap_map_api2result( rs );
+               if ( sendok & LDAP_BACK_SENDERR ) {
+                       if ( rs->sr_text == NULL ) {
+                               rs->sr_text = "ldap_initialize() failed";
+                       }
+                       send_ldap_result( op, rs );
+                       rs->sr_text = NULL;
+               }
+       }
+
+       return rs->sr_err;
+}
+
 struct ldapconn *
-ldap_back_getconn(Operation *op, SlapReply *rs)
+ldap_back_getconn( Operation *op, SlapReply *rs, ldap_back_send_t sendok )
 {
        struct ldapinfo *li = (struct ldapinfo *)op->o_bd->be_private;
-       struct ldapconn *lc, lc_curr;
-       LDAP            *ld;
-       int             is_priv = 0;
+       struct ldapconn *lc,
+                       lc_curr = { 0 };
+       int             refcnt = 1;
 
        /* Searches for a ldapconn in the avl tree */
 
        /* Explicit binds must not be shared */
        if ( op->o_tag == LDAP_REQ_BIND
-               || ( op->o_conn && op->o_bd == op->o_conn->c_authz_backend ) ) {
-               lc_curr.conn = op->o_conn;
+               || ( op->o_conn
+                       && op->o_conn->c_authz_backend
+                       && op->o_bd->be_private == op->o_conn->c_authz_backend->be_private ) )
+       {
+               lc_curr.lc_conn = op->o_conn;
 
        } else {
-               lc_curr.conn = NULL;
+#ifdef HAVE_TLS
+               if ( op->o_conn->c_is_tls ) {
+                       lc_curr.lc_conn = LDAP_BACK_PRIV_CONN_TLS;
+               } else
+#endif /* HAVE_TLS */
+               {
+                       lc_curr.lc_conn = LDAP_BACK_PRIV_CONN;
+               }
        }
        
        /* Internal searches are privileged and shared. So is root. */
+       /* FIXME: there seem to be concurrency issues */
        if ( op->o_do_not_cache || be_isroot( op ) ) {
-               lc_curr.local_dn = op->o_bd->be_rootndn;
-               lc_curr.conn = NULL;
-               is_priv = 1;
+               lc_curr.lc_local_ndn = op->o_bd->be_rootndn;
+#ifdef HAVE_TLS
+               if ( op->o_conn->c_is_tls ) {
+                       lc_curr.lc_conn = LDAP_BACK_PRIV_CONN_TLS;
+               } else
+#endif /* HAVE_TLS */
+               {
+                       lc_curr.lc_conn = LDAP_BACK_PRIV_CONN;
+               }
+               lc_curr.lc_ispriv = 1;
 
        } else {
-               lc_curr.local_dn = op->o_ndn;
+               lc_curr.lc_local_ndn = op->o_ndn;
        }
 
        ldap_pvt_thread_mutex_lock( &li->conn_mutex );
+
        lc = (struct ldapconn *)avl_find( li->conntree, 
-               (caddr_t)&lc_curr, ldap_back_conn_cmp );
+                       (caddr_t)&lc_curr, ldap_back_conn_cmp );
+       if ( lc != NULL ) {
+               refcnt = ++lc->lc_refcnt;
+       }
        ldap_pvt_thread_mutex_unlock( &li->conn_mutex );
 
        /* Looks like we didn't get a bind. Open a new session... */
-       if (!lc) {
-               int vers = op->o_protocol;
-               rs->sr_err = ldap_initialize(&ld, li->url);
-               
-               if (rs->sr_err != LDAP_SUCCESS) {
-                       rs->sr_err = slap_map_api2result( rs );
-                       if (rs->sr_text == NULL) {
-                               rs->sr_text = "ldap_initialize() failed";
-                       }
-                       if (op->o_conn) send_ldap_result( op, rs );
-                       rs->sr_text = NULL;
-                       return( NULL );
+       if ( lc == NULL ) {
+               if ( ldap_back_prepare_conn( &lc, op, rs, sendok ) != LDAP_SUCCESS ) {
+                       return NULL;
                }
-               /* Set LDAP version. This will always succeed: If the client
-                * bound with a particular version, then so can we.
-                */
-               ldap_set_option(ld, LDAP_OPT_PROTOCOL_VERSION,
-                               (const void *)&vers);
-               /* FIXME: configurable? */
-               ldap_set_option(ld, LDAP_OPT_REFERRALS, LDAP_OPT_ON);
-
-               lc = (struct ldapconn *)ch_malloc(sizeof(struct ldapconn));
-               lc->conn = lc_curr.conn;
-               lc->ld = ld;
-               ber_dupbv( &lc->local_dn, &lc_curr.local_dn );
 
-#ifdef ENABLE_REWRITE
-               /*
-                * Sets a cookie for the rewrite session
-                *
-                * FIXME: the o_conn might be no longer valid,
-                * since we may have different entries
-                * for the same connection
-                */
-               ( void )rewrite_session_init( li->rwmap.rwm_rw, op->o_conn );
-#endif /* ENABLE_REWRITE */
+               lc->lc_conn = lc_curr.lc_conn;
+               ber_dupbv( &lc->lc_local_ndn, &lc_curr.lc_local_ndn );
 
-               ldap_pvt_thread_mutex_init( &lc->lc_mutex );
-
-               if ( is_priv ) {
-                       ber_dupbv( &lc->cred, &li->acl_passwd );
-                       ber_dupbv( &lc->bound_dn, &li->acl_authcDN );
+               if ( lc_curr.lc_ispriv ) {
+                       ber_dupbv( &lc->lc_cred, &li->acl_passwd );
+                       ber_dupbv( &lc->lc_bound_ndn, &li->acl_authcDN );
+                       lc->lc_ispriv = lc_curr.lc_ispriv;
 
                } else {
-                       BER_BVZERO( &lc->cred );
-                       BER_BVZERO( &lc->bound_dn );
-                       if ( op->o_conn && !BER_BVISEMPTY( &op->o_conn->c_dn )
-                                       && ( op->o_bd == op->o_conn->c_authz_backend ) ) {
-                               
-                               dncookie dc;
-                               struct berval bv;
-
-                               /*
-                                * Rewrite the bind dn if needed
-                                */
-                               dc.rwmap = &li->rwmap;
-#ifdef ENABLE_REWRITE
-                               dc.conn = op->o_conn;
-                               dc.rs = rs;
-                               dc.ctx = "bindDN";
-#else
-                               dc.tofrom = 1;
-                               dc.normalized = 0;
-#endif
+                       BER_BVZERO( &lc->lc_cred );
+                       BER_BVZERO( &lc->lc_bound_ndn );
+                       if ( op->o_conn && !BER_BVISEMPTY( &op->o_ndn )
+                               && op->o_bd->be_private == op->o_conn->c_authz_backend->be_private )
+                       {
+                               ber_dupbv( &lc->lc_bound_ndn, &op->o_ndn );
+                       }
+               }
 
-                               if ( ldap_back_dn_massage( &dc, &op->o_conn->c_dn, &bv ) ) {
-                                       send_ldap_result( op, rs );
-                                       return NULL;
-                               }
+#ifdef HAVE_TLS
+               /* if start TLS failed but it was not mandatory,
+                * check if the non-TLS connection was already
+                * in cache; in case, destroy the newly created
+                * connection and use the existing one */
+               if ( lc->lc_conn == LDAP_BACK_PRIV_CONN_TLS
+                               && !ldap_tls_inplace( lc->lc_ld ) )
+               {
+                       struct ldapconn *tmplc;
+                       
+                       lc_curr.lc_conn = LDAP_BACK_PRIV_CONN;
+                       ldap_pvt_thread_mutex_lock( &li->conn_mutex );
+                       tmplc = (struct ldapconn *)avl_find( li->conntree, 
+                                       (caddr_t)&lc_curr, ldap_back_conn_cmp );
+                       if ( tmplc != NULL ) {
+                               refcnt = ++tmplc->lc_refcnt;
+                               ldap_back_conn_free( lc );
+                               lc = tmplc;
+                       }
+                       ldap_pvt_thread_mutex_unlock( &li->conn_mutex );
 
-                               if ( bv.bv_val == op->o_conn->c_dn.bv_val ) {
-                                       ber_dupbv( &lc->bound_dn, &bv );
-                               } else {
-                                       lc->bound_dn = bv;
-                               }
+                       if ( tmplc != NULL ) {
+                               goto done;
                        }
                }
+#endif /* HAVE_TLS */
 
-               lc->bound = 0;
+               lc->lc_bound = 0;
 
                /* Inserts the newly created ldapconn in the avl tree */
                ldap_pvt_thread_mutex_lock( &li->conn_mutex );
+
+               assert( lc->lc_refcnt == 1 );
                rs->sr_err = avl_insert( &li->conntree, (caddr_t)lc,
                        ldap_back_conn_cmp, ldap_back_conn_dup );
 
@@ -350,42 +571,70 @@ ldap_back_getconn(Operation *op, SlapReply *rs)
                ldap_pvt_thread_mutex_unlock( &li->conn_mutex );
 
                Debug( LDAP_DEBUG_TRACE,
-                       "=>ldap_back_getconn: conn %p inserted\n", (void *) lc, 0, 0 );
+                       "=>ldap_back_getconn: conn %p inserted (refcnt=%u)\n",
+                       (void *)lc, refcnt, 0 );
        
                /* Err could be -1 in case a duplicate ldapconn is inserted */
                if ( rs->sr_err != 0 ) {
                        ldap_back_conn_free( lc );
-                       if (op->o_conn) {
+                       rs->sr_err = LDAP_OTHER;
+                       if ( op->o_conn && ( sendok & LDAP_BACK_SENDERR ) ) {
                                send_ldap_error( op, rs, LDAP_OTHER,
-                               "internal server error" );
+                                       "internal server error" );
                        }
-                       return( NULL );
+                       return NULL;
                }
+
        } else {
                Debug( LDAP_DEBUG_TRACE,
-                       "=>ldap_back_getconn: conn %p fetched\n", (void *) lc, 0, 0 );
+                       "=>ldap_back_getconn: conn %p fetched (refcnt=%u)\n",
+                       (void *)lc, refcnt, 0 );
        }
-       
-       return( lc );
+
+done:;
+       return lc;
+}
+
+void
+ldap_back_release_conn(
+       Operation               *op,
+       SlapReply               *rs,
+       struct ldapconn         *lc )
+{
+       struct ldapinfo *li = (struct ldapinfo *)op->o_bd->be_private;
+
+       ldap_pvt_thread_mutex_lock( &li->conn_mutex );
+       assert( lc->lc_refcnt > 0 );
+       lc->lc_refcnt--;
+       ldap_pvt_thread_mutex_unlock( &li->conn_mutex );
 }
 
 /*
  * ldap_back_dobind
  *
- * Note: as the check for the value of lc->bound was already here, I removed
+ * Note: as the check for the value of lc->lc_bound was already here, I removed
  * it from all the callers, and I made the function return the flag, so
  * it can be used to simplify the check.
+ *
+ * Note: dolock indicates whether li->conn_mutex must be locked or not
  */
-int
-ldap_back_dobind( struct ldapconn *lc, Operation *op, SlapReply *rs )
+static int
+ldap_back_dobind_int(
+       struct ldapconn         *lc,
+       Operation               *op,
+       SlapReply               *rs,
+       ldap_back_send_t        sendok,
+       int                     retries,
+       int                     dolock )
 {      
-       struct ldapinfo *li = (struct ldapinfo *)op->o_bd->be_private;
-       int rc;
-       ber_int_t msgid;
+       int             rc;
+       ber_int_t       msgid;
+
+       assert( retries >= 0 );
+
+       if ( !lc->lc_bound ) {
+               struct ldapinfo *li = (struct ldapinfo *)op->o_bd->be_private;
 
-       ldap_pvt_thread_mutex_lock( &lc->lc_mutex );
-       if ( !lc->bound ) {
-#ifdef LDAP_BACK_PROXY_AUTHZ
                /*
                 * FIXME: we need to let clients use proxyAuthz
                 * otherwise we cannot do symmetric pools of servers;
@@ -400,170 +649,120 @@ ldap_back_dobind( struct ldapconn *lc, Operation *op, SlapReply *rs )
                 */
                /*
                 * if no bind took place yet, but the connection is bound
-                * and the "proxyauthzdn" is set, then bind as 
-                * "proxyauthzdn" and explicitly add the proxyAuthz 
-                * control to every operation with the dn bound 
-                * to the connection as control value.
+                * and the "idassert-authcDN" (or other ID) is set, 
+                * then bind as the asserting identity and explicitly 
+                * add the proxyAuthz control to every operation with the
+                * dn bound to the connection as control value.
+                * This is done also if this is the authrizing backend,
+                * but the "override" flag is given to idassert.
+                * It allows to use SASL bind and yet proxyAuthz users
                 */
-               if ( op->o_conn != NULL && BER_BVISNULL( &lc->bound_dn ) ) {
-                       struct berval   binddn = slap_empty_bv;
-                       struct berval   bindcred = slap_empty_bv;
-                       int             dobind = 0;
-
-                       /* bind as proxyauthzdn only if no idassert mode
-                        * is requested, or if the client's identity
-                        * is authorized */
-                       switch ( li->idassert_mode ) {
-                       case LDAP_BACK_IDASSERT_LEGACY:
-                               if ( !BER_BVISNULL( &op->o_conn->c_dn ) && !BER_BVISEMPTY( &op->o_conn->c_dn ) ) {
-                                       if ( !BER_BVISNULL( &li->idassert_authcDN ) && !BER_BVISEMPTY( &li->idassert_authcDN ) )
-                                       {
-                                               binddn = li->idassert_authcDN;
-                                               bindcred = li->idassert_passwd;
-                                               dobind = 1;
-                                       }
-                               }
-                               break;
+               if ( op->o_conn != NULL &&
+                               !op->o_do_not_cache &&
+                               ( BER_BVISNULL( &lc->lc_bound_ndn ) ||
+                                 ( li->idassert_flags & LDAP_BACK_AUTH_OVERRIDE ) ) )
+               {
+                       (void)ldap_back_proxy_authz_bind( lc, op, rs );
+                       goto done;
+               }
 
-                       default:
-                               if ( li->idassert_authz ) {
-                                       struct berval   authcDN = BER_BVISNULL( &op->o_conn->c_dn ) ? slap_empty_bv : op->o_conn->c_dn;
-
-                                       rs->sr_err = slap_sasl_matches( op, li->idassert_authz,
-                                                       &authcDN, &authcDN );
-                                       if ( rs->sr_err != LDAP_SUCCESS ) {
-                                               send_ldap_result( op, rs );
-                                               lc->bound = 0;
-                                               goto done;
-                                       }
+#ifdef HAVE_CYRUS_SASL
+               if ( lc->lc_ispriv && li->acl_authmethod == LDAP_AUTH_SASL ) {
+                       void            *defaults = NULL;
+
+#if 1  /* will deal with this later... */
+                       if ( li->acl_secprops != NULL ) {
+                               rc = ldap_set_option( lc->lc_ld,
+                                       LDAP_OPT_X_SASL_SECPROPS, li->acl_secprops);
+
+                               if( rc != LDAP_OPT_SUCCESS ) {
+                                       Debug( LDAP_DEBUG_ANY, "Error: ldap_set_option "
+                                               "(%s,SECPROPS,\"%s\") failed!\n",
+                                               li->url, li->acl_secprops, 0 );
+                                       goto done;
                                }
-
-                               binddn = li->idassert_authcDN;
-                               bindcred = li->idassert_passwd;
-                               dobind = 1;
-                               break;
                        }
+#endif
 
-                       if ( dobind && li->idassert_authmethod == LDAP_AUTH_SASL ) {
-#ifdef HAVE_CYRUS_SASL
-                               void            *defaults = NULL;
-                               struct berval   authzID = BER_BVNULL;
-                               int             freeauthz = 0;
-
-                               /* if SASL supports native authz, prepare for it */
-                               if ( ( !op->o_do_not_cache || !op->o_is_auth_check ) &&
-                                               ( li->idassert_flags & LDAP_BACK_AUTH_NATIVE_AUTHZ ) )
-                               {
-                                       switch ( li->idassert_mode ) {
-                                       case LDAP_BACK_IDASSERT_OTHERID:
-                                       case LDAP_BACK_IDASSERT_OTHERDN:
-                                               authzID = li->idassert_authzID;
-                                               break;
-
-                                       case LDAP_BACK_IDASSERT_ANONYMOUS:
-                                               BER_BVSTR( &authzID, "dn:" );
-                                               break;
-
-                                       case LDAP_BACK_IDASSERT_SELF:
-                                               if ( BER_BVISNULL( &op->o_conn->c_dn ) ) {
-                                                       /* connection is not authc'd, so don't idassert */
-                                                       BER_BVSTR( &authzID, "dn:" );
-                                                       break;
-                                               }
-                                               authzID.bv_len = STRLENOF( "dn:" ) + op->o_conn->c_dn.bv_len;
-                                               authzID.bv_val = slap_sl_malloc( authzID.bv_len + 1, op->o_tmpmemctx );
-                                               AC_MEMCPY( authzID.bv_val, "dn:", STRLENOF( "dn:" ) );
-                                               AC_MEMCPY( authzID.bv_val + STRLENOF( "dn:" ),
-                                                               op->o_conn->c_dn.bv_val, op->o_conn->c_dn.bv_len + 1 );
-                                               freeauthz = 1;
-                                               break;
-
-                                       default:
-                                               break;
-                                       }
-                               }
+                       defaults = lutil_sasl_defaults( lc->lc_ld,
+                                       li->acl_sasl_mech.bv_val,
+                                       li->acl_sasl_realm.bv_val,
+                                       li->acl_authcID.bv_val,
+                                       li->acl_passwd.bv_val,
+                                       NULL );
 
-#if 0  /* will deal with this later... */
-                               if ( sasl_secprops != NULL ) {
-                                       rs->sr_err = ldap_set_option( lc->ld, LDAP_OPT_X_SASL_SECPROPS,
-                                               (void *) sasl_secprops );
-
-                                       if ( rs->sr_err != LDAP_OPT_SUCCESS ) {
-                                               send_ldap_result( op, rs );
-                                               lc->bound = 0;
-                                               goto done;
-                                               
-                                       }
-                               }
-#endif
+                       rs->sr_err = ldap_sasl_interactive_bind_s( lc->lc_ld,
+                                       li->acl_authcDN.bv_val,
+                                       li->acl_sasl_mech.bv_val, NULL, NULL,
+                                       LDAP_SASL_QUIET, lutil_sasl_interact,
+                                       defaults );
 
-                               defaults = lutil_sasl_defaults( lc->ld,
-                                               li->idassert_sasl_mech.bv_val,
-                                               li->idassert_sasl_realm.bv_val,
-                                               li->idassert_authcID.bv_val,
-                                               li->idassert_passwd.bv_val,
-                                               authzID.bv_val );
-
-                               rs->sr_err = ldap_sasl_interactive_bind_s( lc->ld, binddn.bv_val,
-                                               li->idassert_sasl_mech.bv_val, NULL, NULL,
-                                               li->idassert_sasl_flags, lutil_sasl_interact,
-                                               defaults );
-
-                               lutil_sasl_freedefs( defaults );
-                               if ( freeauthz ) {
-                                       slap_sl_free( authzID.bv_val, op->o_tmpmemctx );
-                               }
+                       lutil_sasl_freedefs( defaults );
 
-                               rs->sr_err = slap_map_api2result( rs );
-                               if ( rs->sr_err != LDAP_SUCCESS ) {
-                                       lc->bound = 0;
-                                       send_ldap_result( op, rs );
+                       rs->sr_err = slap_map_api2result( rs );
+                       if ( rs->sr_err != LDAP_SUCCESS ) {
+                               lc->lc_bound = 0;
+                               send_ldap_result( op, rs );
 
-                               } else {
-                                       lc->bound = 1;
-                               }
-                               goto done;
-#endif /* HAVE_CYRUS_SASL */
+                       } else {
+                               lc->lc_bound = 1;
                        }
+                       goto done;
+               }
+#endif /* HAVE_CYRUS_SASL */
 
-                       switch ( li->idassert_authmethod ) {
-                       case LDAP_AUTH_SIMPLE:
-                               rs->sr_err = ldap_sasl_bind(lc->ld,
-                                               binddn.bv_val, LDAP_SASL_SIMPLE,
-                                               &bindcred, NULL, NULL, &msgid);
-                               break;
+retry:;
+               rs->sr_err = ldap_sasl_bind( lc->lc_ld,
+                               lc->lc_bound_ndn.bv_val,
+                               LDAP_SASL_SIMPLE, &lc->lc_cred,
+                               NULL, NULL, &msgid );
 
-                       case LDAP_AUTH_NONE:
-                               lc->bound = 1;
-                               goto done;
+               if ( rs->sr_err == LDAP_SERVER_DOWN ) {
+                       if ( retries > 0 ) {
+                               if ( dolock ) {
+                                       ldap_pvt_thread_mutex_lock( &li->conn_mutex );
+                               }
 
-                       default:
-                               /* unsupported! */
-                               lc->bound = 0;
-                               rs->sr_err = LDAP_AUTH_METHOD_NOT_SUPPORTED;
-                               send_ldap_result( op, rs );
-                               goto done;
+                               assert( lc->lc_refcnt > 0 );
+                               if ( lc->lc_refcnt == 1 ) {
+                                       ldap_unbind_ext( lc->lc_ld, NULL, NULL );
+                                       lc->lc_ld = NULL;
+
+                                       /* lc here must be the regular lc, reset and ready for init */
+                                       rs->sr_err = ldap_back_prepare_conn( &lc, op, rs, sendok );
+                               }
+                               if ( dolock ) {
+                                       ldap_pvt_thread_mutex_unlock( &li->conn_mutex );
+                               }
+                               if ( rs->sr_err == LDAP_SUCCESS ) {
+                                       retries--;
+                                       goto retry;
+                               }
                        }
 
-               } else
-#endif /* LDAP_BACK_PROXY_AUTHZ */
-               {
-                       rs->sr_err = ldap_sasl_bind(lc->ld, lc->bound_dn.bv_val,
-                               LDAP_SASL_SIMPLE, &lc->cred, NULL, NULL, &msgid);
+                       ldap_back_freeconn( op, lc );
+                       rs->sr_err = slap_map_api2result( rs );
+
+                       return 0;
                }
-               
-               rc = ldap_back_op_result( lc, op, rs, msgid, 0 );
-               if (rc == LDAP_SUCCESS) {
-                       lc->bound = 1;
+
+               rc = ldap_back_op_result( lc, op, rs, msgid, sendok );
+               if ( rc == LDAP_SUCCESS ) {
+                       lc->lc_bound = 1;
                }
        }
 
 done:;
-       rc = lc->bound;
-       ldap_pvt_thread_mutex_unlock( &lc->lc_mutex );
+       rc = lc->lc_bound;
        return rc;
 }
 
+int
+ldap_back_dobind( struct ldapconn *lc, Operation *op, SlapReply *rs, ldap_back_send_t sendok )
+{
+       return ldap_back_dobind_int( lc, op, rs, sendok, 1, 1 );
+}
+
 /*
  * ldap_back_rebind
  *
@@ -574,69 +773,41 @@ static int
 ldap_back_rebind( LDAP *ld, LDAP_CONST char *url, ber_tag_t request,
        ber_int_t msgid, void *params )
 {
-       struct ldapconn *lc = params;
+       struct ldapconn *lc = (struct ldapconn *)params;
 
-       return ldap_bind_s( ld, lc->bound_dn.bv_val, lc->cred.bv_val, LDAP_AUTH_SIMPLE );
-}
+#ifdef HAVE_TLS
+       /* ... otherwise we couldn't get here */
+       assert( lc != NULL );
 
-#if 0 /* deprecated in favour of slap_map_api2result() */
-/* Map API errors to protocol errors... */
-int
-ldap_back_map_result( SlapReply *rs )
-{
-       switch(rs->sr_err)
-       {
-       case LDAP_SERVER_DOWN:
-               return LDAP_UNAVAILABLE;
-       case LDAP_LOCAL_ERROR:
-               return LDAP_OTHER;
-       case LDAP_ENCODING_ERROR:
-       case LDAP_DECODING_ERROR:
-               return LDAP_PROTOCOL_ERROR;
-       case LDAP_TIMEOUT:
-               return LDAP_UNAVAILABLE;
-       case LDAP_AUTH_UNKNOWN:
-               return LDAP_AUTH_METHOD_NOT_SUPPORTED;
-       case LDAP_FILTER_ERROR:
-               rs->sr_text = "Filter error";
-               return LDAP_OTHER;
-       case LDAP_USER_CANCELLED:
-               rs->sr_text = "User cancelled";
-               return LDAP_OTHER;
-       case LDAP_PARAM_ERROR:
-               return LDAP_PROTOCOL_ERROR;
-       case LDAP_NO_MEMORY:
-               return LDAP_OTHER;
-       case LDAP_CONNECT_ERROR:
-               return LDAP_UNAVAILABLE;
-       case LDAP_NOT_SUPPORTED:
-               return LDAP_UNWILLING_TO_PERFORM;
-       case LDAP_CONTROL_NOT_FOUND:
-               return LDAP_PROTOCOL_ERROR;
-       case LDAP_NO_RESULTS_RETURNED:
-               return LDAP_NO_SUCH_OBJECT;
-       case LDAP_MORE_RESULTS_TO_RETURN:
-               rs->sr_text = "More results to return";
-               return LDAP_OTHER;
-       case LDAP_CLIENT_LOOP:
-       case LDAP_REFERRAL_LIMIT_EXCEEDED:
-               return LDAP_LOOP_DETECT;
-       default:
-               if ( LDAP_API_ERROR(rs->sr_err) )
-                       return LDAP_OTHER;
-               return rs->sr_err;
+       if ( !ldap_tls_inplace( ld ) ) {
+               int             is_tls = lc->lc_is_tls,
+                               rc;
+               const char      *text = NULL;
+
+               rc = ldap_back_start_tls( ld, 0, &is_tls, url, lc->lc_flags, &text );
+               if ( rc != LDAP_SUCCESS ) {
+                       return rc;
+               }
        }
+#endif /* HAVE_TLS */
+
+       /* FIXME: add checks on the URL/identity? */
+
+       return ldap_sasl_bind_s( ld, lc->lc_bound_ndn.bv_val,
+                       LDAP_SASL_SIMPLE, &lc->lc_cred, NULL, NULL, NULL );
 }
-#endif
 
 int
-ldap_back_op_result(struct ldapconn *lc, Operation *op, SlapReply *rs,
-       ber_int_t msgid, int sendok)
+ldap_back_op_result(
+               struct ldapconn         *lc,
+               Operation               *op,
+               SlapReply               *rs,
+               ber_int_t               msgid,
+               ldap_back_send_t        sendok )
 {
-       struct ldapinfo *li = (struct ldapinfo *)op->o_bd->be_private;
-       char *match = NULL;
-       LDAPMessage *res = NULL;
-       char *text = NULL;
+       char            *match = NULL;
+       LDAPMessage     *res = NULL;
+       char            *text = NULL;
 
 #define        ERR_OK(err) ((err) == LDAP_SUCCESS || (err) == LDAP_COMPARE_FALSE || (err) == LDAP_COMPARE_TRUE)
 
@@ -647,20 +818,35 @@ ldap_back_op_result(struct ldapconn *lc, Operation *op, SlapReply *rs,
         * to a successful state, get the error from the
         * remote server response */
        if ( ERR_OK( rs->sr_err ) ) {
+               int             rc;
+               struct timeval  tv = { 0, 0 };
+
+retry:;
                /* if result parsing fails, note the failure reason */
-               if ( ldap_result( lc->ld, msgid, 1, NULL, &res ) == -1 ) {
-                       ldap_get_option( lc->ld, LDAP_OPT_ERROR_NUMBER,
+               switch ( ldap_result( lc->lc_ld, msgid, 1, &tv, &res ) ) {
+               case 0:
+                       tv.tv_sec = 0;
+                       tv.tv_usec = 100000;    /* 0.1 s */
+                       ldap_pvt_thread_yield();
+                       goto retry;
+
+               case -1:
+                       ldap_get_option( lc->lc_ld, LDAP_OPT_ERROR_NUMBER,
                                        &rs->sr_err );
+                       break;
+
 
                /* otherwise get the result; if it is not
                 * LDAP_SUCCESS, record it in the reply
                 * structure (this includes 
                 * LDAP_COMPARE_{TRUE|FALSE}) */
-               } else {
-                       int rc = ldap_parse_result( lc->ld, res, &rs->sr_err,
+               default:
+                       rc = ldap_parse_result( lc->lc_ld, res, &rs->sr_err,
                                        &match, &text, NULL, NULL, 1 );
                        rs->sr_text = text;
-                       if ( rc != LDAP_SUCCESS ) rs->sr_err = rc;
+                       if ( rc != LDAP_SUCCESS ) {
+                               rs->sr_err = rc;
+                       }
                }
        }
 
@@ -673,28 +859,16 @@ ldap_back_op_result(struct ldapconn *lc, Operation *op, SlapReply *rs,
                /* internal ops ( op->o_conn == NULL ) 
                 * must not reply to client */
                if ( op->o_conn && !op->o_do_not_cache && match ) {
-                       struct berval dn, mdn;
-                       dncookie dc;
-
-                       dc.rwmap = &li->rwmap;
-#ifdef ENABLE_REWRITE
-                       dc.conn = op->o_conn;
-                       dc.rs = rs;
-                       dc.ctx = "matchedDN";
-#else
-                       dc.tofrom = 0;
-                       dc.normalized = 0;
-#endif
-                       ber_str2bv(match, 0, 0, &dn);
-                       ldap_back_dn_massage(&dc, &dn, &mdn);
 
                        /* record the (massaged) matched
                         * DN into the reply structure */
-                       rs->sr_matched = mdn.bv_val;
-                               
+                       rs->sr_matched = match;
                }
        }
-       if ( op->o_conn && ( sendok || rs->sr_err != LDAP_SUCCESS ) ) {
+       if ( op->o_conn &&
+                       ( ( sendok & LDAP_BACK_SENDOK ) 
+                         || ( ( sendok & LDAP_BACK_SENDERR ) && rs->sr_err != LDAP_SUCCESS ) ) )
+       {
                send_ldap_result( op, rs );
        }
        if ( match ) {
@@ -711,7 +885,220 @@ ldap_back_op_result(struct ldapconn *lc, Operation *op, SlapReply *rs,
        return( ERR_OK( rs->sr_err ) ? 0 : -1 );
 }
 
-#ifdef LDAP_BACK_PROXY_AUTHZ
+/* return true if bound, false if failed */
+int
+ldap_back_retry( struct ldapconn *lc, Operation *op, SlapReply *rs, ldap_back_send_t sendok )
+{
+       int             rc = 0;
+       struct ldapinfo *li = (struct ldapinfo *)op->o_bd->be_private;
+       
+       ldap_pvt_thread_mutex_lock( &li->conn_mutex );
+
+       if ( lc->lc_refcnt == 1 ) {
+               ldap_unbind_ext( lc->lc_ld, NULL, NULL );
+               lc->lc_ld = NULL;
+               lc->lc_bound = 0;
+
+               /* lc here must be the regular lc, reset and ready for init */
+               rc = ldap_back_prepare_conn( &lc, op, rs, sendok );
+               if ( rc == LDAP_SUCCESS ) {
+                       rc = ldap_back_dobind_int( lc, op, rs, sendok, 0, 0 );
+               }
+       }
+
+       ldap_pvt_thread_mutex_unlock( &li->conn_mutex );
+
+       return rc;
+}
+
+static int
+ldap_back_proxy_authz_bind( struct ldapconn *lc, Operation *op, SlapReply *rs )
+{
+       struct ldapinfo *li = (struct ldapinfo *)op->o_bd->be_private;
+       struct berval   binddn = slap_empty_bv;
+       struct berval   bindcred = slap_empty_bv;
+       int             dobind = 0;
+       int             msgid;
+       int             rc;
+
+       /*
+        * FIXME: we need to let clients use proxyAuthz
+        * otherwise we cannot do symmetric pools of servers;
+        * we have to live with the fact that a user can
+        * authorize itself as any ID that is allowed
+        * by the authzTo directive of the "proxyauthzdn".
+        */
+       /*
+        * NOTE: current Proxy Authorization specification
+        * and implementation do not allow proxy authorization
+        * control to be provided with Bind requests
+        */
+       /*
+        * if no bind took place yet, but the connection is bound
+        * and the "proxyauthzdn" is set, then bind as 
+        * "proxyauthzdn" and explicitly add the proxyAuthz 
+        * control to every operation with the dn bound 
+        * to the connection as control value.
+        */
+
+       /* bind as proxyauthzdn only if no idassert mode
+        * is requested, or if the client's identity
+        * is authorized */
+       switch ( li->idassert_mode ) {
+       case LDAP_BACK_IDASSERT_LEGACY:
+               if ( !BER_BVISNULL( &op->o_conn->c_ndn ) && !BER_BVISEMPTY( &op->o_conn->c_ndn ) ) {
+                       if ( !BER_BVISNULL( &li->idassert_authcDN ) && !BER_BVISEMPTY( &li->idassert_authcDN ) )
+                       {
+                               binddn = li->idassert_authcDN;
+                               bindcred = li->idassert_passwd;
+                               dobind = 1;
+                       }
+               }
+               break;
+
+       default:
+               /* NOTE: rootdn can always idassert */
+               if ( li->idassert_authz && !be_isroot( op ) ) {
+                       struct berval authcDN;
+
+                       if ( BER_BVISNULL( &op->o_conn->c_ndn ) ) {
+                               authcDN = slap_empty_bv;
+
+                       } else {
+                               authcDN = op->o_conn->c_ndn;
+                       }       
+                       rs->sr_err = slap_sasl_matches( op, li->idassert_authz,
+                                       &authcDN, &authcDN );
+                       if ( rs->sr_err != LDAP_SUCCESS ) {
+                               if ( li->idassert_flags & LDAP_BACK_AUTH_PRESCRIPTIVE ) {
+                                       send_ldap_result( op, rs );
+                                       lc->lc_bound = 0;
+
+                               } else {
+                                       rs->sr_err = LDAP_SUCCESS;
+                                       binddn = slap_empty_bv;
+                                       bindcred = slap_empty_bv;
+                                       break;
+                               }
+
+                               goto done;
+                       }
+               }
+
+               binddn = li->idassert_authcDN;
+               bindcred = li->idassert_passwd;
+               dobind = 1;
+               break;
+       }
+
+       if ( dobind && li->idassert_authmethod == LDAP_AUTH_SASL ) {
+#ifdef HAVE_CYRUS_SASL
+               void            *defaults = NULL;
+               struct berval   authzID = BER_BVNULL;
+               int             freeauthz = 0;
+
+               /* if SASL supports native authz, prepare for it */
+               if ( ( !op->o_do_not_cache || !op->o_is_auth_check ) &&
+                               ( li->idassert_flags & LDAP_BACK_AUTH_NATIVE_AUTHZ ) )
+               {
+                       switch ( li->idassert_mode ) {
+                       case LDAP_BACK_IDASSERT_OTHERID:
+                       case LDAP_BACK_IDASSERT_OTHERDN:
+                               authzID = li->idassert_authzID;
+                               break;
+
+                       case LDAP_BACK_IDASSERT_ANONYMOUS:
+                               BER_BVSTR( &authzID, "dn:" );
+                               break;
+
+                       case LDAP_BACK_IDASSERT_SELF:
+                               if ( BER_BVISNULL( &op->o_conn->c_ndn ) ) {
+                                       /* connection is not authc'd, so don't idassert */
+                                       BER_BVSTR( &authzID, "dn:" );
+                                       break;
+                               }
+                               authzID.bv_len = STRLENOF( "dn:" ) + op->o_conn->c_ndn.bv_len;
+                               authzID.bv_val = slap_sl_malloc( authzID.bv_len + 1, op->o_tmpmemctx );
+                               AC_MEMCPY( authzID.bv_val, "dn:", STRLENOF( "dn:" ) );
+                               AC_MEMCPY( authzID.bv_val + STRLENOF( "dn:" ),
+                                               op->o_conn->c_ndn.bv_val, op->o_conn->c_ndn.bv_len + 1 );
+                               freeauthz = 1;
+                               break;
+
+                       default:
+                               break;
+                       }
+               }
+
+#if 0  /* will deal with this later... */
+               if ( sasl_secprops != NULL ) {
+                       rs->sr_err = ldap_set_option( lc->lc_ld, LDAP_OPT_X_SASL_SECPROPS,
+                               (void *) sasl_secprops );
+
+                       if ( rs->sr_err != LDAP_OPT_SUCCESS ) {
+                               send_ldap_result( op, rs );
+                               lc->lc_bound = 0;
+                               goto done;
+                       }
+               }
+#endif
+
+               defaults = lutil_sasl_defaults( lc->lc_ld,
+                               li->idassert_sasl_mech.bv_val,
+                               li->idassert_sasl_realm.bv_val,
+                               li->idassert_authcID.bv_val,
+                               li->idassert_passwd.bv_val,
+                               authzID.bv_val );
+
+               rs->sr_err = ldap_sasl_interactive_bind_s( lc->lc_ld, binddn.bv_val,
+                               li->idassert_sasl_mech.bv_val, NULL, NULL,
+                               LDAP_SASL_QUIET, lutil_sasl_interact,
+                               defaults );
+
+               lutil_sasl_freedefs( defaults );
+               if ( freeauthz ) {
+                       slap_sl_free( authzID.bv_val, op->o_tmpmemctx );
+               }
+
+               rs->sr_err = slap_map_api2result( rs );
+               if ( rs->sr_err != LDAP_SUCCESS ) {
+                       lc->lc_bound = 0;
+                       send_ldap_result( op, rs );
+
+               } else {
+                       lc->lc_bound = 1;
+               }
+               goto done;
+#endif /* HAVE_CYRUS_SASL */
+       }
+
+       switch ( li->idassert_authmethod ) {
+       case LDAP_AUTH_SIMPLE:
+               rs->sr_err = ldap_sasl_bind( lc->lc_ld,
+                               binddn.bv_val, LDAP_SASL_SIMPLE,
+                               &bindcred, NULL, NULL, &msgid );
+               break;
+
+       case LDAP_AUTH_NONE:
+               lc->lc_bound = 1;
+               goto done;
+
+       default:
+               /* unsupported! */
+               lc->lc_bound = 0;
+               rs->sr_err = LDAP_AUTH_METHOD_NOT_SUPPORTED;
+               send_ldap_result( op, rs );
+               goto done;
+       }
+
+       rc = ldap_back_op_result( lc, op, rs, msgid, LDAP_BACK_SENDERR );
+       if ( rc == LDAP_SUCCESS ) {
+               lc->lc_bound = 1;
+       }
+done:;
+       return lc->lc_bound;
+}
+
 /*
  * ldap_back_proxy_authz_ctrl() prepends a proxyAuthz control
  * to existing server-side controls if required; if not,
@@ -731,8 +1118,12 @@ ldap_back_op_result(struct ldapconn *lc, Operation *op, SlapReply *rs,
  * If no server-side controls are defined for the operation,
  * simply add the proxyAuthz control; otherwise, if the
  * proxyAuthz control is not already set, add it as
- * the first one (FIXME: is controls order significant
- * for security?).
+ * the first one
+ *
+ * FIXME: is controls order significant for security?
+ * ANSWER: controls ordering and interoperability
+ * must be indicated by the specs of each control; if none
+ * is specified, the order is irrelevant.
  */
 int
 ldap_back_proxy_authz_ctrl(
@@ -749,12 +1140,18 @@ ldap_back_proxy_authz_ctrl(
 
        *pctrls = NULL;
 
+       rs->sr_err = LDAP_SUCCESS;
+
+       /* FIXME: SASL/EXTERNAL over ldapi:// doesn't honor the authcID,
+        * but if it is not set this test fails.  We need a different
+        * means to detect if idassert is enabled */
        if ( ( BER_BVISNULL( &li->idassert_authcID ) || BER_BVISEMPTY( &li->idassert_authcID ) )
-                       && ( BER_BVISNULL( &li->idassert_authcDN ) || BER_BVISEMPTY( &li->idassert_authcDN ) ) ) {
+                       && ( BER_BVISNULL( &li->idassert_authcDN ) || BER_BVISEMPTY( &li->idassert_authcDN ) ) )
+       {
                goto done;
        }
 
-       if ( !op->o_conn ) {
+       if ( !op->o_conn || op->o_do_not_cache || be_isroot( op ) ) {
                goto done;
        }
 
@@ -777,11 +1174,11 @@ ldap_back_proxy_authz_ctrl(
                        goto done;
                }
 
-               if ( !BER_BVISNULL( &lc->bound_dn ) ) {
+               if ( !BER_BVISNULL( &lc->lc_bound_ndn ) ) {
                        goto done;
                }
 
-               if ( BER_BVISNULL( &op->o_conn->c_dn ) ) {
+               if ( BER_BVISNULL( &op->o_conn->c_ndn ) ) {
                        goto done;
                }
 
@@ -791,26 +1188,34 @@ ldap_back_proxy_authz_ctrl(
 
        } else if ( li->idassert_authmethod == LDAP_AUTH_SASL ) {
                if ( ( li->idassert_flags & LDAP_BACK_AUTH_NATIVE_AUTHZ )
-                               /* && ( !BER_BVISNULL( &op->o_conn->c_dn ) || lc->bound ) */ )
+                               /* && ( !BER_BVISNULL( &op->o_conn->c_ndn ) || lc->lc_bound ) */ )
                {
                        /* already asserted in SASL via native authz */
-                       /* NOTE: the test on lc->bound is used to trap
+                       /* NOTE: the test on lc->lc_bound is used to trap
                         * native authorization of anonymous users,
-                        * since in that case op->o_conn->c_dn is NULL */
+                        * since in that case op->o_conn->c_ndn is NULL */
                        goto done;
                }
 
-       } else if ( li->idassert_authz ) {
+       } else if ( li->idassert_authz && !be_isroot( op ) ) {
                int             rc;
-               struct berval   authcDN = BER_BVISNULL( &op->o_conn->c_dn ) ? slap_empty_bv : op->o_conn->c_dn;
-
+               struct berval authcDN;
 
+               if ( BER_BVISNULL( &op->o_conn->c_ndn ) ) {
+                       authcDN = slap_empty_bv;
+               } else {
+                       authcDN = op->o_conn->c_ndn;
+               }
                rc = slap_sasl_matches( op, li->idassert_authz,
                                &authcDN, & authcDN );
                if ( rc != LDAP_SUCCESS ) {
-                       /* op->o_conn->c_dn is not authorized
-                        * to use idassert */
-                       return rc;
+                       if ( li->idassert_flags & LDAP_BACK_AUTH_PRESCRIPTIVE )
+                       {
+                               /* op->o_conn->c_ndn is not authorized
+                                * to use idassert */
+                               return rc;
+                       }
+                       return rs->sr_err;
                }
        }
 
@@ -848,7 +1253,11 @@ ldap_back_proxy_authz_ctrl(
        case LDAP_BACK_IDASSERT_SELF:
                /* original behavior:
                 * assert the client's identity */
-               assertedID = BER_BVISNULL( &op->o_conn->c_dn ) ? slap_empty_bv : op->o_conn->c_dn;
+               if ( BER_BVISNULL( &op->o_conn->c_ndn ) ) {
+                       assertedID = slap_empty_bv;
+               } else {
+                       assertedID = op->o_conn->c_ndn;
+               }
                break;
 
        case LDAP_BACK_IDASSERT_ANONYMOUS:
@@ -874,6 +1283,11 @@ ldap_back_proxy_authz_ctrl(
                assertedID = slap_empty_bv;
        }
 
+       if ( op->o_ctrls ) {
+               for ( i = 0; op->o_ctrls[ i ]; i++ )
+                       /* just count ctrls */ ;
+       }
+
        ctrls = ch_malloc( sizeof( LDAPControl * ) * (i + 2) );
        ctrls[ 0 ] = ch_malloc( sizeof( LDAPControl ) );
        
@@ -892,7 +1306,7 @@ ldap_back_proxy_authz_ctrl(
                ctrls[ 0 ]->ldctl_value.bv_len = assertedID.bv_len + STRLENOF( "dn:" );
                ctrls[ 0 ]->ldctl_value.bv_val = ch_malloc( ctrls[ 0 ]->ldctl_value.bv_len + 1 );
                AC_MEMCPY( ctrls[ 0 ]->ldctl_value.bv_val, "dn:", STRLENOF( "dn:" ) );
-               AC_MEMCPY( ctrls[ 0 ]->ldctl_value.bv_val + STRLENOF( "dn:" ),
+               AC_MEMCPY( &ctrls[ 0 ]->ldctl_value.bv_val[ STRLENOF( "dn:" ) ],
                                assertedID.bv_val, assertedID.bv_len + 1 );
                break;
        }
@@ -913,4 +1327,27 @@ done:;
        
        return rs->sr_err;
 }
-#endif /* LDAP_BACK_PROXY_AUTHZ */
+
+int
+ldap_back_proxy_authz_ctrl_free( Operation *op, LDAPControl ***pctrls )
+{
+       LDAPControl     **ctrls = *pctrls;
+
+       /* we assume that the first control is the proxyAuthz
+        * added by back-ldap, so it's the only one we explicitly 
+        * free */
+       if ( ctrls && ctrls != op->o_ctrls ) {
+               assert( ctrls[ 0 ] != NULL );
+
+               if ( !BER_BVISNULL( &ctrls[ 0 ]->ldctl_value ) ) {
+                       free( ctrls[ 0 ]->ldctl_value.bv_val );
+               }
+
+               free( ctrls[ 0 ] );
+               free( ctrls );
+       } 
+
+       *pctrls = NULL;
+
+       return 0;
+}