error message from be_entry_put tool backend function

[openldap] / servers / slapd / back-ldbm / attribute.c

diff --git a/servers/slapd/back-ldbm/attribute.c b/servers/slapd/back-ldbm/attribute.c
index bad3eed2e20b709588a201a414c4592e854fa297..5a7dfa08691b14d858f7df2c409646d024bddf6e 100644 (file)
--- a/servers/slapd/back-ldbm/attribute.c
+++ b/servers/slapd/back-ldbm/attribute.c
@@ -1,7 +1,7 @@
 /* attribute.c - ldbm backend acl attribute routine */
 /* $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
  */
 
@@ -26,29 +26,30 @@ ldbm_back_attribute(
        Connection *conn,
        Operation *op,
        Entry   *target,
-       const char      *e_ndn,
+       struct berval   *entry_ndn,
        AttributeDescription *entry_at,
-       struct berval ***vals )
+       BVarray *vals )
 {
        struct ldbminfo *li = (struct ldbminfo *) be->be_private;    
        Entry        *e;
-       int          i, j, rc;
+       int          rc;
        Attribute   *attr;
-       struct berval **v;
-       const char *entry_at_name = entry_at->ad_cname->bv_val;
+       BVarray v;
+       const char *entry_at_name = entry_at->ad_cname.bv_val;
+       struct berval *iv, *jv;
 
 #ifdef NEW_LOGGING
        LDAP_LOG(( "backend", LDAP_LEVEL_ARGS,
-                  "ldbm_back_attribute: gr dn: \"%s\"\n", e_ndn ));
+               "ldbm_back_attribute: gr dn: \"%s\"\n", entry_ndn->bv_val ));
        LDAP_LOG(( "backend", LDAP_LEVEL_ARGS,
-                  "ldbm_back_attribute: at: \"%s\"\n", entry_at_name));
+               "ldbm_back_attribute: at: \"%s\"\n", entry_at_name));
        LDAP_LOG(( "backend", LDAP_LEVEL_ARGS,
-                  "ldbm_back_attribute: tr dn: \"%s\"\n",
-                  target ? target->e_ndn : "" ));
+               "ldbm_back_attribute: tr dn: \"%s\"\n",
+               target ? target->e_ndn : "" ));
 #else
        Debug( LDAP_DEBUG_ARGS,
                "=> ldbm_back_attribute: gr dn: \"%s\"\n",
-               e_ndn, 0, 0 ); 
+               entry_ndn->bv_val, 0, 0 ); 
        Debug( LDAP_DEBUG_ARGS,
                "=> ldbm_back_attribute: at: \"%s\"\n", 
                entry_at_name, 0, 0 ); 
@@ -58,31 +59,31 @@ ldbm_back_attribute(
                target ? target->e_ndn : "", 0, 0 ); 
 #endif
 
-       if (target != NULL && strcmp(target->e_ndn, e_ndn) == 0) {
+       if (target != NULL && strcmp(target->e_ndn, entry_ndn->bv_val) == 0) {
                /* we already have a LOCKED copy of the entry */
                e = target;
 #ifdef NEW_LOGGING
                LDAP_LOG(( "backend", LDAP_LEVEL_DETAIL1,
-                          "ldbm_back_attribute: target is LOCKED (%s)\n",
-                          e_ndn ));
+                       "ldbm_back_attribute: target is LOCKED (%s)\n",
+                       entry_ndn->bv_val ));
 #else
                Debug( LDAP_DEBUG_ARGS,
                        "=> ldbm_back_attribute: target is entry: \"%s\"\n",
-                       e_ndn, 0, 0 );
+                       entry_ndn->bv_val, 0, 0 );
 #endif
 
 
        } else {
                /* can we find entry with reader lock */
-               if ((e = dn2entry_r(be, e_ndn, NULL )) == NULL) {
+               if ((e = dn2entry_r(be, entry_ndn, NULL )) == NULL) {
 #ifdef NEW_LOGGING
                        LDAP_LOG(( "backend", LDAP_LEVEL_INFO,
-                                  "ldbm_back_attribute: cannot find entry (%s)\n",
-                                  e_ndn ));
+                               "ldbm_back_attribute: cannot find entry (%s)\n",
+                               entry_ndn->bv_val ));
 #else
                        Debug( LDAP_DEBUG_ACL,
                                "=> ldbm_back_attribute: cannot find entry: \"%s\"\n",
-                                       e_ndn, 0, 0 ); 
+                                       entry_ndn->bv_val, 0, 0 ); 
 #endif
 
                        return LDAP_NO_SUCH_OBJECT; 
