need this outside for back monitor ...

[openldap] / servers / slapd / dn.c

diff --git a/servers/slapd/dn.c b/servers/slapd/dn.c
index 73bc489d3c4e471fe6b6715ed7554328a22287e1..de3ae7dbac2de9706811fba8ab8bf926f854c907 100644 (file)
--- a/servers/slapd/dn.c
+++ b/servers/slapd/dn.c
@@ -1,7 +1,7 @@
 /* dn.c - routines for dealing with distinguished names */
 /* $OpenLDAP$ */
 /*
- * Copyright 1998-2000 The OpenLDAP Foundation, All Rights Reserved.
+ * Copyright 1998-2002 The OpenLDAP Foundation, All Rights Reserved.
  * COPYING RESTRICTIONS APPLY, see COPYRIGHT file
  */
 
@@ -76,8 +76,7 @@ LDAPDN_validate( LDAPDN *dn )
                        /* 
                         * Replace attr oid/name with the canonical name
                         */
-                       free( ava->la_attr.bv_val );
-                       ber_dupbv( &ava->la_attr, &ad->ad_cname );
+                       ava->la_attr = ad->ad_cname;
 
                        validate = ad->ad_type->sat_syntax->ssyn_validate;
 
@@ -233,6 +232,7 @@ LDAPDN_rewrite( LDAPDN *dn, unsigned flags )
                        slap_syntax_transform_func *transf = NULL;
                        MatchingRule *mr;
                        struct berval           bv = { 0, NULL };
+                       int                     do_sort = 0;
 
                        assert( ava );
 
@@ -245,13 +245,13 @@ LDAPDN_rewrite( LDAPDN *dn, unsigned flags )
                                }
                                
                                ava->la_private = ( void * )ad;
+                               do_sort = 1;
                        }
 
                        /* 
                         * Replace attr oid/name with the canonical name
                         */
-                       free( ava->la_attr.bv_val );
-                       ber_dupbv( &ava->la_attr, &ad->ad_cname );
+                       ava->la_attr = ad->ad_cname;
 
                        if( flags & SLAP_LDAPDN_PRETTY ) {
                                transf = ad->ad_type->sat_syntax->ssyn_pretty;
@@ -277,7 +277,7 @@ LDAPDN_rewrite( LDAPDN *dn, unsigned flags )
                                char *s = bv.bv_val;
 
                                ber_str2bv( UTF8normalize( bv.bv_val ? &bv
-                                       : &ava->la_value, UTF8_CASEFOLD ),
+                                       : &ava->la_value, LDAP_UTF8_CASEFOLD ),
                                        0, 0, &bv );
                                free( s );
                        }
@@ -287,7 +287,7 @@ LDAPDN_rewrite( LDAPDN *dn, unsigned flags )
                                ava->la_value = bv;
                        }
 
-                       AVA_Sort( rdn, iAVA );
+                       if( do_sort ) AVA_Sort( rdn, iAVA );
                }
        }
 
@@ -646,11 +646,18 @@ dnParent(
        const char      *p;
        int             rc;
 
-       rc = ldap_str2rdn( dn, NULL, &p, LDAP_DN_FORMAT_LDAP | LDAP_DN_SKIP );
+       rc = ldap_str2rdn( dn, NULL, (char **)&p,
+               LDAP_DN_FORMAT_LDAP | LDAP_DN_SKIP );
        if ( rc != LDAP_SUCCESS ) {
                return rc;
        }
 
+       /* Parent is root */
+       if (*p == '\0') {
+               *pdn = "";
+               return LDAP_SUCCESS;
+       }
+
        assert( DN_SEPARATOR( p[ 0 ] ) );
        p++;
 
@@ -713,7 +720,7 @@ dnExtractRdn(
                return LDAP_OTHER;
        }
 
-       rc = ldap_str2rdn( dn->bv_val, &tmpRDN, &p, LDAP_DN_FORMAT_LDAP );
+       rc = ldap_str2rdn( dn->bv_val, &tmpRDN, (char **)&p, LDAP_DN_FORMAT_LDAP );
        if ( rc != LDAP_SUCCESS ) {
                return rc;
        }
@@ -752,7 +759,7 @@ dn_rdnlen(
                return 0;
        }
 
-       rc = ldap_str2rdn( dn_in->bv_val, NULL, &p, 
+       rc = ldap_str2rdn( dn_in->bv_val, NULL, (char **)&p, 
                        LDAP_DN_FORMAT_LDAP | LDAP_DN_SKIP );
        if ( rc != LDAP_SUCCESS ) {
                return 0;
@@ -898,7 +905,7 @@ rdn_attrs( const char * rdn, char ***types, char ***values)
        assert( *values == NULL );
        assert( types == NULL || *types == NULL );
 
-       rc = ldap_str2rdn( rdn, &tmpRDN, &p, LDAP_DN_FORMAT_LDAP );
+       rc = ldap_str2rdn( rdn, &tmpRDN, (char **)&p, LDAP_DN_FORMAT_LDAP );
        if ( rc != LDAP_SUCCESS ) {
                return rc;
        }
@@ -969,7 +976,7 @@ rdnValidate( struct berval *rdn )
        /*
         * must be parsable
         */
-       rc = ldap_str2rdn( rdn, &RDN, &p, LDAP_DN_FORMAT_LDAP );
+       rc = ldap_str2rdn( rdn, &RDN, (char **)&p, LDAP_DN_FORMAT_LDAP );
        if ( rc != LDAP_SUCCESS ) {
                return 0;
        }