mirror of
https://git.openldap.org/openldap/openldap.git
synced 2025-01-06 10:46:21 +08:00
note an issue
This commit is contained in:
parent
acae40faa0
commit
6adfb5dd2f
@ -1090,8 +1090,12 @@ ldap_back_proxy_authz_ctrl(
|
|||||||
|
|
||||||
rs->sr_err = LDAP_SUCCESS;
|
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 ) )
|
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;
|
goto done;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -1153,7 +1157,8 @@ ldap_back_proxy_authz_ctrl(
|
|||||||
rc = slap_sasl_matches( op, li->idassert_authz,
|
rc = slap_sasl_matches( op, li->idassert_authz,
|
||||||
&authcDN, & authcDN );
|
&authcDN, & authcDN );
|
||||||
if ( rc != LDAP_SUCCESS ) {
|
if ( rc != LDAP_SUCCESS ) {
|
||||||
if ( li->idassert_flags & LDAP_BACK_AUTH_PRESCRIPTIVE ) {
|
if ( li->idassert_flags & LDAP_BACK_AUTH_PRESCRIPTIVE )
|
||||||
|
{
|
||||||
/* op->o_conn->c_ndn is not authorized
|
/* op->o_conn->c_ndn is not authorized
|
||||||
* to use idassert */
|
* to use idassert */
|
||||||
return rc;
|
return rc;
|
||||||
@ -1249,7 +1254,7 @@ ldap_back_proxy_authz_ctrl(
|
|||||||
ctrls[ 0 ]->ldctl_value.bv_len = assertedID.bv_len + STRLENOF( "dn:" );
|
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 );
|
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, "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 );
|
assertedID.bv_val, assertedID.bv_len + 1 );
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user