@@ -90,11 +91,12 @@ ldbm_back_attribute(
                
 #ifdef NEW_LOGGING
                LDAP_LOG(( "backend", LDAP_LEVEL_DETAIL1,
-                          "ldbm_back_attribute: found entry (%s)\n", e_ndn ));
+                       "ldbm_back_attribute: found entry (%s)\n",
+                       entry_ndn->bv_val ));
 #else
                Debug( LDAP_DEBUG_ACL,
                        "=> ldbm_back_attribute: found entry: \"%s\"\n",
-                       e_ndn, 0, 0 ); 
+                       entry_ndn->bv_val, 0, 0 ); 
 #endif
 
     }
@@ -104,7 +106,7 @@ ldbm_back_attribute(
        if( is_entry_alias( e ) ) {
 #ifdef NEW_LOGGING
                LDAP_LOG(( "backend", LDAP_LEVEL_INFO,
-                          "ldbm_back_attribute: entry (%s) is an alias\n", e->e_dn ));
+                       "ldbm_back_attribute: entry (%s) is an alias\n", e->e_dn ));
 #else
                Debug( LDAP_DEBUG_ACL,
                        "<= ldbm_back_attribute: entry is an alias\n", 0, 0, 0 );
@@ -117,7 +119,7 @@ ldbm_back_attribute(
        if( is_entry_referral( e ) ) {
 #ifdef NEW_LOGGING
                LDAP_LOG(( "backend", LDAP_LEVEL_INFO,
-                          "ldbm_back_attribute: entry (%s) is a referral.\n", e->e_dn ));
+                       "ldbm_back_attribute: entry (%s) is a referral.\n", e->e_dn ));
 #else
                Debug( LDAP_DEBUG_ACL,
                        "<= ldbm_back_attribute: entry is an referral\n", 0, 0, 0 );
@@ -138,7 +140,7 @@ ldbm_back_attribute(
        if ((attr = attr_find(e->e_attrs, entry_at)) == NULL) {
 #ifdef NEW_LOGGING
                LDAP_LOG(( "backend", LDAP_LEVEL_INFO,
-                          "ldbm_back_attribute: failed to find %s.\n", entry_at_name ));
+                       "ldbm_back_attribute: failed to find %s.\n", entry_at_name ));
 #else
                Debug( LDAP_DEBUG_ACL,
                        "<= ldbm_back_attribute: failed to find %s\n",
@@ -156,31 +158,31 @@ ldbm_back_attribute(
                goto return_results;
        }
 
-       for ( i = 0; attr->a_vals[i] != NULL; i++ ) {
+       for ( iv = attr->a_vals; iv->bv_val != NULL; iv++ ) {
                /* count them */
        }
 
-       v = (struct berval **) ch_malloc( sizeof(struct berval *) * (i+1) );
+       v = (BVarray) ch_malloc( sizeof(struct berval) * ((iv - attr->a_vals)+1) );
 
-       for ( i=0, j=0; attr->a_vals[i] != NULL; i++ ) {
+       for ( iv=attr->a_vals, jv=v; iv->bv_val; iv++ ) {
                if( conn != NULL
                        && op != NULL
                        && access_allowed(be, conn, op, e, entry_at,
-                               attr->a_vals[i], ACL_READ) == 0)
+                               iv, ACL_READ) == 0)
                {
                        continue;
                }
-               v[j] = ber_bvdup( attr->a_vals[i] );
+               ber_dupbv( jv, iv );
 
-               if( v[j] != NULL ) j++;
+               if( jv->bv_val != NULL ) jv++;
        }
 
-       if( j == 0 ) {
+       if( jv == v ) {
                ch_free( v );
                *vals = NULL;
                rc = LDAP_INSUFFICIENT_ACCESS;
        } else {
-               v[j] = NULL;
+               jv->bv_val = NULL;
                *vals = v;
                rc = LDAP_SUCCESS;
        }
@@ -194,11 +196,11 @@ return_results:
 #ifdef NEW_LOGGING
        LDAP_LOG(( "backend", LDAP_LEVEL_ENTRY,
                   "ldbm_back_attribute: rc=%d nvals=%d.\n",
-                  rc, j ));
+                  rc, jv - v ));
 #else
        Debug( LDAP_DEBUG_TRACE,
                "ldbm_back_attribute: rc=%d nvals=%d\n",
-               rc, j, 0 ); 
+               rc, jv - v, 0 ); 
 #endif
 
        return(rc);