mirror/openldap
2
0
Fork 0
mirror of https://git.openldap.org/openldap/openldap.git synced 2025-01-24 13:24:56 +08:00
Code Issues Packages Projects Releases Wiki Activity

normalize rewritten DN-valued attrs (ITS#5772)

Browse Source
This commit is contained in:
Pierangelo Masarati 2008-10-26 19:29:36 +00:00
parent f29d1b1455
commit 9401bd3817
3 changed files with 17 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
servers/slapd/overlays/rwm.c
View File

@ -1225,7 +1225,7 @@ remove_oc:;
|| ( mapping != NULL && mapping->m_src_ad->ad_type->sat_syntax == slap_schema.si_syn_distinguishedName ) )
{
dc.ctx = "searchAttrDN";
rc = rwm_dnattr_result_rewrite( &dc, (*ap)->a_vals );
rc = rwm_dnattr_result_rewrite( &dc, (*ap)->a_vals, (*ap)->a_nvals );
if ( rc != LDAP_SUCCESS ) {
goto cleanup_attr;
}

2
servers/slapd/overlays/rwm.h
View File

@ -175,7 +175,7 @@ rwm_referral_rewrite(
void *cookie,
BerVarray a_vals,
BerVarray *pa_nvals );
extern int rwm_dnattr_result_rewrite( dncookie *dc, BerVarray a_vals );
extern int rwm_dnattr_result_rewrite( dncookie *dc, BerVarray a_vals, BerVarray a_nvals );
extern int rwm_referral_result_rewrite( dncookie *dc, BerVarray a_vals );
LDAP_END_DECL

21
servers/slapd/overlays/rwmmap.c
View File

@ -1178,7 +1178,8 @@ rwm_referral_result_rewrite(
int
rwm_dnattr_result_rewrite(
dncookie *dc,
BerVarray a_vals )
BerVarray a_vals,
BerVarray a_nvals )
{
int i, last;
@ -1186,11 +1187,11 @@ rwm_dnattr_result_rewrite(
last--;
for ( i = 0; !BER_BVISNULL( &a_vals[i] ); i++ ) {
struct berval dn;
struct berval pdn, ndn;
int rc;
dn = a_vals[i];
rc = rwm_dn_massage_pretty( dc, &a_vals[i], &dn );
pdn = a_vals[i];
rc = rwm_dn_massage_pretty_normalize( dc, &a_vals[i], &pdn, &ndn );
switch ( rc ) {
case LDAP_UNWILLING_TO_PERFORM:
/*
@ -1198,19 +1199,27 @@ rwm_dnattr_result_rewrite(
* legal to trim values when adding/modifying;
* it should be when searching (e.g. ACLs).
*/
assert( a_vals[i].bv_val != a_nvals[i].bv_val );
ch_free( a_vals[i].bv_val );
ch_free( a_nvals[i].bv_val );
if ( last > i ) {
a_vals[i] = a_vals[last];
a_nvals[i] = a_nvals[last];
}
BER_BVZERO( &a_vals[last] );
BER_BVZERO( &a_nvals[last] );
last--;
break;
default:
/* leave attr untouched if massage failed */
if ( !BER_BVISNULL( &dn ) && a_vals[i].bv_val != dn.bv_val ) {
if ( !BER_BVISNULL( &pdn ) && a_vals[i].bv_val != pdn.bv_val ) {
ch_free( a_vals[i].bv_val );
a_vals[i] = dn;
a_vals[i] = pdn;
}
if ( !BER_BVISNULL( &ndn ) && a_nvals[i].bv_val != ndn.bv_val ) {
ch_free( a_nvals[i].bv_val );
a_nvals[i] = ndn;
}
break;
}