]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/attr.c
ITS#4587: selfwrite access is broken
[openldap] / servers / slapd / attr.c
index c1bc60faff618a9ec432ea391173edef77eb2f7a..bb4b4ca6062e464c3d7a54be8088c1340b777b59 100644 (file)
@@ -2,7 +2,7 @@
 /* $OpenLDAP$ */
 /* This work is part of OpenLDAP Software <http://www.openldap.org/>.
  *
- * Copyright 1998-2004 The OpenLDAP Foundation.
+ * Copyright 1998-2006 The OpenLDAP Foundation.
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
 #include <ac/string.h>
 #include <ac/time.h>
 
-#include "ldap_pvt.h"
 #include "slap.h"
 
+Attribute *
+attr_alloc( AttributeDescription *ad )
+{
+       Attribute *a = ch_malloc( sizeof(Attribute) );
+
+       a->a_desc = ad;
+       a->a_next = NULL;
+       a->a_flags = 0;
+       a->a_vals = NULL;
+       a->a_nvals = NULL;
+#ifdef LDAP_COMP_MATCH
+       a->a_comp_data = NULL;
+#endif
+
+       return a;
+}
+
 void
 attr_free( Attribute *a )
 {
-       ber_bvarray_free( a->a_vals );
-       if (a->a_nvals != a->a_vals) ber_bvarray_free( a->a_nvals );
+       if ( a->a_nvals && a->a_nvals != a->a_vals ) {
+               ber_bvarray_free( a->a_nvals );
+       }
+       /* a_vals may be equal to slap_dummy_bv, a static empty berval;
+        * this is used as a placeholder for attributes that do not carry
+        * values, e.g. when proxying search entries with the "attrsonly"
+        * bit set. */
+       if ( a->a_vals != &slap_dummy_bv ) {
+               ber_bvarray_free( a->a_vals );
+       }
        free( a );
 }
 
+#ifdef LDAP_COMP_MATCH
+void
+comp_tree_free( Attribute *a )
+{
+       Attribute *next;
+
+       for( ; a != NULL ; a = next ) {
+               next = a->a_next;
+               if ( component_destructor && a->a_comp_data ) {
+                       if ( a->a_comp_data->cd_mem_op )
+                               component_destructor( a->a_comp_data->cd_mem_op );
+                       free ( a->a_comp_data );
+               }
+       }
+}
+#endif
+
 void
 attrs_free( Attribute *a )
 {
@@ -60,35 +101,45 @@ attrs_free( Attribute *a )
        }
 }
 
