ITS#5077 again

This commit is contained in:
Howard Chu 2007-08-25 15:40:39 +00:00
parent 96611ac703
commit a1c088ed02

View file

@ -2677,78 +2677,63 @@ attr_cmp( Operation *op, Attribute *old, Attribute *new,
modtail = *mret; modtail = *mret;
if ( old ) { if ( old ) {
int n, o, d, a, *adds, *dels; int n, o, nn, no;
struct berval **adds, **dels;
/* count old and new */ /* count old and new */
for ( o=0; old->a_vals[o].bv_val; o++ ) ; for ( o=0; old->a_vals[o].bv_val; o++ ) ;
for ( n=0; new->a_vals[n].bv_val; n++ ) ; for ( n=0; new->a_vals[n].bv_val; n++ ) ;
adds = op->o_tmpalloc( sizeof(int) * n, op->o_tmpmemctx ); adds = op->o_tmpalloc( sizeof(struct berval *) * n, op->o_tmpmemctx );
dels = op->o_tmpalloc( sizeof(int) * o, op->o_tmpmemctx ); dels = op->o_tmpalloc( sizeof(struct berval *) * o, op->o_tmpmemctx );
d = 0;
a = 0;
i = 0;
j = 0;
while ( i < o && j < n ) { for ( i=0; i<o; i++ ) dels[i] = &old->a_vals[i];
int k; for ( i=0; i<n; i++ ) adds[i] = &new->a_vals[i];
if ( bvmatch( &old->a_vals[i], &new->a_vals[j] ) ) {
i++; nn = n; no = o;
j++;
for ( i=0; i<o; i++ ) {
for ( j=0; j<n; j++ ) {
if ( !adds[j] )
continue; continue;
} if ( bvmatch( dels[i], adds[j] ) ) {
for ( k = j + 1; k<n; k++ ) { no--;
if ( bvmatch( &old->a_vals[i], &new->a_vals[k] ) ) { nn--;
adds[j] = NULL;
dels[i] = NULL;
break; break;
} }
} }
/* an old value was deleted */
if ( k == n ) {
dels[d++] = i++;
continue;
} }
/* old value still exists, move to next */
i++;
for ( k = i; k < o; k++ ) {
if ( bvmatch( &old->a_vals[k], &new->a_vals[j] ) ) {
break;
}
}
if ( k == o ) {
adds[a++] = j++;
} else {
j++;
}
}
while ( i < o )
dels[d++] = i++;
while ( j < n )
adds[a++] = j++;
i = j;
/* all old values were deleted, just use the replace op */ /* all old values were deleted, just use the replace op */
if ( d == o ) { if ( no == o ) {
i = j-1; i = j-1;
} else if ( d ) { } else if ( no ) {
/* delete some values */ /* delete some values */
mod = ch_malloc( sizeof( Modifications ) ); mod = ch_malloc( sizeof( Modifications ) );
mod->sml_op = LDAP_MOD_DELETE; mod->sml_op = LDAP_MOD_DELETE;
mod->sml_flags = 0; mod->sml_flags = 0;
mod->sml_desc = old->a_desc; mod->sml_desc = old->a_desc;
mod->sml_type = mod->sml_desc->ad_cname; mod->sml_type = mod->sml_desc->ad_cname;
mod->sml_values = ch_malloc( ( d + 1 ) * sizeof(struct berval) ); mod->sml_values = ch_malloc( ( o - no + 1 ) * sizeof(struct berval) );
if ( old->a_vals != old->a_nvals ) { if ( old->a_vals != old->a_nvals ) {
mod->sml_nvalues = ch_malloc( ( d + 1 ) * sizeof(struct berval) ); mod->sml_nvalues = ch_malloc( ( o - no + 1 ) * sizeof(struct berval) );
} else { } else {
mod->sml_nvalues = NULL; mod->sml_nvalues = NULL;
} }
for ( i = 0; i < d; i++ ) { j = 0;
ber_dupbv( &mod->sml_values[i], &old->a_vals[dels[i]] ); for ( i = 0; i < o; i++ ) {
if ( !dels[i] ) continue;
ber_dupbv( &mod->sml_values[j], &old->a_vals[i] );
if ( mod->sml_nvalues ) { if ( mod->sml_nvalues ) {
ber_dupbv( &mod->sml_nvalues[i], &old->a_nvals[dels[i]] ); ber_dupbv( &mod->sml_nvalues[j], &old->a_nvals[i] );
} }
j++;
} }
BER_BVZERO( &mod->sml_values[i] ); BER_BVZERO( &mod->sml_values[j] );
if ( mod->sml_nvalues ) { if ( mod->sml_nvalues ) {
BER_BVZERO( &mod->sml_nvalues[i] ); BER_BVZERO( &mod->sml_nvalues[j] );
} }
*modtail = mod; *modtail = mod;
modtail = &mod->sml_next; modtail = &mod->sml_next;
@ -2756,27 +2741,30 @@ attr_cmp( Operation *op, Attribute *old, Attribute *new,
} }
op->o_tmpfree( dels, op->o_tmpmemctx ); op->o_tmpfree( dels, op->o_tmpmemctx );
/* some values were added */ /* some values were added */
if ( a && d < o ) { if ( nn && no < o ) {
mod = ch_malloc( sizeof( Modifications ) ); mod = ch_malloc( sizeof( Modifications ) );
mod->sml_op = LDAP_MOD_ADD; mod->sml_op = LDAP_MOD_ADD;
mod->sml_flags = 0; mod->sml_flags = 0;
mod->sml_desc = old->a_desc; mod->sml_desc = old->a_desc;
mod->sml_type = mod->sml_desc->ad_cname; mod->sml_type = mod->sml_desc->ad_cname;
mod->sml_values = ch_malloc( ( a + 1 ) * sizeof(struct berval) ); mod->sml_values = ch_malloc( ( n - nn + 1 ) * sizeof(struct berval) );
if ( old->a_vals != old->a_nvals ) { if ( old->a_vals != old->a_nvals ) {
mod->sml_nvalues = ch_malloc( ( a + 1 ) * sizeof(struct berval) ); mod->sml_nvalues = ch_malloc( ( n - nn + 1 ) * sizeof(struct berval) );
} else { } else {
mod->sml_nvalues = NULL; mod->sml_nvalues = NULL;
} }
for ( i = 0; i < a; i++ ) { j = 0;
ber_dupbv( &mod->sml_values[i], &new->a_vals[adds[i]] ); for ( i = 0; i < n; i++ ) {
if ( !adds[i] ) continue;
ber_dupbv( &mod->sml_values[j], &new->a_vals[i] );
if ( mod->sml_nvalues ) { if ( mod->sml_nvalues ) {
ber_dupbv( &mod->sml_nvalues[i], &new->a_nvals[adds[i]] ); ber_dupbv( &mod->sml_nvalues[j], &new->a_nvals[i] );
} }
j++;
} }
BER_BVZERO( &mod->sml_values[i] ); BER_BVZERO( &mod->sml_values[j] );
if ( mod->sml_nvalues ) { if ( mod->sml_nvalues ) {
BER_BVZERO( &mod->sml_nvalues[i] ); BER_BVZERO( &mod->sml_nvalues[j] );
} }
*modtail = mod; *modtail = mod;
modtail = &mod->sml_next; modtail = &mod->sml_next;