From: Paul J. Reder <rederpj@apache.org>
Date: Fri, 30 Nov 2007 23:51:48 +0000 (+0000)
Subject: Ooops. Not sure why this didn't get deleted in the last commit to this file. This
X-Git-Tag: 2.3.0~1190
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=4ae49c0ce4e2476753ad12006bafe543c1f9d6ed;p=thirdparty%2Fapache%2Fhttpd.git

Ooops. Not sure why this didn't get deleted in the last commit to this file. This
is no longer required due to the more optimal way the list of subgroup attributes
is now handled.


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@600017 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/modules/aaa/mod_authnz_ldap.c b/modules/aaa/mod_authnz_ldap.c
index f5e0f3739a8..a95bbaabe04 100644
--- a/modules/aaa/mod_authnz_ldap.c
+++ b/modules/aaa/mod_authnz_ldap.c
@@ -661,24 +661,6 @@ static authz_status ldapgroup_check_authorization(request_rec *r,
 #endif
     }
 
-    /*
-     * If there are no elements in the sub group attribute array, the default
-     * should be member and uniquemember; populate the array now.
-     */
-    if (sec->subgroupattrs->nelts == 0) {
-        struct mod_auth_ldap_groupattr_entry_t *grp;
-#if APR_HAS_THREADS
-        apr_thread_mutex_lock(sec->lock);
-#endif
-        grp = apr_array_push(sec->subgroupattrs);
-        grp->name = "member";
-        grp = apr_array_push(sec->subgroupattrs);
-        grp->name = "uniqueMember";
-#if APR_HAS_THREADS
-        apr_thread_mutex_unlock(sec->lock);
-#endif
-    }
-
     /*
      * If there are no elements in the sub group classes array, the default
      * should be groupOfNames and groupOfUniqueNames; populate the array now.