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

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/1.3.x@99781 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/src/main/http_request.c b/src/main/http_request.c
index f0b2f6a6cbae31f433bba0ca3cfe807faf6529d1..64585ff3a05376dd877458f67e65d37bf54b93c7 100644 (file)
--- a/src/main/http_request.c
+++ b/src/main/http_request.c
@@ -288,8 +288,13 @@ static int get_path_info(request_rec *r)
         }
         else {
 #if defined(EACCES)
-            if (errno != EACCES)
-#endif
+            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);
             return HTTP_FORBIDDEN;