]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/bind.c
fix database operations in tool mode (ITS#3622)
[openldap] / servers / slapd / bind.c
index 7fcf4b498613fc72d4fbdc6e8f46d6b28d61565d..c39caea9f64e37be8a26d0ead16d2685e5fa49bd 100644 (file)
@@ -2,7 +2,7 @@
 /* $OpenLDAP$ */
 /* This work is part of OpenLDAP Software <http://www.openldap.org/>.
  *
- * Copyright 1998-2004 The OpenLDAP Foundation.
+ * Copyright 1998-2005 The OpenLDAP Foundation.
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -31,7 +31,6 @@
 #include <ac/string.h>
 #include <ac/socket.h>
 
-#include "ldap_pvt.h"
 #include "slap.h"
 #ifdef LDAP_SLAPI
 #include "slapi/slapi.h"
@@ -51,11 +50,7 @@ do_bind(
        ber_tag_t tag;
        Backend *be = NULL;
 
-#ifdef NEW_LOGGING
-       LDAP_LOG( OPERATION, ENTRY, "do_bind: conn %d\n", op->o_connid, 0, 0 );
-#else
        Debug( LDAP_DEBUG_TRACE, "do_bind\n", 0, 0, 0 );
-#endif
 
        /*
         * Force to connection to "anonymous" until bind succeeds.
@@ -64,25 +59,26 @@ do_bind(
        if ( op->o_conn->c_sasl_bind_in_progress ) {
                be = op->o_conn->c_authz_backend;
        }
-       if ( op->o_conn->c_dn.bv_len ) {
+       if ( !BER_BVISEMPTY( &op->o_conn->c_dn ) ) {
                /* log authorization identity demotion */
                Statslog( LDAP_DEBUG_STATS,
-                       "conn=%lu op=%lu BIND anonymous mech=implicit ssf=0\n",
-                       op->o_connid, op->o_opid, 0, 0, 0 );
+                       "%s BIND anonymous mech=implicit ssf=0\n",
+                       op->o_log_prefix, 0, 0, 0, 0 );
        }
        connection2anonymous( op->o_conn );
        if ( op->o_conn->c_sasl_bind_in_progress ) {
                op->o_conn->c_authz_backend = be;
        }
        ldap_pvt_thread_mutex_unlock( &op->o_conn->c_mutex );
-       if ( op->o_dn.bv_val != NULL ) {
-               free( op->o_dn.bv_val );
-               op->o_dn.bv_val = ch_strdup( "" );
+       if ( !BER_BVISNULL( &op->o_dn ) ) {
+               /* NOTE: temporarily wasting few bytes
+                * (until bind is completed), but saving
+                * a couple of ch_free() and ch_strdup("") */ 
+               op->o_dn.bv_val[0] = '\0';
                op->o_dn.bv_len = 0;
        }
-       if ( op->o_ndn.bv_val != NULL ) {
-               free( op->o_ndn.bv_val );
-               op->o_ndn.bv_val = ch_strdup( "" );
+       if ( !BER_BVISNULL( &op->o_ndn ) ) {
+               op->o_ndn.bv_val[0] = '\0';
                op->o_ndn.bv_len = 0;
        }
 
@@ -109,12 +105,7 @@ do_bind(
        tag = ber_scanf( ber, "{imt" /*}*/, &version, &dn, &method );
 
        if ( tag == LBER_ERROR ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, ERR, 
-                       "do_bind: conn %d  ber_scanf failed\n", op->o_connid, 0, 0 );
-#else
                Debug( LDAP_DEBUG_ANY, "bind: ber_scanf failed\n", 0, 0, 0 );
-#endif
                send_ldap_discon( op, rs, LDAP_PROTOCOL_ERROR, "decoding error" );
                rs->sr_err = SLAPD_DISCONNECT;
                goto cleanup;
@@ -153,12 +144,7 @@ do_bind(
        }
 
        if( get_ctrls( op, rs, 1 ) != LDAP_SUCCESS ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, INFO, 
-                       "do_bind: conn %d  get_ctrls failed\n", op->o_connid, 0, 0 );
-#else
                Debug( LDAP_DEBUG_ANY, "do_bind: get_ctrls failed\n", 0, 0, 0 );
-#endif
                goto cleanup;
        } 
 
@@ -169,55 +155,30 @@ do_bind(
        rs->sr_err = dnPrettyNormal( NULL, &dn, &op->o_req_dn, &op->o_req_ndn,
                op->o_tmpmemctx );
        if ( rs->sr_err != LDAP_SUCCESS ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, INFO, 
-                       "do_bind: conn %d  invalid dn (%s)\n", 
-                       op->o_connid, dn.bv_val, 0 );
-#else
                Debug( LDAP_DEBUG_ANY, "bind: invalid dn (%s)\n",
                        dn.bv_val, 0, 0 );
-#endif
                send_ldap_error( op, rs, LDAP_INVALID_DN_SYNTAX, "invalid DN" );
                goto cleanup;
        }
 
        if( op->orb_method == LDAP_AUTH_SASL ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION,     DETAIL1, 
-                       "do_sasl_bind: conn %d  dn (%s) mech %s\n", 
-                       op->o_connid, op->o_req_dn.bv_val, mech.bv_val );
-#else
                Debug( LDAP_DEBUG_TRACE, "do_sasl_bind: dn (%s) mech %s\n",
                        op->o_req_dn.bv_val, mech.bv_val, NULL );
-#endif
 
        } else {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, DETAIL1, 
-                       "do_bind: version=%ld dn=\"%s\" method=%ld\n",
-                       (unsigned long) version, op->o_req_dn.bv_val,
-                       (unsigned long) op->orb_method );
-#else
                Debug( LDAP_DEBUG_TRACE,
                        "do_bind: version=%ld dn=\"%s\" method=%ld\n",
                        (unsigned long) version, op->o_req_dn.bv_val,
                        (unsigned long) op->orb_method );
-#endif
        }
 
