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 f5e0f3739a8338ee427ca29cd0d0882337c5e78c..a95bbaabe0440ebb5afef83f1aadef994a953d80 100644 (file)
--- 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.