]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/back-passwd/search.c
Fix access_allowed() error checking bug
[openldap] / servers / slapd / back-passwd / search.c
index fbd84fd209a706adf7965574e9d124a7a32f59b6..bab8871253dd1f4c6fb0f6b211d1379df7c1024c 100644 (file)
 #include <pwd.h>
 
 #include "slap.h"
-#include "external.h"
+#include "back-passwd.h"
 #include <ldap_pvt.h>
 
+static void pw_start( Backend *be );
+
 static Entry *pw2entry(
        Backend *be,
        struct passwd *pw,
-       char *rdn);
+       const char **text);
 
 int
 passwd_back_search(
     Backend    *be,
     Connection *conn,
     Operation  *op,
-    const char *base,
-    const char *nbase,
+    struct berval *base,
+    struct berval *nbase,
     int                scope,
     int                deref,
     int                slimit,
     int                tlimit,
     Filter     *filter,
-    const char *filterstr,
-    char       **attrs,
+    struct berval      *filterstr,
+    AttributeName      *attrs,
     int                attrsonly
 )
 {
@@ -46,10 +48,10 @@ passwd_back_search(
        int sent = 0;
        int err = LDAP_SUCCESS;
 
-       char *rdn = NULL;
-       char *parent = NULL;
+       LDAPRDN *rdn = NULL;
+       struct berval parent = { 0, NULL };
        char *matched = NULL;
-       char *user = NULL;
+       const char *text = NULL;
 
        AttributeDescription *ad_objectClass = slap_schema.si_ad_objectClass;
 
@@ -59,65 +61,46 @@ passwd_back_search(
        slimit = (slimit > be->be_sizelimit || slimit < 1) ? be->be_sizelimit
            : slimit;
 
-       endpwent();
-
-#ifdef HAVE_SETPWFILE
-       if ( be->be_private != NULL ) {
-               (void) setpwfile( (char *) be->be_private );
-       }
-#endif /* HAVE_SETPWFILE */
-
        /* Handle a query for the base of this backend */
-       if ( be_issuffix( be,  nbase ) ) {
-               struct berval   val, *vals[2];
+       if ( be_issuffix( be, nbase ) ) {
+               struct berval   vals[2];
 
-               vals[0] = &val;
-               vals[1] = NULL;
+               vals[1].bv_val = NULL;
 
-               matched = ch_strdup( base );
+               matched = (char *) base;
 
                if( scope != LDAP_SCOPE_ONELEVEL ) {
-                       char *type;
                        AttributeDescription *desc = NULL;
 
                        /* Create an entry corresponding to the base DN */
                        e = (Entry *) ch_calloc(1, sizeof(Entry));
+                       e->e_name.bv_val = ch_strdup( base->bv_val );
+                       e->e_name.bv_len = base->bv_len;
+                       e->e_nname.bv_val =  ch_strdup( nbase->bv_val );
+                       e->e_nname.bv_len = nbase->bv_len;
                        e->e_attrs = NULL;
-                       e->e_dn = ch_strdup( base );
+                       e->e_private = NULL;
 
                        /* Use the first attribute of the DN
                        * as an attribute within the entry itself.
                        */
-                       rdn = dn_rdn(NULL, base);
-
-                       if( rdn == NULL || (s = strchr(rdn, '=')) == NULL ) {
+                       if( ldap_bv2rdn( base, &rdn, (char **)&text, 
+                               LDAP_DN_FORMAT_LDAP ) )
+                       {
                                err = LDAP_INVALID_DN_SYNTAX;
-                               free(rdn);
                                goto done;
                        }
 
-                       val.bv_val = rdn_attr_value(rdn);
-                       val.bv_len = strlen( val.bv_val );
-
-                       type = rdn_attr_type(rdn);
-
-                       {
-                               int rc;
-                               const char *text;
-                               rc = slap_str2ad( type, &desc, &text );
-
-                               if( rc != LDAP_SUCCESS ) {
-                                       err = LDAP_NO_SUCH_OBJECT;
-                                       free(rdn);
-                                       goto done;
-                               }
+                       if( slap_bv2ad( &rdn[0][0]->la_attr, &desc, &text )) {
+                               err = LDAP_NO_SUCH_OBJECT;
+                               ldap_rdnfree(rdn);
+                               goto done;
                        }
 
+                       vals[0] = rdn[0][0]->la_value;
                        attr_merge( e, desc, vals );
 
-                       ad_free( desc, 1 );
-
-                       free(rdn);
+                       ldap_rdnfree(rdn);
                        rdn = NULL;
 
                        /* Every entry needs an objectclass. We don't really
@@ -127,8 +110,8 @@ passwd_back_search(
                         *
                         * should be a configuratable item
                         */
-                       val.bv_val = "organizationalUnit";
-                       val.bv_len = sizeof("organizationalUnit")-1;
+                       vals[0].bv_val = "organizationalUnit";
+                       vals[0].bv_len = sizeof("organizationalUnit")-1;
                        attr_merge( e, ad_objectClass, vals );
        
                        if ( test_filter( be, conn, op, e, filter ) == LDAP_COMPARE_TRUE ) {
@@ -141,25 +124,31 @@ passwd_back_search(
                if ( scope != LDAP_SCOPE_BASE ) {
                        /* check all our "children" */
 
+                       ldap_pvt_thread_mutex_lock( &passwd_mutex );
+                       pw_start( be );
                        for ( pw = getpwent(); pw != NULL; pw = getpwent() ) {
                                /* check for abandon */
-                               ldap_pvt_thread_mutex_lock( &op->o_abandonmutex );
                                if ( op->o_abandon ) {
-                                       ldap_pvt_thread_mutex_unlock( &op->o_abandonmutex );
                                        endpwent();
+                                       ldap_pvt_thread_mutex_unlock( &passwd_mutex );
                                        return( -1 );
                                }
-                               ldap_pvt_thread_mutex_unlock( &op->o_abandonmutex );
 
                                /* check time limit */
                                if ( slap_get_time() > stoptime ) {
                                        send_ldap_result( conn, op, LDAP_TIMELIMIT_EXCEEDED,
                                        NULL, NULL, NULL, NULL );
                                        endpwent();
+                                       ldap_pvt_thread_mutex_unlock( &passwd_mutex );
                                        return( 0 );
                                }
 
-                               e = pw2entry( be, pw, NULL );
+                               if ( !(e = pw2entry( be, pw, &text )) ) {
+                                       err = LDAP_OTHER;
+                                       endpwent();
+                                       ldap_pvt_thread_mutex_unlock( &passwd_mutex );
+                                       goto done;
+                               }
 
                                if ( test_filter( be, conn, op, e, filter ) == LDAP_COMPARE_TRUE ) {
                                        /* check size limit */
@@ -167,6 +156,7 @@ passwd_back_search(
                                                send_ldap_result( conn, op, LDAP_SIZELIMIT_EXCEEDED,
                                                NULL, NULL, NULL, NULL );
                                                endpwent();
+                                               ldap_pvt_thread_mutex_unlock( &passwd_mutex );
                                                return( 0 );
                                        }
 
@@ -178,19 +168,22 @@ passwd_back_search(
                                entry_free( e );
                        }
                        endpwent();
+                       ldap_pvt_thread_mutex_unlock( &passwd_mutex );
                }
 
        } else {
-               parent = dn_parent( be, base );
+               if (! be_issuffix( be, nbase ) ) {
+                       dnParent( nbase, &parent );
+               }
 
                /* This backend is only one layer deep. Don't answer requests for
                 * anything deeper than that.
                 */
-               if( !be_issuffix( be, parent ) ) {
+               if( !be_issuffix( be, &parent ) ) {
                        int i;
-                       for( i=0; be->be_nsuffix[i] != NULL; i++ ) {
-                               if( dn_issuffix( nbase, be->be_nsuffix[i] ) ) {
-                                       matched = ch_strdup( be->be_suffix[i] );
+                       for( i=0; be->be_nsuffix[i].bv_val != NULL; i++ ) {
+                               if( dnIsSuffix( nbase, &be->be_nsuffix[i] ) ) {
+                                       matched = be->be_suffix[i].bv_val;
                                        break;
                                }
                        }
@@ -202,21 +195,28 @@ passwd_back_search(
                        goto done;
                }
 
-               rdn = dn_rdn( NULL, base );
-
-               if ( (user = rdn_attr_value(rdn)) == NULL) {
-                       err = LDAP_OPERATIONS_ERROR;
+               if ( ldap_bv2rdn( base, &rdn, (char **)&text,
+                       LDAP_DN_FORMAT_LDAP ))
+               { 
+                       err = LDAP_OTHER;
                        goto done;
                }
 
-               if ( (pw = getpwnam( user )) == NULL ) {
-                       matched = parent;
-                       parent = NULL;
+               ldap_pvt_thread_mutex_lock( &passwd_mutex );
+               pw_start( be );
+               if ( (pw = getpwnam( rdn[0][0]->la_value.bv_val )) == NULL ) {
+                       matched = parent.bv_val;
                        err = LDAP_NO_SUCH_OBJECT;
+                       ldap_pvt_thread_mutex_unlock( &passwd_mutex );
                        goto done;
                }
 
-               e = pw2entry( be, pw, rdn );
+               e = pw2entry( be, pw, &text );
+               ldap_pvt_thread_mutex_unlock( &passwd_mutex );
+               if ( !e ) {
+                       err = LDAP_OTHER;
+                       goto done;
+               }
 
                if ( test_filter( be, conn, op, e, filter ) == LDAP_COMPARE_TRUE ) {
                        send_search_entry( be, conn, op,
@@ -229,79 +229,95 @@ passwd_back_search(
 
 done:
        send_ldap_result( conn, op,
-               err, err == LDAP_NO_SUCH_OBJECT ? matched : NULL, NULL,
+               err, err == LDAP_NO_SUCH_OBJECT ? matched : NULL, text,
                NULL, NULL );
 
-       if( matched != NULL ) free( matched );
-       if( parent != NULL ) free( parent );
-       if( rdn != NULL ) free( rdn );
-       if( user != NULL ) free( user );
+       if( rdn != NULL ) ldap_rdnfree( rdn );
 
        return( 0 );
 }
 
+static void
+pw_start(
+       Backend *be
+)
+{
+       endpwent();
+
+#ifdef HAVE_SETPWFILE
+       if ( be->be_private != NULL ) {
+               (void) setpwfile( (char *) be->be_private );
+       }
+#endif /* HAVE_SETPWFILE */
+}
+
 static Entry *
-pw2entry( Backend *be, struct passwd *pw, char *rdn )
+pw2entry( Backend *be, struct passwd *pw, const char **text )
 {
+       size_t pwlen;
        Entry           *e;
-       char            buf[256];
-       struct berval   val;
-       struct berval   *vals[2];
+       struct berval   vals[2];
+       struct berval   bv;
 
        int rc;
-       const char *text;
 
-       AttributeDescription *ad_objectClass = NULL;
+       AttributeDescription *ad_objectClass = slap_schema.si_ad_objectClass;
        AttributeDescription *ad_cn = NULL;
        AttributeDescription *ad_sn = NULL;
        AttributeDescription *ad_uid = NULL;
        AttributeDescription *ad_description = NULL;
 
-       rc = slap_str2ad( "objectClass", &ad_objectClass, &text );
-
+       rc = slap_str2ad( "cn", &ad_cn, text );
        if(rc != LDAP_SUCCESS) return NULL;
-       rc = slap_str2ad( "cn", &ad_cn, &text );
+       rc = slap_str2ad( "sn", &ad_sn, text );
        if(rc != LDAP_SUCCESS) return NULL;
-       rc = slap_str2ad( "sn", &ad_sn, &text );
+       rc = slap_str2ad( "uid", &ad_uid, text );
        if(rc != LDAP_SUCCESS) return NULL;
-       rc = slap_str2ad( "uid", &ad_uid, &text );
+       rc = slap_str2ad( "description", &ad_description, text );
        if(rc != LDAP_SUCCESS) return NULL;
-       rc = slap_str2ad( "description", &ad_description, &text );
-       if(rc != LDAP_SUCCESS) return NULL;
-
-
-       vals[0] = &val;
-       vals[1] = NULL;
 
        /*
         * from pw we get pw_name and make it cn
         * give it an objectclass of person.
         */
 
+       pwlen = strlen( pw->pw_name );
+       vals[0].bv_len = (sizeof("uid=,")-1) + ( pwlen + be->be_suffix[0].bv_len );
+       vals[0].bv_val = ch_malloc( vals[0].bv_len + 1 );
+
+       /* rdn attribute type should be a configuratable item */
+       sprintf( vals[0].bv_val, "uid=%s,%s",
+               pw->pw_name, be->be_suffix[0].bv_val );
+
+       rc = dnNormalize2( NULL, vals, &bv );
+       if( rc != LDAP_SUCCESS ) {
+               free( vals[0].bv_val );
+               return NULL;
+       }
+
        e = (Entry *) ch_calloc( 1, sizeof(Entry) );
+       e->e_name = vals[0];
+       e->e_nname = bv;
+
        e->e_attrs = NULL;
 
-       /* objectclasses should be configuratable items */
-       val.bv_val = "top";
-       val.bv_len = sizeof("top")-1;
-       attr_merge( e, ad_objectClass, vals );
+       vals[1].bv_val = NULL;
 
-       val.bv_val = "person";
-       val.bv_len = sizeof("person")-1;
+       /* objectclasses should be configurable items */
+       vals[0].bv_val = "top";
+       vals[0].bv_len = sizeof("top")-1;
        attr_merge( e, ad_objectClass, vals );
 
-       val.bv_val = "uidObject";
-       val.bv_len = sizeof("uidObject")-1;
+       vals[0].bv_val = "person";
+       vals[0].bv_len = sizeof("person")-1;
        attr_merge( e, ad_objectClass, vals );
 
-       /* rdn attribute type should be a configuratable item */
-       sprintf( buf, "uid=%s,%s", pw->pw_name, be->be_suffix[0] );
-       e->e_dn = ch_strdup( buf );
-       e->e_ndn = ch_strdup( buf );
-       (void) dn_normalize( e->e_ndn );
+       vals[0].bv_val = "uidObject";
+       vals[0].bv_len = sizeof("uidObject")-1;
+       attr_merge( e, ad_objectClass, vals );
 
-       val.bv_val = pw->pw_name;
-       val.bv_len = strlen( pw->pw_name );
+       vals[0].bv_val = pw->pw_name;
+       vals[0].bv_len = pwlen;
        attr_merge( e, ad_uid, vals );  /* required by uidObject */
        attr_merge( e, ad_cn, vals );   /* required by person */
        attr_merge( e, ad_sn, vals );   /* required by person */
@@ -315,29 +331,36 @@ pw2entry( Backend *be, struct passwd *pw, char *rdn )
        if (pw->pw_gecos[0]) {
                char *s;
 
-               val.bv_val = pw->pw_gecos;
-               val.bv_len = strlen(val.bv_val);
+               vals[0].bv_val = pw->pw_gecos;
+               vals[0].bv_len = strlen(vals[0].bv_val);
                attr_merge(e, ad_description, vals);
 
-               s = strchr(val.bv_val, ',');
-               if (s)
-                       *s = '\0';
-               s = strchr(val.bv_val, '&');
+               s = strchr(vals[0].bv_val, ',');
+               if (s) *s = '\0';
+
+               s = strchr(vals[0].bv_val, '&');
                if (s) {
-                       int i = s - val.bv_val;
-                       strncpy(buf, val.bv_val, i);
-                       s = buf+i;
-                       strcpy(s, pw->pw_name);
-                       *s = TOUPPER(*s);
-                       strcat(s, val.bv_val+i+1);
-                       val.bv_val = buf;
+                       char buf[1024];
+
+                       if( vals[0].bv_len + pwlen < sizeof(buf) ) {
+                               int i = s - vals[0].bv_val;
+                               strncpy(buf, vals[0].bv_val, i);
+                               s = buf+i;
+                               strcpy(s, pw->pw_name);
+                               *s = TOUPPER((unsigned char)*s);
+                               strcat(s, vals[0].bv_val+i+1);
+                               vals[0].bv_val = buf;
+                       }
                }
-               val.bv_len = strlen(val.bv_val);
-               if ( strcmp( val.bv_val, pw->pw_name ))
+               vals[0].bv_len = strlen(vals[0].bv_val);
+
+               if ( vals[0].bv_len && strcasecmp( vals[0].bv_val, pw->pw_name )) {
                        attr_merge( e, ad_cn, vals );
-               if ( (s=strrchr(val.bv_val, ' '))) {
-                       val.bv_val = s + 1;
-                       val.bv_len = strlen(val.bv_val);
+               }
+
+               if ( (s=strrchr(vals[0].bv_val, ' '))) {
+                       vals[0].bv_val = s + 1;
+                       vals[0].bv_len = strlen(vals[0].bv_val);
                        attr_merge(e, ad_sn, vals);
                }
        }