-       Statslog( LDAP_DEBUG_STATS, "conn=%lu op=%lu BIND dn=\"%s\" method=%ld\n",
-           op->o_connid, op->o_opid, op->o_req_dn.bv_val,
-               (unsigned long) op->orb_method, 0 );
+       Statslog( LDAP_DEBUG_STATS, "%s BIND dn=\"%s\" method=%ld\n",
+           op->o_log_prefix, op->o_req_dn.bv_val,
+               (unsigned long) op->orb_method, 0, 0 );
 
        if ( version < LDAP_VERSION_MIN || version > LDAP_VERSION_MAX ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, INFO, 
-                       "do_bind: conn %d  unknown version = %ld\n",
-                       op->o_connid, (unsigned long)version, 0 );
-#else
                Debug( LDAP_DEBUG_ANY, "do_bind: unknown version=%ld\n",
                        (unsigned long) version, 0, 0 );
-#endif
                send_ldap_error( op, rs, LDAP_PROTOCOL_ERROR,
                        "requested protocol version not supported" );
                goto cleanup;
@@ -252,11 +213,11 @@ cleanup:
 
        op->o_conn->c_sasl_bindop = NULL;
 
-       if( op->o_req_dn.bv_val != NULL ) {
+       if( !BER_BVISNULL( &op->o_req_dn ) ) {
                slap_sl_free( op->o_req_dn.bv_val, op->o_tmpmemctx );
                BER_BVZERO( &op->o_req_dn );
        }
-       if( op->o_req_ndn.bv_val != NULL ) {
+       if( !BER_BVISNULL( &op->o_req_ndn ) ) {
                slap_sl_free( op->o_req_ndn.bv_val, op->o_tmpmemctx );
                BER_BVZERO( &op->o_req_ndn );
        }
@@ -270,7 +231,7 @@ fe_op_bind( Operation *op, SlapReply *rs )
        struct berval   mech = op->orb_tmp_mech;
 
        /* check for inappropriate controls */
-       if( get_manageDSAit( op ) == SLAP_CRITICAL_CONTROL ) {
+       if( get_manageDSAit( op ) == SLAP_CONTROL_CRITICAL ) {
                send_ldap_error( op, rs,
                        LDAP_UNAVAILABLE_CRITICAL_EXTENSION,
                        "manageDSAit control inappropriate" );
@@ -282,30 +243,18 @@ fe_op_bind( Operation *op, SlapReply *rs )
 
        if ( op->orb_method == LDAP_AUTH_SASL ) {
                if ( op->o_protocol < LDAP_VERSION3 ) {
-#ifdef NEW_LOGGING
-                       LDAP_LOG( OPERATION, INFO, 
-                               "do_bind: conn %d  sasl with LDAPv%ld\n",
-                               op->o_connid, (unsigned long)op->o_protocol, 0 );
-#else
                        Debug( LDAP_DEBUG_ANY, "do_bind: sasl with LDAPv%ld\n",
                                (unsigned long)op->o_protocol, 0, 0 );
-#endif
                        send_ldap_discon( op, rs,
                                LDAP_PROTOCOL_ERROR, "SASL bind requires LDAPv3" );
                        rs->sr_err = SLAPD_DISCONNECT;
                        goto cleanup;
                }
 
-               if( mech.bv_len == 0 ) {
-#ifdef NEW_LOGGING
-                       LDAP_LOG( OPERATION, INFO, 
-                                  "do_bind: conn %d  no SASL mechanism provided\n",
-                                  op->o_connid, 0, 0 );
-#else
+               if( BER_BVISNULL( &mech ) || BER_BVISEMPTY( &mech ) ) {
                        Debug( LDAP_DEBUG_ANY,
                                "do_bind: no sasl mechanism provided\n",
                                0, 0, 0 );
-#endif
                        send_ldap_error( op, rs, LDAP_AUTH_METHOD_NOT_SUPPORTED,
                                "no SASL mechanism provided" );
                        goto cleanup;
@@ -333,7 +282,7 @@ fe_op_bind( Operation *op, SlapReply *rs )
                ldap_pvt_thread_mutex_lock( &op->o_conn->c_mutex );
                if( rs->sr_err == LDAP_SUCCESS ) {
                        ber_dupbv(&op->o_conn->c_dn, &op->orb_edn);
-                       if( op->orb_edn.bv_len != 0 ) {
+                       if( !BER_BVISEMPTY( &op->orb_edn ) ) {
                                /* edn is always normalized already */
                                ber_dupbv( &op->o_conn->c_ndn, &op->o_conn->c_dn );
                        }
@@ -348,7 +297,7 @@ fe_op_bind( Operation *op, SlapReply *rs )
                                op->o_conn->c_ssf = op->orb_ssf;
                        }
 
-                       if( op->o_conn->c_dn.bv_len != 0 ) {
+                       if( !BER_BVISEMPTY( &op->o_conn->c_dn ) ) {
                                ber_len_t max = sockbuf_max_incoming_auth;
                                ber_sockbuf_ctrl( op->o_conn->c_sb,
                                        LBER_SB_OPT_SET_MAX_INCOMING, &max );
@@ -356,30 +305,22 @@ fe_op_bind( Operation *op, SlapReply *rs )
 
                        /* log authorization identity */
                        Statslog( LDAP_DEBUG_STATS,
-                               "conn=%lu op=%lu BIND dn=\"%s\" mech=%s ssf=%d\n",
-                               op->o_connid, op->o_opid,
-                               op->o_conn->c_dn.bv_val ? op->o_conn->c_dn.bv_val : "<empty>",
-                               op->o_conn->c_authmech.bv_val, op->orb_ssf );
+                               "%s BIND dn=\"%s\" mech=%s ssf=%d\n",
+                               op->o_log_prefix,
+                               BER_BVISNULL( &op->o_conn->c_dn ) ? "<empty>" : op->o_conn->c_dn.bv_val,
+                               op->o_conn->c_authmech.bv_val, op->orb_ssf, 0 );
 
-#ifdef NEW_LOGGING
-                       LDAP_LOG( OPERATION, DETAIL1, 
-                               "do_bind: SASL/%s bind: dn=\"%s\" ssf=%d\n",
-                               op->o_conn->c_authmech.bv_val,
-                               op->o_conn->c_dn.bv_val ? op->o_conn->c_dn.bv_val : "<empty>",
-                               op->orb_ssf );
-#else
                        Debug( LDAP_DEBUG_TRACE,
                                "do_bind: SASL/%s bind: dn=\"%s\" ssf=%d\n",
                                op->o_conn->c_authmech.bv_val,
-                               op->o_conn->c_dn.bv_val ? op->o_conn->c_dn.bv_val : "<empty>",
+                               BER_BVISNULL( &op->o_conn->c_dn ) ? "<empty>" : op->o_conn->c_dn.bv_val,
                                op->orb_ssf );
-#endif
 
                } else if ( rs->sr_err == LDAP_SASL_BIND_IN_PROGRESS ) {
                        op->o_conn->c_sasl_bind_in_progress = 1;
 
                } else {
-                       if ( op->o_conn->c_sasl_bind_mech.bv_val ) {
+                       if ( !BER_BVISNULL( &op->o_conn->c_sasl_bind_mech ) ) {
                                free( op->o_conn->c_sasl_bind_mech.bv_val );
                                BER_BVZERO( &op->o_conn->c_sasl_bind_mech );
                        }
@@ -415,8 +356,8 @@ fe_op_bind( Operation *op, SlapReply *rs )
                /* Not SASL, cancel any in-progress bind */
                ldap_pvt_thread_mutex_lock( &op->o_conn->c_mutex );
 
-               if ( op->o_conn->c_sasl_bind_mech.bv_val != NULL ) {
-                       free(op->o_conn->c_sasl_bind_mech.bv_val);
+               if ( !BER_BVISNULL( &op->o_conn->c_sasl_bind_mech ) ) {
+                       free( op->o_conn->c_sasl_bind_mech.bv_val );
                        BER_BVZERO( &op->o_conn->c_sasl_bind_mech );
                }
                op->o_conn->c_sasl_bind_in_progress = 0;
@@ -426,18 +367,18 @@ fe_op_bind( Operation *op, SlapReply *rs )
        }
 
        if ( op->orb_method == LDAP_AUTH_SIMPLE ) {
-               ber_str2bv( "SIMPLE", sizeof("SIMPLE")-1, 0, &mech );
+               BER_BVSTR( &mech, "SIMPLE" );
                /* accept "anonymous" binds */
-               if ( op->orb_cred.bv_len == 0 || op->o_req_ndn.bv_len == 0 ) {
+               if ( BER_BVISEMPTY( &op->orb_cred ) || BER_BVISEMPTY( &op->o_req_ndn ) ) {
                        rs->sr_err = LDAP_SUCCESS;
 
-                       if( op->orb_cred.bv_len &&
+                       if( !BER_BVISEMPTY( &op->orb_cred ) &&
                                !( global_allows & SLAP_ALLOW_BIND_ANON_CRED ))
                        {
                                /* cred is not empty, disallow */
                                rs->sr_err = LDAP_INVALID_CREDENTIALS;
 
-                       } else if ( op->o_req_ndn.bv_len &&
+                       } else if ( !BER_BVISEMPTY( &op->o_req_ndn ) &&
                                !( global_allows & SLAP_ALLOW_BIND_ANON_DN ))
                        {
                                /* DN is not empty, disallow */
@@ -459,14 +400,8 @@ fe_op_bind( Operation *op, SlapReply *rs )
                         * just need to send success
                         */
                        send_ldap_result( op, rs );
-#ifdef NEW_LOGGING
-                       LDAP_LOG( OPERATION, DETAIL1, 
-                               "do_bind: conn %d  v%d anonymous bind\n",
-                               op->o_connid, op->o_protocol, 0 );
-#else
                        Debug( LDAP_DEBUG_TRACE, "do_bind: v%d anonymous bind\n",
                                op->o_protocol, 0, 0 );
-#endif
                        goto cleanup;
 
                } else if ( global_disallows & SLAP_DISALLOW_BIND_SIMPLE ) {
@@ -475,15 +410,9 @@ fe_op_bind( Operation *op, SlapReply *rs )
                        rs->sr_text = "unwilling to perform simple authentication";
 
                        send_ldap_result( op, rs );
-#ifdef NEW_LOGGING
-                       LDAP_LOG( OPERATION, INFO, 
-                               "do_bind: conn %d  v%d simple bind(%s) disallowed\n",
-                               op->o_connid, op->o_protocol, op->o_req_ndn.bv_val );
-#else
                        Debug( LDAP_DEBUG_TRACE,
                                "do_bind: v%d simple bind(%s) disallowed\n",
                                op->o_protocol, op->o_req_ndn.bv_val, 0 );
-#endif
                        goto cleanup;
                }
 
@@ -496,33 +425,21 @@ fe_op_bind( Operation *op, SlapReply *rs )
 
                        send_ldap_result( op, rs );
 
-#ifdef NEW_LOGGING
-                       LDAP_LOG( OPERATION, DETAIL1, 
-                               "do_bind: conn %d  v%d Kerberos V4 (step 1) bind refused\n",
-                               op->o_connid, op->o_protocol, 0 );
-#else
                        Debug( LDAP_DEBUG_TRACE,
                                "do_bind: v%d Kerberos V4 (step 1) bind refused\n",
                                op->o_protocol, 0, 0 );
-#endif
                        goto cleanup;
                }
-               ber_str2bv( "KRBV4", sizeof("KRBV4")-1, 0, &mech );
+               BER_BVSTR( &mech, "KRBV4" );
 
        } else if ( op->orb_method == LDAP_AUTH_KRBV42 ) {
                rs->sr_err = LDAP_AUTH_METHOD_NOT_SUPPORTED;
                rs->sr_text = "Kerberos V4 (step 2) bind not supported";
                send_ldap_result( op, rs );
 
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, DETAIL1, 
-                       "do_bind: conn %d  v%d Kerberos V4 (step 2) bind refused\n",
-                       op->o_connid, op->o_protocol, 0 );
-#else
                Debug( LDAP_DEBUG_TRACE,
                        "do_bind: v%d Kerberos V4 (step 2) bind refused\n",
                        op->o_protocol, 0, 0 );
-#endif
                goto cleanup;
 #endif
 
@@ -531,15 +448,9 @@ fe_op_bind( Operation *op, SlapReply *rs )
                rs->sr_text = "unknown authentication method";
 
                send_ldap_result( op, rs );
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, INFO, 
-                       "do_bind: conn %ld  v%d unknown authentication method (%ld)\n",
-                       op->o_connid, op->o_protocol, op->orb_method );
-#else
                Debug( LDAP_DEBUG_TRACE,
                        "do_bind: v%d unknown authentication method (%ld)\n",
                        op->o_protocol, op->orb_method, 0 );
-#endif
                goto cleanup;
        }
 
@@ -550,22 +461,12 @@ fe_op_bind( Operation *op, SlapReply *rs )
         */
 
        if ( (op->o_bd = select_backend( &op->o_req_ndn, 0, 0 )) == NULL ) {
-               if ( default_referral ) {
-                       rs->sr_ref = referral_rewrite( default_referral,
-                               NULL, &op->o_req_dn, LDAP_SCOPE_DEFAULT );
-                       if (!rs->sr_ref) rs->sr_ref = default_referral;
-
-                       rs->sr_err = LDAP_REFERRAL;
-                       send_ldap_result( op, rs );
-
-                       if (rs->sr_ref != default_referral) ber_bvarray_free( rs->sr_ref );
-
-               } else {
-                       /* noSuchObject is not allowed to be returned by bind */
-                       rs->sr_err = LDAP_INVALID_CREDENTIALS;
-                       send_ldap_result( op, rs );
-               }
-
+               /* don't return referral for bind requests */
+               /* noSuchObject is not allowed to be returned by bind */
+               rs->sr_err = LDAP_INVALID_CREDENTIALS;
+               op->o_bd = frontendDB;
+               send_ldap_result( op, rs );
+               op->o_bd = NULL;
                goto cleanup;
        }
 
@@ -587,15 +488,9 @@ fe_op_bind( Operation *op, SlapReply *rs )
 
                rc = slapi_int_call_plugins( op->o_bd, SLAPI_PLUGIN_PRE_BIND_FN, pb );
 
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, INFO,
-                       "do_bind: Bind preoperation plugin returned %d\n",
-                       rs->sr_err, 0, 0);
-#else
                Debug(LDAP_DEBUG_TRACE,
                        "do_bind: Bind preoperation plugin returned %d.\n",
                        rs->sr_err, 0, 0);
-#endif
 
                switch ( rc ) {
                case SLAPI_BIND_SUCCESS:
@@ -639,18 +534,18 @@ fe_op_bind( Operation *op, SlapReply *rs )
                                BER_BVZERO( &op->o_req_dn );
                                op->o_tmpfree( op->o_req_ndn.bv_val, op->o_tmpmemctx );
                                BER_BVZERO( &op->o_req_ndn );
-                               if ( op->o_conn->c_dn.bv_len != 0 ) {
+                               if ( !BER_BVISEMPTY( &op->o_conn->c_dn ) ) {
                                        ber_len_t max = sockbuf_max_incoming_auth;
                                        ber_sockbuf_ctrl( op->o_conn->c_sb,
                                                LBER_SB_OPT_SET_MAX_INCOMING, &max );
                                }
                                /* log authorization identity */
                                Statslog( LDAP_DEBUG_STATS,
-                                       "conn=%lu op=%lu BIND dn=\"%s\" mech=%s (SLAPI) ssf=0\n",
-                                       op->o_connid, op->o_opid,
-                                       op->o_conn->c_dn.bv_val
-                                               ? op->o_conn->c_dn.bv_val : "<empty>",
-                                       mech.bv_val, 0 );
+                                       "%s BIND dn=\"%s\" mech=%s (SLAPI) ssf=0\n",
+                                       op->o_log_prefix,
+                                       BER_BVISNULL( &op->o_conn->c_dn )
+                                               ? "<empty>" : op->o_conn->c_dn.bv_val,
+                                       mech.bv_val, 0, 0 );
                                ldap_pvt_thread_mutex_unlock( &op->o_conn->c_mutex );
                        }
                        goto cleanup;
@@ -670,7 +565,7 @@ fe_op_bind( Operation *op, SlapReply *rs )
                        }
 
                        /* be_bind returns regular/global edn */
-                       if( op->orb_edn.bv_len ) {
+                       if( !BER_BVISEMPTY( &op->orb_edn ) ) {
                                op->o_conn->c_dn = op->orb_edn;
                        } else {
                                ber_dupbv(&op->o_conn->c_dn, &op->o_req_dn);
@@ -678,7 +573,7 @@ fe_op_bind( Operation *op, SlapReply *rs )
 
                        ber_dupbv( &op->o_conn->c_ndn, &op->o_req_ndn );
 
-                       if( op->o_conn->c_dn.bv_len != 0 ) {
+                       if( !BER_BVISEMPTY( &op->o_conn->c_dn ) ) {
                                ber_len_t max = sockbuf_max_incoming_auth;
                                ber_sockbuf_ctrl( op->o_conn->c_sb,
                                        LBER_SB_OPT_SET_MAX_INCOMING, &max );
@@ -686,27 +581,22 @@ fe_op_bind( Operation *op, SlapReply *rs )
 
                        /* log authorization identity */
                        Statslog( LDAP_DEBUG_STATS,
-                               "conn=%lu op=%lu BIND dn=\"%s\" mech=%s ssf=0\n",
-                               op->o_connid, op->o_opid,
-                               op->o_conn->c_dn.bv_val, mech.bv_val, 0 );
-
-#ifdef NEW_LOGGING
-                       LDAP_LOG( OPERATION, DETAIL1, 
-                               "do_bind: v%d bind: \"%s\" to \"%s\" \n",
-                               op->o_protocol, op->o_conn->c_dn.bv_val, op->o_conn->c_dn.bv_val );
-#else
+                               "%s BIND dn=\"%s\" mech=%s ssf=0\n",
+                               op->o_log_prefix,
+                               op->o_conn->c_dn.bv_val, mech.bv_val, 0, 0 );
+
                        Debug( LDAP_DEBUG_TRACE,
                                "do_bind: v%d bind: \"%s\" to \"%s\"\n",
                                op->o_protocol, op->o_req_dn.bv_val, op->o_conn->c_dn.bv_val );
-#endif
 
                        ldap_pvt_thread_mutex_unlock( &op->o_conn->c_mutex );
 
                        /* send this here to avoid a race condition */
                        send_ldap_result( op, rs );
 
-               } else if (op->orb_edn.bv_val != NULL) {
+               } else if ( !BER_BVISNULL( &op->orb_edn ) ) {
                        free( op->orb_edn.bv_val );
+                       BER_BVZERO( &op->orb_edn );
                }
 
        } else {
@@ -718,15 +608,9 @@ fe_op_bind( Operation *op, SlapReply *rs )
        if ( pb != NULL &&
                slapi_int_call_plugins( op->o_bd, SLAPI_PLUGIN_POST_BIND_FN, pb ) < 0 )
        {
-#ifdef NEW_LOGGING
-               LDAP_LOG( OPERATION, INFO,
-                       "do_bind: Bind postoperation plugins failed\n",
-                       0, 0, 0);
-#else
                Debug(LDAP_DEBUG_TRACE,
                        "do_bind: Bind postoperation plugins failed.\n",
                        0, 0, 0);
-#endif
        }
 #endif /* LDAP_SLAPI */