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;
}