X-Git-Url: https://git.sur5r.net/?a=blobdiff_plain;f=servers%2Fslapd%2Fsasl.c;h=60e9db716dde422360101cb4d508e018359c8f0f;hb=1586a68219cf85e9caaafcf2259c9bc249358077;hp=527e95cda90308bfdbe1eb1a1ad30aa886553931;hpb=10811b6f51f991980e124a43beb1eb4ebe4dce53;p=openldap diff --git a/servers/slapd/sasl.c b/servers/slapd/sasl.c index 527e95cda9..60e9db716d 100644 --- a/servers/slapd/sasl.c +++ b/servers/slapd/sasl.c @@ -1,7 +1,16 @@ /* $OpenLDAP$ */ -/* - * Copyright 1998-2003 The OpenLDAP Foundation, All Rights Reserved. - * COPYING RESTRICTIONS APPLY, see COPYRIGHT file +/* This work is part of OpenLDAP Software . + * + * Copyright 1998-2003 The OpenLDAP Foundation. + * All rights reserved. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted only as authorized by the OpenLDAP + * Public License. + * + * A copy of this license is available in the file LICENSE in the + * top-level directory of the distribution or, alternatively, at + * . */ #include "portable.h" @@ -25,19 +34,38 @@ # endif # if SASL_VERSION_MAJOR >= 2 +# ifdef HAVE_SASL_SASL_H # include +# else +# include +# endif # define SASL_CONST const # else # define SASL_CONST # endif +#define SASL_VERSION_FULL ((SASL_VERSION_MAJOR << 16) |\ + (SASL_VERSION_MINOR << 8) | SASL_VERSION_STEP) + static sasl_security_properties_t sasl_secprops; -#endif /* HAVE_CYRUS_SASL */ +#elif defined( SLAP_BUILTIN_SASL ) +/* + * built-in SASL implementation + * only supports EXTERNAL + */ +typedef struct sasl_ctx { + slap_ssf_t sc_external_ssf; + struct berval sc_external_id; +} SASL_CTX; + +#endif #include "ldap_pvt.h" #include "lber_pvt.h" #include +static struct berval ext_bv = BER_BVC( "EXTERNAL" ); + int slap_sasl_config( int cargc, char **cargv, char *line, const char *fname, int lineno ) { @@ -293,6 +321,7 @@ static const char *slap_propnames[] = { "*slapConn", "*authcDN", "*authzDN", NULL }; static Filter generic_filter = { LDAP_FILTER_PRESENT }; +static struct berval generic_filterstr = BER_BVC("(objectclass=*)"); #define PROP_CONN 0 #define PROP_AUTHC 1 @@ -370,7 +399,7 @@ slap_auxprop_lookup( unsigned ulen) { Operation op = {0}; - int rc, i, doit=0; + int i, doit = 0; Connection *conn = NULL; lookup_info sl; @@ -422,7 +451,7 @@ slap_auxprop_lookup( } if (doit) { - slap_callback cb = { sasl_ap_lookup, NULL }; + slap_callback cb = { NULL, sasl_ap_lookup, NULL, NULL }; cb.sc_private = &sl; @@ -440,21 +469,122 @@ slap_auxprop_lookup( 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; -#ifdef LDAP_SLAPI - op.o_pb = conn->c_sasl_bindop->o_pb; -#endif op.o_conn = conn; op.o_connid = conn->c_connid; + op.o_req_dn = op.o_req_ndn; op.ors_scope = LDAP_SCOPE_BASE; op.ors_deref = LDAP_DEREF_NEVER; op.ors_slimit = 1; op.ors_filter = &generic_filter; + op.ors_filterstr = generic_filterstr; op.o_bd->be_search( &op, &rs ); } } } +#if SASL_VERSION_FULL >= 0x020110 +static int +slap_auxprop_store( + void *glob_context, + sasl_server_params_t *sparams, + struct propctx *prctx, + const char *user, + unsigned ulen) +{ + Operation op = {0}; + SlapReply rs = {REP_RESULT}; + int rc, i, j; + Connection *conn = NULL; + const struct propval *pr; + Modifications *modlist = NULL, **modtail = &modlist, *mod; + slap_callback cb = { NULL, slap_null_cb, NULL, NULL }; + char textbuf[SLAP_TEXT_BUFLEN]; + const char *text; + size_t textlen = sizeof(textbuf); + + /* just checking if we are enabled */ + if (!prctx) return SASL_OK; + + if (!sparams || !user) return SASL_BADPARAM; + + pr = sparams->utils->prop_get( sparams->propctx ); + + /* Find our DN and conn first */ + for( i = 0; pr[i].name; i++ ) { + if ( pr[i].name[0] == '*' ) { + if ( !strcmp( pr[i].name, slap_propnames[PROP_CONN] ) ) { + if ( pr[i].values && pr[i].values[0] ) + AC_MEMCPY( &conn, pr[i].values[0], sizeof( conn ) ); + continue; + } + if ( !strcmp( pr[i].name, slap_propnames[PROP_AUTHC] ) ) { + if ( pr[i].values && pr[i].values[0] ) { + AC_MEMCPY( &op.o_req_ndn, pr[i].values[0], sizeof( struct berval ) ); + } + } + } + } + if (!conn || !op.o_req_ndn.bv_val) return SASL_BADPARAM; + + op.o_bd = select_backend( &op.o_req_ndn, 0, 1 ); + + if ( !op.o_bd || !op.o_bd->be_modify ) return SASL_FAIL; + + pr = sparams->utils->prop_get( prctx ); + if (!pr) return SASL_BADPARAM; + + for (i=0; pr[i].name; i++); + if (!i) return SASL_BADPARAM; + + for (i=0; pr[i].name; i++) { + mod = (Modifications *)ch_malloc( sizeof(Modifications) ); + mod->sml_op = LDAP_MOD_REPLACE; + ber_str2bv( pr[i].name, 0, 0, &mod->sml_type ); + mod->sml_values = (struct berval *)ch_malloc( (pr[i].nvalues + 1) * + sizeof(struct berval)); + for (j=0; jsml_values[j]); + } + mod->sml_values[j].bv_val = NULL; + mod->sml_values[j].bv_len = 0; + mod->sml_nvalues = NULL; + mod->sml_desc = NULL; + *modtail = mod; + modtail = &mod->sml_next; + } + *modtail = NULL; + + rc = slap_mods_check( modlist, 0, &text, textbuf, textlen, NULL ); + + if ( rc == LDAP_SUCCESS ) { + rc = slap_mods_opattrs( &op, modlist, modtail, &text, textbuf, + textlen ); + } + + if ( rc == LDAP_SUCCESS ) { + op.o_tag = LDAP_REQ_MODIFY; + op.o_protocol = LDAP_VERSION3; + op.o_ndn = op.o_req_ndn; + op.o_callback = &cb; + op.o_time = slap_get_time(); + 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.o_req_dn = op.o_req_ndn; + op.orm_modlist = modlist; + + rc = op.o_bd->be_modify( &op, &rs ); + } + slap_mods_free( modlist ); + return rc ? SASL_FAIL : SASL_OK; +} +#endif /* SASL_VERSION_FULL >= 2.1.16 */ + static sasl_auxprop_plug_t slap_auxprop_plugin = { 0, /* Features */ 0, /* spare */ @@ -462,7 +592,12 @@ static sasl_auxprop_plug_t slap_auxprop_plugin = { NULL, /* auxprop_free */ slap_auxprop_lookup, "slapd", /* name */ - NULL /* spare */ +#if SASL_VERSION_FULL >= 0x020110 + slap_auxprop_store /* the declaration of this member changed + * in cyrus SASL from 2.1.15 to 2.1.16 */ +#else + NULL +#endif }; static int @@ -551,7 +686,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 = { sasl_cb_checkpass, NULL }; + slap_callback cb = { NULL, sasl_cb_checkpass, NULL, NULL }; SlapReply rs = {REP_RESULT}; ci.cred.bv_val = (char *)pass; @@ -568,15 +703,14 @@ slap_sasl_checkpass( 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; -#ifdef LDAP_SLAPI - op.o_pb = conn->c_sasl_bindop->o_pb; -#endif op.o_conn = conn; op.o_connid = conn->c_connid; + op.o_req_dn = op.o_req_ndn; op.ors_scope = LDAP_SCOPE_BASE; op.ors_deref = LDAP_DEREF_NEVER; op.ors_slimit = 1; op.ors_filter = &generic_filter; + op.ors_filterstr = generic_filterstr; op.o_bd->be_search( &op, &rs ); } @@ -594,7 +728,7 @@ slap_sasl_checkpass( * auxiliary property, so that we can refer to it in sasl_authorize * without interfering with anything else. Also, the SASL username * buffer is constrained to 256 characters, and our DNs could be - * much longer (totally arbitrary length)... + * much longer (SLAP_LDAPDN_MAXLEN, currently set to 8192) */ static int slap_sasl_canonicalize( @@ -735,7 +869,7 @@ slap_sasl_authorize( "slap_sasl_authorize: conn %d authcid=\"%s\" authzid=\"%s\"\n", conn ? conn->c_connid : -1, auth_identity, requested_user); #else - Debug( LDAP_DEBUG_ARGS, "SASL Authorize [conn=%ld]: " + Debug( LDAP_DEBUG_ARGS, "SASL proxy authorize [conn=%ld]: " "authcid=\"%s\" authzid=\"%s\"\n", conn ? conn->c_connid : -1, auth_identity, requested_user ); #endif @@ -763,11 +897,12 @@ slap_sasl_authorize( if ( rc != LDAP_SUCCESS ) { #ifdef NEW_LOGGING LDAP_LOG( TRANSPORT, INFO, - "slap_sasl_authorize: conn %ld authorization disallowed (%d)\n", + "slap_sasl_authorize: conn %ld " + "proxy authorization disallowed (%d)\n", (long)(conn ? conn->c_connid : -1), rc, 0 ); #else - Debug( LDAP_DEBUG_TRACE, "SASL Authorize [conn=%ld]: " - " authorization disallowed (%d)\n", + Debug( LDAP_DEBUG_TRACE, "SASL Proxy Authorize [conn=%ld]: " + "proxy authorization disallowed (%d)\n", (long) (conn ? conn->c_connid : -1), rc, 0 ); #endif @@ -787,11 +922,11 @@ ok: #ifdef NEW_LOGGING LDAP_LOG( TRANSPORT, ENTRY, - "slap_sasl_authorize: conn %d authorization allowed\n", + "slap_sasl_authorize: conn %d proxy authorization allowed\n", (long)(conn ? conn->c_connid : -1), 0, 0 ); #else Debug( LDAP_DEBUG_TRACE, "SASL Authorize [conn=%ld]: " - " authorization allowed\n", + " proxy authorization allowed\n", (long) (conn ? conn->c_connid : -1), 0, 0 ); #endif return SASL_OK; @@ -878,11 +1013,12 @@ slap_sasl_authorize( if( rc ) { #ifdef NEW_LOGGING LDAP_LOG( TRANSPORT, INFO, - "slap_sasl_authorize: conn %ld authorization disallowed (%d)\n", + "slap_sasl_authorize: conn %ld " + "proxy authorization disallowed (%d)\n", (long)(conn ? conn->c_connid : -1), rc, 0 ); #else Debug( LDAP_DEBUG_TRACE, "SASL Authorize [conn=%ld]: " - " authorization disallowed (%d)\n", + "proxy authorization disallowed (%d)\n", (long) (conn ? conn->c_connid : -1), rc, 0 ); #endif @@ -895,7 +1031,7 @@ slap_sasl_authorize( ok: #ifdef NEW_LOGGING LDAP_LOG( TRANSPORT, RESULTS, - "slap_sasl_authorize: conn %d authorization allowed\n", + "slap_sasl_authorize: conn %d proxy authorization allowed\n", (long)(conn ? conn->c_connid : -1 ), 0, 0 ); #else Debug( LDAP_DEBUG_TRACE, "SASL Authorize [conn=%ld]: " @@ -973,9 +1109,10 @@ int slap_sasl_init( void ) sasl_version( NULL, &rc ); if ( ((rc >> 16) != ((SASL_VERSION_MAJOR << 8)|SASL_VERSION_MINOR)) || - (rc & 0xffff) < SASL_VERSION_STEP) { + (rc & 0xffff) < SASL_VERSION_STEP) + { char version[sizeof("xxx.xxx.xxxxx")]; - sprintf( version, "%d.%d.%d", rc >> 24, rc >> 16 & 0xff, + sprintf( version, "%u.%d.%d", (unsigned)rc >> 24, (rc >> 16) & 0xff, rc & 0xffff ); #ifdef NEW_LOGGING LDAP_LOG( TRANSPORT, INFO, @@ -1038,7 +1175,6 @@ int slap_sasl_init( void ) 0, 0, 0 ); #endif - /* default security properties */ memset( &sasl_secprops, '\0', sizeof(sasl_secprops) ); sasl_secprops.max_ssf = INT_MAX; @@ -1062,15 +1198,17 @@ int slap_sasl_destroy( void ) int slap_sasl_open( Connection *conn, int reopen ) { - int cb, sc = LDAP_SUCCESS; -#if SASL_VERSION_MAJOR >= 2 - char *ipremoteport = NULL, *iplocalport = NULL; -#endif - + int sc = LDAP_SUCCESS; #ifdef HAVE_CYRUS_SASL + int cb; + sasl_conn_t *ctx = NULL; sasl_callback_t *session_callbacks; +#if SASL_VERSION_MAJOR >= 2 + char *ipremoteport = NULL, *iplocalport = NULL; +#endif + assert( conn->c_sasl_authctx == NULL ); if ( !reopen ) { @@ -1200,14 +1338,26 @@ int slap_sasl_open( Connection *conn, int reopen ) } sc = slap_sasl_err2ldap( sc ); + +#elif defined(SLAP_BUILTIN_SASL) + /* built-in SASL implementation */ + SASL_CTX *ctx = (SASL_CTX *) SLAP_MALLOC(sizeof(SASL_CTX)); + if( ctx == NULL ) return -1; + + ctx->sc_external_ssf = 0; + ctx->sc_external_id.bv_len = 0; + ctx->sc_external_id.bv_val = NULL; + + conn->c_sasl_authctx = ctx; #endif + return sc; } int slap_sasl_external( Connection *conn, slap_ssf_t ssf, - const char *auth_id ) + struct berval *auth_id ) { #if SASL_VERSION_MAJOR >= 2 int sc; @@ -1223,7 +1373,8 @@ int slap_sasl_external( return LDAP_OTHER; } - sc = sasl_setprop( ctx, SASL_AUTH_EXTERNAL, auth_id ); + sc = sasl_setprop( ctx, SASL_AUTH_EXTERNAL, + auth_id ? auth_id->bv_val : NULL ); if ( sc != SASL_OK ) { return LDAP_OTHER; @@ -1240,7 +1391,7 @@ int slap_sasl_external( memset( &extprops, '\0', sizeof(extprops) ); extprops.ssf = ssf; - extprops.auth_id = (char *) auth_id; + extprops.auth_id = auth_id ? auth_id->bv_val : NULL; sc = sasl_setprop( ctx, SASL_SSF_EXTERNAL, (void *) &extprops ); @@ -1248,6 +1399,20 @@ int slap_sasl_external( if ( sc != SASL_OK ) { return LDAP_OTHER; } +#elif defined(SLAP_BUILTIN_SASL) + /* built-in SASL implementation */ + SASL_CTX *ctx = conn->c_sasl_authctx; + if ( ctx == NULL ) return LDAP_UNAVAILABLE; + + ctx->sc_external_ssf = ssf; + if( auth_id ) { + ctx->sc_external_id = *auth_id; + auth_id->bv_len = 0; + auth_id->bv_val = NULL; + } else { + ctx->sc_external_id.bv_len = 0; + ctx->sc_external_id.bv_val = NULL; + } #endif return LDAP_SUCCESS; @@ -1293,6 +1458,13 @@ char ** slap_sasl_mechs( Connection *conn ) ch_free( mechstr ); #endif } +#elif defined(SLAP_BUILTIN_SASL) + /* builtin SASL implementation */ + SASL_CTX *ctx = conn->c_sasl_authctx; + if ( ctx != NULL && ctx->sc_external_id.bv_val ) { + /* should check ssf */ + mechs = ldap_str2charray( "EXTERNAL", "," ); + } #endif return mechs; @@ -1306,7 +1478,9 @@ int slap_sasl_close( Connection *conn ) if( ctx != NULL ) { sasl_dispose( &ctx ); } - if ( conn->c_sasl_sockctx && conn->c_sasl_authctx != conn->c_sasl_sockctx ) { + if ( conn->c_sasl_sockctx && + conn->c_sasl_authctx != conn->c_sasl_sockctx ) + { ctx = conn->c_sasl_sockctx; sasl_dispose( &ctx ); } @@ -1317,6 +1491,17 @@ int slap_sasl_close( Connection *conn ) free( conn->c_sasl_extra ); conn->c_sasl_extra = NULL; + +#elif defined(SLAP_BUILTIN_SASL) + SASL_CTX *ctx = conn->c_sasl_authctx; + if( ctx ) { + if( ctx->sc_external_id.bv_val ) { + free( ctx->sc_external_id.bv_val ); + ctx->sc_external_id.bv_val = NULL; + } + free( ctx ); + conn->c_sasl_authctx = NULL; + } #endif return LDAP_SUCCESS; @@ -1346,7 +1531,6 @@ int slap_sasl_bind( Operation *op, SlapReply *rs ) op->orb_cred.bv_len ); #endif - if( ctx == NULL ) { send_ldap_error( op, rs, LDAP_UNAVAILABLE, "SASL unavailable on this session" ); @@ -1475,9 +1659,35 @@ int slap_sasl_bind( Operation *op, SlapReply *rs ) Debug(LDAP_DEBUG_TRACE, "<== slap_sasl_bind: rc=%d\n", rs->sr_err, 0, 0); #endif +#elif defined(SLAP_BUILTIN_SASL) + /* built-in SASL implementation */ + SASL_CTX *ctx = op->o_conn->c_sasl_authctx; + + if ( ctx == NULL ) { + send_ldap_error( op, rs, LDAP_OTHER, + "Internal SASL Error" ); + + } else if ( bvmatch( &ext_bv, &op->o_conn->c_sasl_bind_mech ) ) { + /* EXTERNAL */ + + if( op->orb_cred.bv_len ) { + rs->sr_text = "proxy authorization not support"; + rs->sr_err = LDAP_UNWILLING_TO_PERFORM; + send_ldap_result( op, rs ); + } else { + op->orb_edn = ctx->sc_external_id; + rs->sr_err = LDAP_SUCCESS; + rs->sr_sasldata = NULL; + send_ldap_sasl( op, rs ); + } + + } else { + send_ldap_error( op, rs, LDAP_AUTH_METHOD_NOT_SUPPORTED, + "requested SASL mechanism not supported" ); + } #else - send_ldap_error( op, rs, LDAP_UNAVAILABLE, + send_ldap_error( op, rs, LDAP_AUTH_METHOD_NOT_SUPPORTED, "SASL not supported" ); #endif @@ -1573,32 +1783,29 @@ done: #endif /* HAVE_CYRUS_SASL */ /* Take any sort of identity string and return a DN with the "dn:" prefix. The - string returned in *dn is in its own allocated memory, and must be free'd - by the calling process. - -Mark Adamson, Carnegie Mellon - - The "dn:" prefix is no longer used anywhere inside slapd. It is only used - on strings passed in directly from SASL. - -Howard Chu, Symas Corp. -*/ + * string returned in *dn is in its own allocated memory, and must be free'd + * by the calling process. -Mark Adamson, Carnegie Mellon + * + * The "dn:" prefix is no longer used anywhere inside slapd. It is only used + * on strings passed in directly from SASL. -Howard Chu, Symas Corp. + */ #define SET_NONE 0 #define SET_DN 1 #define SET_U 2 -static struct berval ext_bv = BER_BVC( "EXTERNAL" ); - int slap_sasl_getdn( Connection *conn, Operation *op, char *id, int len, char *user_realm, struct berval *dn, int flags ) { - char *c1; int rc, is_dn = SET_NONE, do_norm = 1; - struct berval dn2; + struct berval dn2, *mech; + + assert( conn ); #ifdef NEW_LOGGING LDAP_LOG( TRANSPORT, ENTRY, "slap_sasl_getdn: conn %d id=%s [len=%d]\n", - conn ? conn->c_connid : -1, id ? (*id ? id : "") : "NULL", len ); + conn->c_connid, id ? (*id ? id : "") : "NULL", len ); #else Debug( LDAP_DEBUG_ARGS, "slap_sasl_getdn: id=%s [len=%d]\n", id ? ( *id ? id : "" ) : "NULL", len, 0 ); @@ -1623,15 +1830,19 @@ int slap_sasl_getdn( Connection *conn, Operation *op, char *id, int len, len = 0; } + if ( conn->c_sasl_bind_mech.bv_len ) { + mech = &conn->c_sasl_bind_mech; + } else { + mech = &conn->c_authmech; + } + /* An authcID needs to be converted to authzID form. Set the * values directly into *dn; they will be normalized later. (and * normalizing always makes a new copy.) An ID from a TLS certificate * is already normalized, so copy it and skip normalization. */ if( flags & SLAP_GETDN_AUTHCID ) { - if( conn->c_sasl_bind_mech.bv_len == ext_bv.bv_len && - strcasecmp( ext_bv.bv_val, conn->c_sasl_bind_mech.bv_val ) == 0 ) - { + if( bvmatch( mech, &ext_bv )) { /* EXTERNAL DNs are already normalized */ do_norm = 0; is_dn = SET_DN; @@ -1665,23 +1876,45 @@ int slap_sasl_getdn( Connection *conn, Operation *op, char *id, int len, /* Username strings */ if( is_dn == SET_U ) { - char *p, *realm; + char *p; + struct berval realm = { 0, NULL }, c1 = *dn; + len = dn->bv_len + sizeof("uid=")-1 + sizeof(",cn=auth")-1; +#if 0 /* username may have embedded realm name */ - if( ( realm = strchr( dn->bv_val, '@') ) ) { - *realm++ = '\0'; - len += sizeof(",cn=")-2; - } else if( user_realm && *user_realm ) { - len += strlen( user_realm ) + sizeof(",cn=")-1; + /* FIXME: + * userids can legally have embedded '@' chars; + * the realm should be set by those mechanisms + * that support it by means of the user_realm + * variable + */ + if( ( realm.bv_val = strrchr( dn->bv_val, '@') ) ) { + char *r = realm.bv_val; + + realm.bv_val++; + realm.bv_len = dn->bv_len - ( realm.bv_val - dn->bv_val ); + len += sizeof( ",cn=" ) - 2; + c1.bv_len -= realm.bv_len + 1; + + if ( strchr( dn->bv_val, '@') == r ) { + /* FIXME: ambiguity, is it the realm + * or something else? */ + } + + } else +#endif + if( user_realm && *user_realm ) { + realm.bv_val = user_realm; + realm.bv_len = strlen( user_realm ); + len += realm.bv_len + sizeof(",cn=") - 1; } - if( conn->c_sasl_bind_mech.bv_len ) { - len += conn->c_sasl_bind_mech.bv_len + sizeof(",cn=")-1; + if( mech->bv_len ) { + len += mech->bv_len + sizeof(",cn=")-1; } /* Build the new dn */ - c1 = dn->bv_val; dn->bv_val = sl_malloc( len+1, op->o_tmpmemctx ); if( dn->bv_val == NULL ) { #ifdef NEW_LOGGING @@ -1694,21 +1927,16 @@ int slap_sasl_getdn( Connection *conn, Operation *op, char *id, int len, return LDAP_OTHER; } p = lutil_strcopy( dn->bv_val, "uid=" ); - p = lutil_strncopy( p, c1, dn->bv_len ); + p = lutil_strncopy( p, c1.bv_val, c1.bv_len ); - if( realm ) { - int rlen = dn->bv_len - ( realm - c1 ); - p = lutil_strcopy( p, ",cn=" ); - p = lutil_strncopy( p, realm, rlen ); - realm[-1] = '@'; - } else if( user_realm && *user_realm ) { + if( realm.bv_len ) { p = lutil_strcopy( p, ",cn=" ); - p = lutil_strcopy( p, user_realm ); + p = lutil_strncopy( p, realm.bv_val, realm.bv_len ); } - if( conn->c_sasl_bind_mech.bv_len ) { + if( mech->bv_len ) { p = lutil_strcopy( p, ",cn=" ); - p = lutil_strcopy( p, conn->c_sasl_bind_mech.bv_val ); + p = lutil_strcopy( p, mech->bv_val ); } p = lutil_strcopy( p, ",cn=auth" ); dn->bv_len = p - dn->bv_val; @@ -1717,7 +1945,7 @@ int slap_sasl_getdn( Connection *conn, Operation *op, char *id, int len, LDAP_LOG( TRANSPORT, ENTRY, "slap_sasl_getdn: u:id converted to %s.\n", dn->bv_val, 0, 0 ); #else - Debug( LDAP_DEBUG_TRACE, "getdn: u:id converted to %s\n", dn->bv_val,0,0 ); + Debug( LDAP_DEBUG_TRACE, "slap_sasl_getdn: u:id converted to %s\n", dn->bv_val,0,0 ); #endif } else { @@ -1744,7 +1972,7 @@ int slap_sasl_getdn( Connection *conn, Operation *op, char *id, int len, } /* Run thru regexp */ - slap_sasl2dn( op, dn, &dn2 ); + slap_sasl2dn( op, dn, &dn2, flags ); if( dn2.bv_val ) { sl_free( dn->bv_val, op->o_tmpmemctx ); *dn = dn2;