[Fedora-directory-commits] mod_admserv mod_admserv.c,1.21,1.22

Richard Allen Megginson (rmeggins) fedora-directory-commits at redhat.com
Fri Mar 3 20:16:48 UTC 2006


Author: rmeggins

Update of /cvs/dirsec/mod_admserv
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv26683

Modified Files:
	mod_admserv.c 
Log Message:
Bug(s) fixed: 183925
Bug Description: nsAdminAccessAddresses not working
Reviewed by: one line commit rule
Fix Description: The comparison was reversed so that if the match is successful the access is allowed.
Platforms tested: Fedora Core 4
Flag Day: no
Doc impact: no
QA impact: should be covered by regular nightly and manual testing
New Tests integrated into TET: none



Index: mod_admserv.c
===================================================================
RCS file: /cvs/dirsec/mod_admserv/mod_admserv.c,v
retrieving revision 1.21
retrieving revision 1.22
diff -u -r1.21 -r1.22
--- mod_admserv.c	1 Mar 2006 18:43:51 -0000	1.21
+++ mod_admserv.c	3 Mar 2006 20:16:38 -0000	1.22
@@ -1913,7 +1913,7 @@
     if (accessAddresses && *accessAddresses) {
         int matchflags = APR_FNM_PERIOD;
         apr_status_t rc = admserv_match_list(apr_pstrdup(r->pool, accessAddresses), clientIP, matchflags);
-        if (rc == APR_SUCCESS) {
+        if (rc != APR_SUCCESS) {
         } else {
             return DECLINED;
         }




More information about the Fedora-directory-commits mailing list