]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/back-ldbm/attribute.c
Add start of discussion of strings in LDAP/X.500 and OpenLDAP.
[openldap] / servers / slapd / back-ldbm / attribute.c
index bad3eed2e20b709588a201a414c4592e854fa297..0dc5daf03a20fb31a7e4f2f50743b0f0b0145520 100644 (file)
@@ -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
  */
 
@@ -16,7 +16,6 @@
 #include "back-ldbm.h"
 #include "proto-back-ldbm.h"
 
-
 /* return LDAP_SUCCESS IFF we can retrieve the attributes
  * of entry with e_ndn
  */
@@ -26,29 +25,31 @@ ldbm_back_attribute(
        Connection *conn,
        Operation *op,
        Entry   *target,
-       const char      *e_ndn,
+       struct berval   *entry_ndn,
        AttributeDescription *entry_at,
-       struct berval ***vals )
+       BerVarray *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;
+       BerVarray v;
+       const char *entry_at_name = entry_at->ad_cname.bv_val;
+       struct berval *iv, *jv;
+       AccessControlState acl_state = ACL_STATE_INIT;
+       int nvals = 0;
 
 #ifdef NEW_LOGGING
-       LDAP_LOG(( "backend", LDAP_LEVEL_ARGS,
-                  "ldbm_back_attribute: gr dn: \"%s\"\n", e_ndn ));
-       LDAP_LOG(( "backend", LDAP_LEVEL_ARGS,
-                  "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 : "" ));
+       LDAP_LOG( BACK_LDBM, ARGS,
+               "ldbm_back_attribute: gr dn: \"%s\"\n", entry_ndn->bv_val, 0, 0 );
+       LDAP_LOG( BACK_LDBM, ARGS, 
+               "ldbm_back_attribute: at: \"%s\"\n", entry_at_name, 0, 0);
+       LDAP_LOG( BACK_LDBM, ARGS, "ldbm_back_attribute: tr dn: \"%s\"\n",
+               target ? target->e_ndn : "", 0, 0 );
 #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,53 +59,53 @@ ldbm_back_attribute(
                target ? target->e_ndn : "", 0, 0 ); 
 #endif
 
-       if (target != NULL && strcmp(target->e_ndn, e_ndn) == 0) {
+       if (target != NULL && dn_match( &target->e_nname, entry_ndn) ) {
                /* 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 ));
+               LDAP_LOG( BACK_LDBM, DETAIL1, 
+                       "ldbm_back_attribute: target is LOCKED (%s)\n", 
+                       entry_ndn->bv_val, 0, 0);
 #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 ));
+                       LDAP_LOG( BACK_LDBM, INFO, 
+                               "ldbm_back_attribute: cannot find entry (%s)\n",
+                               entry_ndn->bv_val, 0, 0 );
 #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; 
                }
                
 #ifdef NEW_LOGGING
-               LDAP_LOG(( "backend", LDAP_LEVEL_DETAIL1,
-                          "ldbm_back_attribute: found entry (%s)\n", e_ndn ));
+               LDAP_LOG( BACK_LDBM, DETAIL1, 
+                       "ldbm_back_attribute: found entry (%s)\n", entry_ndn->bv_val, 0, 0);
 #else
                Debug( LDAP_DEBUG_ACL,
                        "=> ldbm_back_attribute: found entry: \"%s\"\n",
-                       e_ndn, 0, 0 ); 
+                       entry_ndn->bv_val, 0, 0 ); 
 #endif
 
     }
 
        /* find attribute values */
-       
+
        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 ));
+               LDAP_LOG( BACK_LDBM, INFO, 
+                       "ldbm_back_attribute: entry (%s) is an alias\n", e->e_dn, 0, 0 );
 #else
                Debug( LDAP_DEBUG_ACL,
                        "<= ldbm_back_attribute: entry is an alias\n", 0, 0, 0 );
@@ -116,8 +117,8 @@ 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 ));
+               LDAP_LOG( BACK_LDBM, INFO, 
+                       "ldbm_back_attribute: entry (%s) is a referral.\n", e->e_dn, 0, 0 );
 #else
                Debug( LDAP_DEBUG_ACL,
                        "<= ldbm_back_attribute: entry is an referral\n", 0, 0, 0 );
@@ -128,8 +129,8 @@ ldbm_back_attribute(
        }
 
        if (conn != NULL && op != NULL
-               && access_allowed(be, conn, op, e, slap_schema.si_ad_entry,
-                       NULL, ACL_READ) == 0)
+               && access_allowed( be, conn, op, e, slap_schema.si_ad_entry,
+                       NULL, ACL_READ, NULL ) == 0)
        {
                rc = LDAP_INSUFFICIENT_ACCESS;
                goto return_results;
@@ -137,8 +138,8 @@ 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 ));
+               LDAP_LOG( BACK_LDBM, INFO, 
+                       "ldbm_back_attribute: failed to find %s.\n", entry_at_name, 0, 0 );
 #else
                Debug( LDAP_DEBUG_ACL,
                        "<= ldbm_back_attribute: failed to find %s\n",
@@ -150,37 +151,40 @@ ldbm_back_attribute(
        }
 
        if (conn != NULL && op != NULL
-               && access_allowed(be, conn, op, e, entry_at, NULL, ACL_READ) == 0)
+               && access_allowed( be, conn, op, e, entry_at, NULL,
+                       ACL_READ, &acl_state ) == 0)
        {
                rc = LDAP_INSUFFICIENT_ACCESS;
                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 = (BerVarray) 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)
+                       && access_allowed( be, conn, op, e, entry_at,
+                               iv, ACL_READ, &acl_state ) == 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 ) {
+       nvals = jv - v;
+
+       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;
        }
@@ -192,13 +196,12 @@ return_results:
        }
 
 #ifdef NEW_LOGGING
-       LDAP_LOG(( "backend", LDAP_LEVEL_ENTRY,
-                  "ldbm_back_attribute: rc=%d nvals=%d.\n",
-                  rc, j ));
+       LDAP_LOG( BACK_LDBM, ENTRY, 
+               "ldbm_back_attribute: rc=%d nvals=%d.\n", rc, nvals, 0 );
 #else
        Debug( LDAP_DEBUG_TRACE,
                "ldbm_back_attribute: rc=%d nvals=%d\n",
-               rc, j, 0 ); 
+               rc, nvals, 0 ); 
 #endif
 
        return(rc);