You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by sl...@apache.org on 2003/05/13 17:19:30 UTC
cvs commit: apache-1.3/src/main http_request.c
slive 2003/05/13 08:19:30
Modified: src/main http_request.c
Log:
Reverse an earlier decision not to log EACESS results. This will
result in multiple log-entries when requesting a directory where
the DirectoryIndex contains multiple entires, but it should eliminate
a major source of confusion which results from a 403 error with
no error log entry.
The fix was pointed out by Jeff; I just adjusted the error message.
Reviewed by: Jeff Trawick, Rich Bowen
Revision Changes Path
1.169 +7 -2 apache-1.3/src/main/http_request.c
Index: http_request.c
===================================================================
RCS file: /home/cvs/apache-1.3/src/main/http_request.c,v
retrieving revision 1.168
retrieving revision 1.169
diff -u -d -b -u -r1.168 -r1.169
--- http_request.c 3 Feb 2003 17:13:23 -0000 1.168
+++ http_request.c 13 May 2003 15:19:30 -0000 1.169
@@ -288,7 +288,12 @@
}
else {
#if defined(EACCES)
- if (errno != EACCES)
+ if (errno == EACCES)
+ ap_log_rerror(APLOG_MARK, APLOG_ERR, r,
+ "access to %s failed because search "
+ "permissions are missing on a component "
+ "of the path", r->uri);
+ else
#endif
ap_log_rerror(APLOG_MARK, APLOG_ERR, r,
"access to %s failed", r->uri);