]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/sasl.c
Change slap_sasl_authorized to take an Operation instead of a Connection,
[openldap] / servers / slapd / sasl.c
index cd5588579463df07b7826cfc6fd907b3a5372658..a8f3eca30ff548d4bc4300941ebc93a9bd18a2c3 100644 (file)
@@ -292,7 +292,7 @@ slap_sasl_log(
 static const char *slap_propnames[] = {
        "*slapConn", "*authcDN", "*authzDN", NULL };
 
-static Filter *generic_filter;
+static Filter generic_filter = { LDAP_FILTER_PRESENT };
 
 #define        PROP_CONN       0
 #define        PROP_AUTHC      1
@@ -305,7 +305,7 @@ typedef struct lookup_info {
        sasl_server_params_t *sparams;
 } lookup_info;
 
-static slap_sendentry sasl_ap_lookup, sasl_cb_checkpass;
+static slap_response sasl_ap_lookup, sasl_cb_checkpass;
 
 static int
 sasl_ap_lookup( Operation *op, SlapReply *rs )
@@ -318,6 +318,8 @@ sasl_ap_lookup( Operation *op, SlapReply *rs )
        slap_callback *tmp = op->o_callback;
        lookup_info *sl = tmp->sc_private;
 
+       if (rs->sr_type != REP_SEARCH) return 0;
+
        for( i = 0; i < sl->last; i++ ) {
                const char *name = sl->list[i].name;
 
@@ -420,8 +422,7 @@ slap_auxprop_lookup(
        }
 
        if (doit) {
-               slap_callback cb = { slap_cb_null_response,
-                       slap_cb_null_sresult, sasl_ap_lookup, slap_cb_null_sreference, NULL };
+               slap_callback cb = { sasl_ap_lookup, NULL };
 
                cb.sc_private = &sl;
 
@@ -437,12 +438,14 @@ slap_auxprop_lookup(
                        op.o_do_not_cache = 1;
                        op.o_is_auth_check = 1;
                        op.o_threadctx = conn->c_sasl_bindop->o_threadctx;
+                       op.o_tmpmemctx = conn->c_sasl_bindop->o_tmpmemctx;
+                       op.o_tmpmfuncs = conn->c_sasl_bindop->o_tmpmfuncs;
                        op.o_conn = conn;
                        op.o_connid = conn->c_connid;
-                       op.oq_search.rs_scope = LDAP_SCOPE_BASE;
-                       op.oq_search.rs_deref = LDAP_DEREF_NEVER;
-                       op.oq_search.rs_slimit = 1;
-                       op.oq_search.rs_filter = generic_filter;
+                       op.ors_scope = LDAP_SCOPE_BASE;
+                       op.ors_deref = LDAP_DEREF_NEVER;
+                       op.ors_slimit = 1;
+                       op.ors_filter = &generic_filter;
 
                        op.o_bd->be_search( &op, &rs );
                }
@@ -489,6 +492,8 @@ sasl_cb_checkpass( Operation *op, SlapReply *rs )
        Attribute *a;
        struct berval *bv;
        
+       if (rs->sr_type != REP_SEARCH) return 0;
+
        ci->rc = SASL_NOVERIFY;
 
        a = attr_find( rs->sr_entry->e_attrs, slap_schema.si_ad_userPassword );
@@ -497,7 +502,7 @@ sasl_cb_checkpass( Operation *op, SlapReply *rs )
                NULL, ACL_AUTH, NULL ) ) return 0;
 
        for ( bv = a->a_vals; bv->bv_val != NULL; bv++ ) {
-               if ( !lutil_passwd( bv, &ci->cred, NULL ) ) {
+               if ( !lutil_passwd( bv, &ci->cred, NULL, &rs->sr_text ) ) {
                        ci->rc = SASL_OK;
                        break;
                }
@@ -525,7 +530,7 @@ slap_sasl_checkpass(
         * find an answer here.
         */
 
-       rc = slap_sasl_getdn( conn, (char *)username, 0, NULL, &op.o_req_ndn,
+       rc = slap_sasl_getdn( conn, NULL, (char *)username, 0, NULL, &op.o_req_ndn,
                SLAP_GETDN_AUTHCID );
        if ( rc != LDAP_SUCCESS ) {
                sasl_seterror( sconn, 0, ldap_err2string( rc ) );
@@ -543,8 +548,7 @@ slap_sasl_checkpass(
 
        op.o_bd = select_backend( &op.o_req_ndn, 0, 1 );
        if ( op.o_bd && op.o_bd->be_search ) {
-               slap_callback cb = { slap_cb_null_response,
-                       slap_cb_null_sresult, sasl_cb_checkpass, slap_cb_null_sreference, NULL };
+               slap_callback cb = { sasl_cb_checkpass, NULL };
                SlapReply rs = {REP_RESULT};
 
                ci.cred.bv_val = (char *)pass;
@@ -559,12 +563,14 @@ slap_sasl_checkpass(
                op.o_do_not_cache = 1;
                op.o_is_auth_check = 1;
                op.o_threadctx = conn->c_sasl_bindop->o_threadctx;
+               op.o_tmpmemctx = conn->c_sasl_bindop->o_tmpmemctx;
+               op.o_tmpmfuncs = conn->c_sasl_bindop->o_tmpmfuncs;
                op.o_conn = conn;
                op.o_connid = conn->c_connid;
-               op.oq_search.rs_scope = LDAP_SCOPE_BASE;
-               op.oq_search.rs_deref = LDAP_DEREF_NEVER;
-               op.oq_search.rs_slimit = 1;
-               op.oq_search.rs_filter = generic_filter;
+               op.ors_scope = LDAP_SCOPE_BASE;
+               op.ors_deref = LDAP_DEREF_NEVER;
+               op.ors_slimit = 1;
+               op.ors_filter = &generic_filter;
 
                op.o_bd->be_search( &op, &rs );
        }
@@ -669,7 +675,7 @@ slap_sasl_canonicalize(
                if ( !rc ) goto done;
        }
 
-       rc = slap_sasl_getdn( conn, (char *)in, inlen, (char *)user_realm, &dn,
+       rc = slap_sasl_getdn( conn, NULL, (char *)in, inlen, (char *)user_realm, &dn,
                (flags & SASL_CU_AUTHID) ? SLAP_GETDN_AUTHCID : SLAP_GETDN_AUTHZID );
        if ( rc != LDAP_SUCCESS ) {
                sasl_seterror( sconn, 0, ldap_err2string( rc ) );
@@ -746,7 +752,7 @@ slap_sasl_authorize(
        
        AC_MEMCPY( &authzDN, auxvals[1].values[0], sizeof(authzDN) );
 
-       rc = slap_sasl_authorized( conn, &authcDN, &authzDN );
+       rc = slap_sasl_authorized( conn->c_sasl_bindop, &authcDN, &authzDN );
        ch_free( authcDN.bv_val );
        if ( rc != LDAP_SUCCESS ) {
 #ifdef NEW_LOGGING
@@ -819,7 +825,7 @@ slap_sasl_authorize(
 #endif
 
        /* Figure out how much data we have for the dn */
-       rc = sasl_getprop( conn->c_sasl_context, SASL_REALM, (void **)&realm );
+       rc = sasl_getprop( conn->c_sasl_authctx, SASL_REALM, (void **)&realm );
        if( rc != SASL_OK && rc != SASL_NOTDONE ) {
 #ifdef NEW_LOGGING
                LDAP_LOG( TRANSPORT, ERR,
@@ -834,7 +840,7 @@ slap_sasl_authorize(
 
        /* Convert the identities to DN's. If no authzid was given, client will
           be bound as the DN matching their username */
-       rc = slap_sasl_getdn( conn, (char *)authcid, 0, realm,
+       rc = slap_sasl_getdn( conn, NULL, (char *)authcid, 0, realm,
                &authcDN, SLAP_GETDN_AUTHCID );
        if( rc != LDAP_SUCCESS ) {
                *errstr = ldap_err2string( rc );
@@ -853,7 +859,7 @@ slap_sasl_authorize(
                conn->c_sasl_dn = authcDN;
                goto ok;
        }
-       rc = slap_sasl_getdn( conn, (char *)authzid, 0, realm,
+       rc = slap_sasl_getdn( conn, NULL, (char *)authzid, 0, realm,
                &authzDN, SLAP_GETDN_AUTHZID );
        if( rc != LDAP_SUCCESS ) {
                ch_free( authcDN.bv_val );
@@ -861,7 +867,7 @@ slap_sasl_authorize(
                return SASL_NOAUTHZ;
        }
 
-       rc = slap_sasl_authorized(conn, &authcDN, &authzDN );
+       rc = slap_sasl_authorized(conn->c_sasl_bindop, &authcDN, &authzDN );
        ch_free( authcDN.bv_val );
        if( rc ) {
 #ifdef NEW_LOGGING
@@ -909,6 +915,9 @@ slap_sasl_err2ldap( int saslerr )
        int rc;
 
        switch (saslerr) {
+               case SASL_OK:
+                       rc = LDAP_SUCCESS;
+                       break;
                case SASL_CONTINUE:
                        rc = LDAP_SASL_BIND_IN_PROGRESS;
                        break;
@@ -986,6 +995,8 @@ int slap_sasl_init( void )
                ldap_pvt_sasl_mutex_dispose );
 
 #if SASL_VERSION_MAJOR >= 2
+       generic_filter.f_desc = slap_schema.si_ad_objectClass;
+
        sasl_auxprop_add_plugin( "slapd", slap_auxprop_init );
 #endif
        /* should provide callbacks for logging */
@@ -1029,9 +1040,6 @@ int slap_sasl_destroy( void )
 {
 #ifdef HAVE_CYRUS_SASL
        sasl_done();
-#endif
-#if SASL_VERSION_MAJOR >= 2
-       filter_free( generic_filter );
 #endif
        free( global_host );
        global_host = NULL;
@@ -1039,7 +1047,7 @@ int slap_sasl_destroy( void )
        return 0;
 }
 
-int slap_sasl_open( Connection *conn )
+int slap_sasl_open( Connection *conn, int reopen )
 {
        int cb, sc = LDAP_SUCCESS;
 #if SASL_VERSION_MAJOR >= 2
@@ -1050,18 +1058,18 @@ int slap_sasl_open( Connection *conn )
        sasl_conn_t *ctx = NULL;
        sasl_callback_t *session_callbacks;
 
-       assert( conn->c_sasl_context == NULL );
-       assert( conn->c_sasl_extra == NULL );
+       assert( conn->c_sasl_authctx == NULL );
 
-       conn->c_sasl_layers = 0;
+       if ( !reopen ) {
+               assert( conn->c_sasl_extra == NULL );
 
-       session_callbacks =
+               session_callbacks =
 #if SASL_VERSION_MAJOR >= 2
-               SLAP_CALLOC( 5, sizeof(sasl_callback_t));
+                       SLAP_CALLOC( 5, sizeof(sasl_callback_t));
 #else
-               SLAP_CALLOC( 3, sizeof(sasl_callback_t));
+                       SLAP_CALLOC( 3, sizeof(sasl_callback_t));
 #endif
-       if( session_callbacks == NULL ) {
+               if( session_callbacks == NULL ) {
 #ifdef NEW_LOGGING
                        LDAP_LOG( TRANSPORT, ERR, 
                                "slap_sasl_open: SLAP_MALLOC failed", 0, 0, 0 );
@@ -1070,31 +1078,36 @@ int slap_sasl_open( Connection *conn )
                                "slap_sasl_open: SLAP_MALLOC failed", 0, 0, 0 );
 #endif
                        return -1;
-       }
-       conn->c_sasl_extra = session_callbacks;
+               }
+               conn->c_sasl_extra = session_callbacks;
 
-       session_callbacks[cb=0].id = SASL_CB_LOG;
-       session_callbacks[cb].proc = &slap_sasl_log;
-       session_callbacks[cb++].context = conn;
+               session_callbacks[cb=0].id = SASL_CB_LOG;
+               session_callbacks[cb].proc = &slap_sasl_log;
+               session_callbacks[cb++].context = conn;
 
-       session_callbacks[cb].id = SASL_CB_PROXY_POLICY;
-       session_callbacks[cb].proc = &slap_sasl_authorize;
-       session_callbacks[cb++].context = conn;
+               session_callbacks[cb].id = SASL_CB_PROXY_POLICY;
+               session_callbacks[cb].proc = &slap_sasl_authorize;
+               session_callbacks[cb++].context = conn;
 
 #if SASL_VERSION_MAJOR >= 2
-       session_callbacks[cb].id = SASL_CB_CANON_USER;
-       session_callbacks[cb].proc = &slap_sasl_canonicalize;
-       session_callbacks[cb++].context = conn;
-
-       /* XXXX: this should be conditional */
-       session_callbacks[cb].id = SASL_CB_SERVER_USERDB_CHECKPASS;
-       session_callbacks[cb].proc = &slap_sasl_checkpass;
-       session_callbacks[cb++].context = conn;
+               session_callbacks[cb].id = SASL_CB_CANON_USER;
+               session_callbacks[cb].proc = &slap_sasl_canonicalize;
+               session_callbacks[cb++].context = conn;
+
+               /* XXXX: this should be conditional */
+               session_callbacks[cb].id = SASL_CB_SERVER_USERDB_CHECKPASS;
+               session_callbacks[cb].proc = &slap_sasl_checkpass;
+               session_callbacks[cb++].context = conn;
 #endif
 
-       session_callbacks[cb].id = SASL_CB_LIST_END;
-       session_callbacks[cb].proc = NULL;
-       session_callbacks[cb++].context = NULL;
+               session_callbacks[cb].id = SASL_CB_LIST_END;
+               session_callbacks[cb].proc = NULL;
+               session_callbacks[cb++].context = NULL;
+       } else {
+               session_callbacks = conn->c_sasl_extra;
+       }
+
+       conn->c_sasl_layers = 0;
 
        if( global_host == NULL ) {
                global_host = ldap_pvt_get_fqdn( NULL );
@@ -1102,9 +1115,6 @@ int slap_sasl_open( Connection *conn )
 
        /* create new SASL context */
 #if SASL_VERSION_MAJOR >= 2
-       if ( generic_filter == NULL ) {
-               generic_filter = str2filter( "(objectclass=*)" );
-       }
        if ( conn->c_sock_name.bv_len != 0 &&
             strncmp( conn->c_sock_name.bv_val, "IP=", 3 ) == 0) {
                char *p;
@@ -1156,7 +1166,7 @@ int slap_sasl_open( Connection *conn )
                return -1;
        }
 
-       conn->c_sasl_context = ctx;
+       conn->c_sasl_authctx = ctx;
 
        if( sc == SASL_OK ) {
                sc = sasl_setprop( ctx,
@@ -1188,7 +1198,7 @@ int slap_sasl_external(
 {
 #if SASL_VERSION_MAJOR >= 2
        int sc;
-       sasl_conn_t *ctx = conn->c_sasl_context;
+       sasl_conn_t *ctx = conn->c_sasl_authctx;
 
        if ( ctx == NULL ) {
                return LDAP_UNAVAILABLE;
@@ -1208,7 +1218,7 @@ int slap_sasl_external(
 
 #elif defined(HAVE_CYRUS_SASL)
        int sc;
-       sasl_conn_t *ctx = conn->c_sasl_context;
+       sasl_conn_t *ctx = conn->c_sasl_authctx;
        sasl_external_properties_t extprops;
 
        if ( ctx == NULL ) {
@@ -1232,13 +1242,6 @@ int slap_sasl_external(
 
 int slap_sasl_reset( Connection *conn )
 {
-#ifdef HAVE_CYRUS_SASL
-       sasl_conn_t *ctx = conn->c_sasl_context;
-
-       if( ctx != NULL ) {
-       }
-#endif
-       /* must return "anonymous" */
        return LDAP_SUCCESS;
 }
 
@@ -1247,7 +1250,9 @@ char ** slap_sasl_mechs( Connection *conn )
        char **mechs = NULL;
 
 #ifdef HAVE_CYRUS_SASL
-       sasl_conn_t *ctx = conn->c_sasl_context;
+       sasl_conn_t *ctx = conn->c_sasl_authctx;
+
+       if( ctx == NULL ) ctx = conn->c_sasl_sockctx;
 
        if( ctx != NULL ) {
                int sc;
@@ -1283,13 +1288,19 @@ char ** slap_sasl_mechs( Connection *conn )
 int slap_sasl_close( Connection *conn )
 {
 #ifdef HAVE_CYRUS_SASL
-       sasl_conn_t *ctx = conn->c_sasl_context;
+       sasl_conn_t *ctx = conn->c_sasl_authctx;
 
        if( ctx != NULL ) {
                sasl_dispose( &ctx );
        }
+       if ( conn->c_sasl_sockctx && conn->c_sasl_authctx != conn->c_sasl_sockctx ) {
+               ctx = conn->c_sasl_sockctx;
+               sasl_dispose( &ctx );
+       }
 
-       conn->c_sasl_context = NULL;
+       conn->c_sasl_authctx = NULL;
+       conn->c_sasl_sockctx = NULL;
+       conn->c_sasl_done = 0;
 
        free( conn->c_sasl_extra );
        conn->c_sasl_extra = NULL;
@@ -1301,7 +1312,7 @@ int slap_sasl_close( Connection *conn )
 int slap_sasl_bind( Operation *op, SlapReply *rs )
 {
 #ifdef HAVE_CYRUS_SASL
-       sasl_conn_t *ctx = op->o_conn->c_sasl_context;
+       sasl_conn_t *ctx = op->o_conn->c_sasl_authctx;
        struct berval response;
        unsigned reslen = 0;
        int sc;
@@ -1312,14 +1323,14 @@ int slap_sasl_bind( Operation *op, SlapReply *rs )
                op->o_req_dn.bv_len ? op->o_req_dn.bv_val : "",
                op->o_conn->c_sasl_bind_in_progress ? "<continuing>" : 
                op->o_conn->c_sasl_bind_mech.bv_val,
-               op->oq_bind.rb_cred.bv_len );
+               op->orb_cred.bv_len );
 #else
        Debug(LDAP_DEBUG_ARGS,
                "==> sasl_bind: dn=\"%s\" mech=%s datalen=%ld\n",
                op->o_req_dn.bv_len ? op->o_req_dn.bv_val : "",
                op->o_conn->c_sasl_bind_in_progress ? "<continuing>" : 
                op->o_conn->c_sasl_bind_mech.bv_val,
-               op->oq_bind.rb_cred.bv_len );
+               op->orb_cred.bv_len );
 #endif
 
 
@@ -1342,14 +1353,38 @@ int slap_sasl_bind( Operation *op, SlapReply *rs )
 #endif
 
        if ( !op->o_conn->c_sasl_bind_in_progress ) {
+               /* If we already authenticated once, must use a new context */
+               if ( op->o_conn->c_sasl_done ) {
+                       slap_ssf_t ssf = 0;
+                       const char *authid = NULL;
+#if SASL_VERSION_MAJOR >= 2
+                       sasl_getprop( ctx, SASL_SSF_EXTERNAL, (void *)&ssf );
+                       sasl_getprop( ctx, SASL_AUTH_EXTERNAL, (void *)&authid );
+                       if ( authid ) authid = ch_strdup( authid );
+#endif
+                       if ( ctx != op->o_conn->c_sasl_sockctx ) {
+                               sasl_dispose( &ctx );
+                       }
+                       op->o_conn->c_sasl_authctx = NULL;
+                               
+                       slap_sasl_open( op->o_conn, 1 );
+                       ctx = op->o_conn->c_sasl_authctx;
+#if SASL_VERSION_MAJOR >= 2
+                       if ( authid ) {
+                               sasl_setprop( ctx, SASL_SSF_EXTERNAL, &ssf );
+                               sasl_setprop( ctx, SASL_AUTH_EXTERNAL, authid );
+                               ch_free( (char *)authid );
+                       }
+#endif
+               }
                sc = START( ctx,
                        op->o_conn->c_sasl_bind_mech.bv_val,
-                       op->oq_bind.rb_cred.bv_val, op->oq_bind.rb_cred.bv_len,
+                       op->orb_cred.bv_val, op->orb_cred.bv_len,
                        (SASL_CONST char **)&response.bv_val, &reslen, &rs->sr_text );
 
        } else {
                sc = STEP( ctx,
-                       op->oq_bind.rb_cred.bv_val, op->oq_bind.rb_cred.bv_len,
+                       op->orb_cred.bv_val, op->orb_cred.bv_len,
                        (SASL_CONST char **)&response.bv_val, &reslen, &rs->sr_text );
        }
 
@@ -1358,24 +1393,50 @@ int slap_sasl_bind( Operation *op, SlapReply *rs )
        if ( sc == SASL_OK ) {
                sasl_ssf_t *ssf = NULL;
 
-               op->oq_bind.rb_edn = op->o_conn->c_sasl_dn;
+               op->orb_edn = op->o_conn->c_sasl_dn;
                op->o_conn->c_sasl_dn.bv_val = NULL;
                op->o_conn->c_sasl_dn.bv_len = 0;
+               op->o_conn->c_sasl_done = 1;
 
                rs->sr_err = LDAP_SUCCESS;
 
                (void) sasl_getprop( ctx, SASL_SSF, (void *)&ssf );
-               op->oq_bind.rb_ssf = ssf ? *ssf : 0;
+               op->orb_ssf = ssf ? *ssf : 0;
 
-               if( op->oq_bind.rb_ssf ) {
+               ctx = NULL;
+               if( op->orb_ssf ) {
                        ldap_pvt_thread_mutex_lock( &op->o_conn->c_mutex );
                        op->o_conn->c_sasl_layers++;
+
+                       /* If there's an old layer, set sockctx to NULL to
+                        * tell connection_read() to wait for us to finish.
+                        * Otherwise there is a race condition: we have to
+                        * send the Bind response using the old security
+                        * context and then remove it before reading any
+                        * new messages.
+                        */
+                       if ( op->o_conn->c_sasl_sockctx ) {
+                               ctx = op->o_conn->c_sasl_sockctx;
+                               op->o_conn->c_sasl_sockctx = NULL;
+                       } else {
+                               op->o_conn->c_sasl_sockctx = op->o_conn->c_sasl_authctx;
+                       }
                        ldap_pvt_thread_mutex_unlock( &op->o_conn->c_mutex );
                }
 
+               /* Must send response using old security layer */
                if (response.bv_len) rs->sr_sasldata = &response;
                send_ldap_sasl( op, rs );
-
+               
+               /* Now dispose of the old security layer.
+                */
+               if ( ctx ) {
+                       ldap_pvt_thread_mutex_lock( &op->o_conn->c_mutex );
+                       ldap_pvt_sasl_remove( op->o_conn->c_sb );
+                       op->o_conn->c_sasl_sockctx = op->o_conn->c_sasl_authctx;
+                       ldap_pvt_thread_mutex_unlock( &op->o_conn->c_mutex );
+                       sasl_dispose( &ctx );
+               }
        } else if ( sc == SASL_CONTINUE ) {
                rs->sr_err = LDAP_SASL_BIND_IN_PROGRESS,
                rs->sr_sasldata = &response;
@@ -1429,9 +1490,9 @@ slap_sasl_setpass( Operation *op, SlapReply *rs )
        struct berval new = { 0, NULL };
        struct berval old = { 0, NULL };
 
-       assert( ber_bvcmp( &slap_EXOP_MODIFY_PASSWD, &op->oq_extended.rs_reqoid ) == 0 );
+       assert( ber_bvcmp( &slap_EXOP_MODIFY_PASSWD, &op->ore_reqoid ) == 0 );
 
-       rs->sr_err = sasl_getprop( op->o_conn->c_sasl_context, SASL_USERNAME,
+       rs->sr_err = sasl_getprop( op->o_conn->c_sasl_authctx, SASL_USERNAME,
                (SASL_CONST void **)&id.bv_val );
 
        if( rs->sr_err != SASL_OK ) {
@@ -1449,7 +1510,7 @@ slap_sasl_setpass( Operation *op, SlapReply *rs )
                id.bv_val ? id.bv_val : "", 0, 0 );
 #endif
 
-       rs->sr_err = slap_passwd_parse( op->oq_extended.rs_reqdata,
+       rs->sr_err = slap_passwd_parse( op->ore_reqdata,
                NULL, &old, &new, &rs->sr_text );
 
        if( rs->sr_err != LDAP_SUCCESS ) {
@@ -1469,13 +1530,13 @@ slap_sasl_setpass( Operation *op, SlapReply *rs )
        }
 
 #if SASL_VERSION_MAJOR < 2
-       rs->sr_err = sasl_setpass( op->o_conn->c_sasl_context,
+       rs->sr_err = sasl_setpass( op->o_conn->c_sasl_authctx,
                id.bv_val, new.bv_val, new.bv_len, 0, &rs->sr_text );
 #else
-       rs->sr_err = sasl_setpass( op->o_conn->c_sasl_context, id.bv_val,
+       rs->sr_err = sasl_setpass( op->o_conn->c_sasl_authctx, id.bv_val,
                new.bv_val, new.bv_len, old.bv_val, old.bv_len, 0 );
        if( rs->sr_err != SASL_OK ) {
-               rs->sr_text = sasl_errdetail( op->o_conn->c_sasl_context );
+               rs->sr_text = sasl_errdetail( op->o_conn->c_sasl_authctx );
        }
 #endif
        switch(rs->sr_err) {
@@ -1514,7 +1575,7 @@ done:
 
 static struct berval ext_bv = BER_BVC( "EXTERNAL" );
 
-int slap_sasl_getdn( Connection *conn, char *id, int len,
+int slap_sasl_getdn( Connection *conn, Operation *op, char *id, int len,
        char *user_realm, struct berval *dn, int flags )
 {
        char *c1;
@@ -1530,6 +1591,10 @@ int slap_sasl_getdn( Connection *conn, char *id, int len,
                id ? ( *id ? id : "<empty>" ) : "NULL", len, 0 );
 #endif
 
+       if ( !op ) {
+               op = conn->c_sasl_bindop;
+       }
+
        dn->bv_val = NULL;
        dn->bv_len = 0;
 
@@ -1557,7 +1622,7 @@ int slap_sasl_getdn( Connection *conn, char *id, int len,
                        /* EXTERNAL DNs are already normalized */
                        do_norm = 0;
                        is_dn = SET_DN;
-                       ber_str2bv( id, len, 1, dn );
+                       ber_str2bv_x( id, len, 1, dn, op->o_tmpmemctx );
 
                } else {
                        /* convert to u:<username> form */
@@ -1604,7 +1669,7 @@ int slap_sasl_getdn( Connection *conn, char *id, int len,
 
                /* Build the new dn */
                c1 = dn->bv_val;
-               dn->bv_val = SLAP_MALLOC( len+1 );
+               dn->bv_val = sl_malloc( len+1, op->o_tmpmemctx );
                if( dn->bv_val == NULL ) {
 #ifdef NEW_LOGGING
                        LDAP_LOG( TRANSPORT, ERR, 
@@ -1646,16 +1711,16 @@ int slap_sasl_getdn( Connection *conn, char *id, int len,
                /* Dup the DN in any case, so we don't risk 
                 * leaks or dangling pointers later,
                 * and the DN value is '\0' terminated */
-               ber_dupbv( &dn2, dn );
+               ber_dupbv_x( &dn2, dn, op->o_tmpmemctx );
                dn->bv_val = dn2.bv_val;
        }
 
        /* All strings are in DN form now. Normalize if needed. */
        if ( do_norm ) {
-               rc = dnNormalize2( NULL, dn, &dn2 );
+               rc = dnNormalize( 0, NULL, NULL, dn, &dn2, op->o_tmpmemctx );
 
                /* User DNs were constructed above and must be freed now */
-               ch_free( dn->bv_val );
+               sl_free( dn->bv_val, op->o_tmpmemctx );
 
                if ( rc != LDAP_SUCCESS ) {
                        dn->bv_val = NULL;
@@ -1666,9 +1731,9 @@ int slap_sasl_getdn( Connection *conn, char *id, int len,
        }
 
        /* Run thru regexp */
-       slap_sasl2dn( conn, dn, &dn2 );
+       slap_sasl2dn( op, dn, &dn2 );
        if( dn2.bv_val ) {
-               ch_free( dn->bv_val );
+               sl_free( dn->bv_val, op->o_tmpmemctx );
                *dn = dn2;
 #ifdef NEW_LOGGING
                LDAP_LOG( TRANSPORT, ENTRY,