From: Pierangelo Masarati Date: Tue, 10 Jul 2001 18:19:22 +0000 (+0000) Subject: reworked rdn_attrs to use ldap_explode_rdn; maybe we should remove escapes "\" from... X-Git-Tag: LDBM_PRE_GIANT_RWLOCK~1244 X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=27e5c484e699b81f62e007bd168a16a20add726c;p=openldap reworked rdn_attrs to use ldap_explode_rdn; maybe we should remove escapes "\" from parts directly in ldap_explode_rdn --- diff --git a/servers/slapd/back-ldbm/modrdn.c b/servers/slapd/back-ldbm/modrdn.c index fcdc66e18c..f10a4b353d 100644 --- a/servers/slapd/back-ldbm/modrdn.c +++ b/servers/slapd/back-ldbm/modrdn.c @@ -619,7 +619,7 @@ ldbm_back_modrdn( /* Get attribute types and values of our new rdn, we will * need to add that to our new entry */ - if ( !rdn_attrs( newrdn, &new_rdn_types, &new_rdn_vals ) ) { + if ( rdn_attrs( newrdn, &new_rdn_types, &new_rdn_vals ) ) { #ifdef NEW_LOGGING LDAP_LOG(( "backend", LDAP_LEVEL_INFO, "ldbm_back_modrdn: can't figure out type(s)/value(s) of newrdn\n" )); @@ -661,7 +661,7 @@ ldbm_back_modrdn( goto return_results; } - if ( !rdn_attrs( old_rdn, &old_rdn_types, &old_rdn_vals ) ) { + if ( rdn_attrs( old_rdn, &old_rdn_types, &old_rdn_vals ) ) { #ifdef NEW_LOGGING LDAP_LOG(( "backend", LDAP_LEVEL_INFO, "ldbm_back_modrdn: can't figure out the old_rdn type(s)/value(s).\n" )); diff --git a/servers/slapd/dn.c b/servers/slapd/dn.c index 920b5d3d5e..92d2d590c2 100644 --- a/servers/slapd/dn.c +++ b/servers/slapd/dn.c @@ -494,201 +494,56 @@ rdn_attr_value( const char * rdn ) * array of strings "attribute_type" which is placed in newly allocated * memory, and the values of the attributes in pvalues, that is the * array of strings "attribute_value" which is placed in newly allocated - * memory. Returns 1 on success, 0 on failure. + * memory. Returns 0 on success, -1 on failure. * * note: got part of the code from dn_validate */ + int rdn_attrs( const char * rdn_in, char ***ptypes, char ***pvalues) { - char *start, *end, *s; - int state, gotesc, t = 0, v = 0; - char *dn = ch_strdup( rdn_in ); + char **parts, **p; *ptypes = NULL; *pvalues = NULL; - gotesc = 0; - state = B4LEADTYPE; - for ( start = end = s = dn; *s; s++ ) { - switch ( state ) { - case B4LEADTYPE: - case B4TYPE: - if ( OID_LEADCHAR(*s) ) { - state = INOIDTYPE; - start = end; - *end++ = *s; - } else if ( ATTR_LEADCHAR(*s) ) { - state = INKEYTYPE; - start = end; - *end++ = *s; - } else if ( ! ASCII_SPACE( *s ) ) { - goto failure; - state = INKEYTYPE; - *end++ = *s; - } - break; - - case INOIDTYPE: - if ( OID_CHAR(*s) ) { - *end++ = *s; - } else if ( *s == '=' ) { - state = B4VALUE; - charray_add_n( ptypes, start, ( end - start ) ); - t++; - *end++ = *s; - } else if ( ASCII_SPACE( *s ) ) { - state = B4EQUAL; - charray_add_n( ptypes, start, ( end - start ) ); - t++; - } else { - *end++ = *s; - goto failure; - } - break; - - case INKEYTYPE: - if ( ATTR_CHAR(*s) ) { - *end++ = *s; - } else if ( *s == '=' ) { - state = B4VALUE; - charray_add_n( ptypes, start, ( end - start ) ); - t++; - *end++ = *s; - } else if ( ASCII_SPACE( *s ) ) { - state = B4EQUAL; - charray_add_n( ptypes, start, ( end - start ) ); - t++; - } else { - *end++ = *s; - goto failure; - } - break; - - case B4EQUAL: - if ( *s == '=' ) { - state = B4VALUE; - *end++ = *s; - } else if ( ! ASCII_SPACE( *s ) ) { - /* not a valid dn - but what can we do here? */ - *end++ = *s; - goto failure; - } - break; - - case B4VALUE: - if ( *s == '"' ) { - state = INQUOTEDVALUE; - start = end; - *end++ = *s; - } else if ( ! ASCII_SPACE( *s ) ) { - state = INVALUE; - start = end; - *end++ = *s; - } - break; - - case INVALUE: - if ( !gotesc && RDN_SEPARATOR( *s ) ) { - while ( ASCII_SPACE( *(end - 1) ) ) - end--; - state = B4TYPE; - if ( RDN_ATTRTYPEANDVALUE_SEPARATOR( *s ) ) { - if ( ++v != t ) { - goto failure; - } - charray_add_n( pvalues, start, ( end - start ) ); - *end++ = *s; - } else { - /* not a rdn! */ - goto failure; - } - } else if ( gotesc && !RDN_NEEDSESCAPE( *s ) && - !RDN_SEPARATOR( *s ) ) { - *--end = *s; - end++; - } else if( !ASCII_SPACE( *s ) || !ASCII_SPACE( *(end - 1) ) ) { - *end++ = *s; - } - break; - - case INQUOTEDVALUE: - if ( !gotesc && *s == '"' ) { - state = B4SEPARATOR; - *end++ = *s; - } else if ( gotesc && !RDN_NEEDSESCAPE( *s ) ) { - *--end = *s; - end++; - } else if( !ASCII_SPACE( *s ) || !ASCII_SPACE( *(end - 1) ) ) { - *end++ = *s; - } - break; - - case B4SEPARATOR: - if ( RDN_SEPARATOR( *s ) ) { - state = B4TYPE; - if ( RDN_ATTRTYPEANDVALUE_SEPARATOR( *s ) ) { - if ( ++v != t ) { - goto failure; - } - charray_add_n( pvalues, start, ( end - start ) ); - *end++ = *s; - } else { - /* not a rdn! */ - goto failure; - } - } else if ( !ASCII_SPACE( *s ) ) { - goto failure; - } - break; - - default: -#ifdef NEW_LOGGING - LDAP_LOG(( "operation", LDAP_LEVEL_ERR, - "rdn_attrs: unknown state %d for rdn \"%s\".\n", - state, dn_in )); -#else - Debug( LDAP_DEBUG_ANY, - "rdn_attrs - unknown state %d\n", state, 0, 0 ); -#endif - goto failure; - } - - if ( *s == '\\' ) { - gotesc = 1; - } else { - gotesc = 0; - } - } + /* + * explode the rdn in parts + */ + parts = ldap_explode_rdn( rdn_in, 0 ); - if( gotesc ) { - /* shouldn't be left in escape */ - goto failure; + if ( parts == NULL ) { + return( -1 ); } - /* check end state */ - switch( state ) { - case B4LEADTYPE: /* looking for first type */ - case B4SEPARATOR: /* looking for separator */ - case INVALUE: /* inside value */ - if ( ++v != t ) { - goto failure; + for ( p = parts; p[0]; p++ ) { + char *s, *e, *d; + + /* split each rdn part in type value */ + s = strchr( p[0], '=' ); + if ( s == NULL ) { + charray_free( *ptypes ); + charray_free( *pvalues ); + charray_free( parts ); + return( -1 ); } - charray_add_n( pvalues, start, ( end - start ) ); - break; - default: - goto failure; + + /* type should be fine */ + charray_add_n( ptypes, p[0], ( s-p[0] ) ); + + /* value needs to be unescaped + * (maybe this should be moved to ldap_explode_rdn?) */ + for ( e = d = s + 1; e[0]; e++ ) { + if ( *e != '\\' ) { + *d++ = *e; + } + } + d[0] = '\0'; + charray_add( pvalues, s + 1 ); } - ch_free( dn ); - - return( 1 ); -failure: - ch_free( dn ); - charray_free( *ptypes ); - *ptypes = NULL; - charray_free( *pvalues ); - *pvalues = NULL; + /* free array */ + charray_free( parts ); return( 0 ); }