You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by ja...@apache.org on 2012/11/20 22:03:53 UTC

svn commit: r1411862 - /httpd/httpd/trunk/modules/aaa/mod_authz_groupfile.c

Author: jailletc36
Date: Tue Nov 20 21:03:52 2012
New Revision: 1411862

URL: http://svn.apache.org/viewvc?rev=1411862&view=rev
Log:
Use apr_is_empty_table() instead of getting a table and checking the value of the 'nelts' field.

Modified:
    httpd/httpd/trunk/modules/aaa/mod_authz_groupfile.c

Modified: httpd/httpd/trunk/modules/aaa/mod_authz_groupfile.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/aaa/mod_authz_groupfile.c?rev=1411862&r1=1411861&r2=1411862&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/aaa/mod_authz_groupfile.c (original)
+++ httpd/httpd/trunk/modules/aaa/mod_authz_groupfile.c Tue Nov 20 21:03:52 2012
@@ -176,7 +176,7 @@ static authz_status group_check_authoriz
         return AUTHZ_DENIED;
     }
 
-    if (apr_table_elts(grpstatus)->nelts == 0) {
+    if (apr_is_empty_table(grpstatus)) {
         /* no groups available, so exit immediately */
         ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r, APLOGNO(01666)
                       "Authorization of user %s to access %s failed, reason: "
@@ -235,7 +235,7 @@ static authz_status filegroup_check_auth
         return AUTHZ_DENIED;
     }
 
-    if (apr_table_elts(grpstatus)->nelts == 0) {
+    if (apr_is_empty_table(grpstatus)) {
         /* no groups available, so exit immediately */
         ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r, APLOGNO(01670)
                         "Authorization of user %s to access %s failed, reason: "