mirror of
https://git.openldap.org/openldap/openldap.git
synced 2025-01-06 10:46:21 +08:00
ITS#6195 must normalize cookies before comparing them
This commit is contained in:
parent
d3e7717473
commit
cc6dc57746
@ -176,7 +176,7 @@ slap_parse_sync_cookie(
|
||||
char *csn_str;
|
||||
char *cval;
|
||||
char *next, *end;
|
||||
AttributeDescription *ad = slap_schema.si_ad_modifyTimestamp;
|
||||
AttributeDescription *ad = slap_schema.si_ad_entryCSN;
|
||||
|
||||
if ( cookie == NULL )
|
||||
return -1;
|
||||
@ -230,13 +230,11 @@ slap_parse_sync_cookie(
|
||||
continue;
|
||||
}
|
||||
if ( !strncmp( next, "csn=", STRLENOF("csn=") )) {
|
||||
slap_syntax_validate_func *validate;
|
||||
struct berval stamp;
|
||||
|
||||
next += STRLENOF("csn=");
|
||||
while ( next < end ) {
|
||||
csn_str = next;
|
||||
/* FIXME use csnValidate when it gets implemented */
|
||||
csn_ptr = strchr( csn_str, '#' );
|
||||
if ( !csn_ptr || csn_ptr > end )
|
||||
break;
|
||||
@ -244,14 +242,6 @@ slap_parse_sync_cookie(
|
||||
* want to parse the rid then. But we still iterate
|
||||
* through the string to find the end.
|
||||
*/
|
||||
if ( ad ) {
|
||||
stamp.bv_val = csn_str;
|
||||
stamp.bv_len = csn_ptr - csn_str;
|
||||
validate = ad->ad_type->sat_syntax->ssyn_validate;
|
||||
if ( validate( ad->ad_type->sat_syntax, &stamp )
|
||||
!= LDAP_SUCCESS )
|
||||
break;
|
||||
}
|
||||
cval = strchr( csn_ptr, ';' );
|
||||
if ( !cval )
|
||||
cval = strchr(csn_ptr, ',' );
|
||||
@ -261,7 +251,16 @@ slap_parse_sync_cookie(
|
||||
stamp.bv_len = end - csn_str;
|
||||
if ( ad ) {
|
||||
struct berval bv;
|
||||
ber_dupbv_x( &bv, &stamp, memctx );
|
||||
stamp.bv_val = csn_str;
|
||||
if ( ad->ad_type->sat_syntax->ssyn_validate(
|
||||
ad->ad_type->sat_syntax, &stamp ) != LDAP_SUCCESS )
|
||||
break;
|
||||
if ( ad->ad_type->sat_equality->smr_normalize(
|
||||
SLAP_MR_VALUE_OF_ATTRIBUTE_SYNTAX,
|
||||
ad->ad_type->sat_syntax,
|
||||
ad->ad_type->sat_equality,
|
||||
&stamp, &bv, memctx ) != LDAP_SUCCESS )
|
||||
break;
|
||||
ber_bvarray_add_x( &cookie->ctxcsn, &bv, memctx );
|
||||
cookie->numcsns++;
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user