-Attribute *attr_dup( Attribute *a )
+Attribute *
+attr_dup( Attribute *a )
 {
        Attribute *tmp;
 
-       if( a == NULL) return NULL;
+       if ( a == NULL) return NULL;
 
-       tmp = ch_malloc( sizeof(Attribute) );
+       tmp = attr_alloc( a->a_desc );
 
-       if( a->a_vals != NULL ) {
-               int i;
+       if ( a->a_vals != NULL ) {
+               int     i;
 
-               for( i=0; a->a_vals[i].bv_val != NULL; i++ ) {
+               for ( i = 0; !BER_BVISNULL( &a->a_vals[i] ); i++ ) {
                        /* EMPTY */ ;
                }
 
-               tmp->a_vals = ch_malloc((i+1) * sizeof(struct berval));
-               for( i=0; a->a_vals[i].bv_val != NULL; i++ ) {
+               tmp->a_vals = ch_malloc( (i + 1) * sizeof(struct berval) );
+               for ( i = 0; !BER_BVISNULL( &a->a_vals[i] ); i++ ) {
                        ber_dupbv( &tmp->a_vals[i], &a->a_vals[i] );
-                       if( tmp->a_vals[i].bv_val == NULL ) break;
+                       if ( BER_BVISNULL( &tmp->a_vals[i] ) ) break;
+                       /* FIXME: error? */
                }
-               tmp->a_vals[i].bv_val = NULL;
+               BER_BVZERO( &tmp->a_vals[i] );
+
+               /* a_nvals must be non null; it may be equal to a_vals */
+               assert( a->a_nvals != NULL );
+
+               if ( a->a_nvals != a->a_vals ) {
+                       int     j;
 
-               if( a->a_nvals != a->a_vals ) {
-                       tmp->a_nvals = ch_malloc((i+1) * sizeof(struct berval));
-                       for( i=0; a->a_nvals[i].bv_val != NULL; i++ ) {
-                               ber_dupbv( &tmp->a_nvals[i], &a->a_nvals[i] );
-                               if( tmp->a_nvals[i].bv_val == NULL ) break;
+                       tmp->a_nvals = ch_malloc( (i + 1) * sizeof(struct berval) );
+                       for ( j = 0; !BER_BVISNULL( &a->a_nvals[j] ); j++ ) {
+                               assert( j < i );
+                               ber_dupbv( &tmp->a_nvals[j], &a->a_nvals[j] );
+                               if ( BER_BVISNULL( &tmp->a_nvals[j] ) ) break;
+                               /* FIXME: error? */
                        }
-                       tmp->a_nvals[i].bv_val = NULL;
+                       assert( j == i );
+                       BER_BVZERO( &tmp->a_nvals[j] );
 
                } else {
                        tmp->a_nvals = tmp->a_vals;
@@ -98,15 +149,11 @@ Attribute *attr_dup( Attribute *a )
                tmp->a_vals = NULL;
                tmp->a_nvals = NULL;
        }
-
-       tmp->a_desc = a->a_desc;
-       tmp->a_next = NULL;
-       tmp->a_flags = 0;
-
        return tmp;
 }
 
-Attribute *attrs_dup( Attribute *a )
+Attribute *
+attrs_dup( Attribute *a )
 {
        Attribute *tmp, **next;
 
@@ -125,7 +172,6 @@ Attribute *attrs_dup( Attribute *a )
 }
 
 
-
 /*
  * attr_merge - merge the given type and value with the list of
  * attributes in attrs.
@@ -149,24 +195,32 @@ attr_merge(
        Attribute       **a;
 
        for ( a = &e->e_attrs; *a != NULL; a = &(*a)->a_next ) {
-               if ( ad_cmp( (*a)->a_desc, desc ) == 0 ) {
+               if (  (*a)->a_desc == desc ) {
                        break;
                }
        }
 
        if ( *a == NULL ) {
-               *a = (Attribute *) ch_malloc( sizeof(Attribute) );
-               (*a)->a_desc = desc;
-               (*a)->a_vals = NULL;
-               (*a)->a_nvals = NULL;
-               (*a)->a_next = NULL;
-               (*a)->a_flags = 0;
+               *a = attr_alloc( desc );
+       } else {
+               /*
+                * FIXME: if the attribute already exists, the presence
+                * of nvals and the value of (*a)->a_nvals must be consistent
+                */
+               assert( ( nvals == NULL && (*a)->a_nvals == (*a)->a_vals )
+                               || ( nvals != NULL && (*a)->a_nvals != (*a)->a_vals ) );
        }
 
        rc = value_add( &(*a)->a_vals, vals );
 
-       if( !rc && nvals ) rc = value_add( &(*a)->a_nvals, nvals );
-       else (*a)->a_nvals = (*a)->a_vals;
+       if ( rc == LDAP_SUCCESS ) {
+               if ( nvals ) {
+                       rc = value_add( &(*a)->a_nvals, nvals );
+                       /* FIXME: what if rc != LDAP_SUCCESS ? */
+               } else {
+                       (*a)->a_nvals = (*a)->a_vals;
+               }
+       }
 
        return rc;
 }
@@ -186,10 +240,10 @@ attr_merge_normalize(
        {
                int     i;
                
-               for ( i = 0; vals[i].bv_val; i++ );
+               for ( i = 0; !BER_BVISNULL( &vals[i] ); i++ );
 
                nvals = slap_sl_calloc( sizeof(struct berval), i + 1, memctx );
-               for ( i = 0; vals[i].bv_val; i++ ) {
+               for ( i = 0; !BER_BVISNULL( &vals[i] ); i++ ) {
                        rc = (*desc->ad_type->sat_equality->smr_normalize)(
                                        SLAP_MR_VALUE_OF_ATTRIBUTE_SYNTAX,
                                        desc->ad_type->sat_syntax,
@@ -197,11 +251,11 @@ attr_merge_normalize(
                                        &vals[i], &nvals[i], memctx );
 
                        if ( rc != LDAP_SUCCESS ) {
-                               nvals[i+1].bv_val = NULL;
+                               BER_BVZERO( &nvals[i + 1] );
                                goto error_return;
                        }
                }
-               nvals[i].bv_val = NULL;
+               BER_BVZERO( &nvals[i] );
        }
 
        rc = attr_merge( e, desc, vals, nvals );
@@ -224,24 +278,25 @@ attr_merge_one(
        Attribute       **a;
 
        for ( a = &e->e_attrs; *a != NULL; a = &(*a)->a_next ) {
-               if ( ad_cmp( (*a)->a_desc, desc ) == 0 ) {
+               if ( (*a)->a_desc == desc ) {
                        break;
                }
        }
 
        if ( *a == NULL ) {
-               *a = (Attribute *) ch_malloc( sizeof(Attribute) );
-               (*a)->a_desc = desc;
-               (*a)->a_vals = NULL;
-               (*a)->a_nvals = NULL;
-               (*a)->a_next = NULL;
-               (*a)->a_flags = 0;
+               *a = attr_alloc( desc );
        }
 
        rc = value_add_one( &(*a)->a_vals, val );
 
-       if( !rc && nval ) rc = value_add_one( &(*a)->a_nvals, nval );
-       else (*a)->a_nvals = (*a)->a_vals;
+       if ( rc == LDAP_SUCCESS ) {
+               if ( nval ) {
+                       rc = value_add_one( &(*a)->a_nvals, nval );
+                       /* FIXME: what if rc != LDAP_SUCCESS ? */
+               } else {
+                       (*a)->a_nvals = (*a)->a_vals;
+               }
+       }
        return rc;
 }
 
@@ -253,7 +308,7 @@ attr_merge_normalize_one(
        void            *memctx )
 {
        struct berval   nval;
-       struct berval   *nvalp;
+       struct berval   *nvalp = NULL;
        int             rc;
 
        if ( desc->ad_type->sat_equality &&
@@ -269,8 +324,6 @@ attr_merge_normalize_one(
                        return rc;
                }
                nvalp = &nval;
-       } else {
-               nvalp = NULL;
        }
 
        rc = attr_merge_one( e, desc, val, nvalp );
@@ -309,7 +362,7 @@ attr_find(
        AttributeDescription *desc )
 {
        for ( ; a != NULL; a = a->a_next ) {
-               if ( ad_cmp( a->a_desc, desc ) == 0 ) {
+               if ( a->a_desc == desc ) {
                        return( a );
                }
        }
@@ -332,7 +385,7 @@ attr_delete(
        Attribute       **a;
 
        for ( a = attrs; *a != NULL; a = &(*a)->a_next ) {
-               if ( ad_cmp( (*a)->a_desc, desc ) == 0 ) {
+               if ( (*a)->a_desc == desc ) {
                        Attribute       *save = *a;
                        *a = (*a)->a_next;
                        attr_free( save );