Mailing-List: contact cvs-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
Date: 13 May 2003 15:19:30 -0000
Message-ID: <20030513151930.99041.qmail@icarus.apache.org>
From: slive@apache.org
To: apache-1.3-cvs@apache.org
Subject: 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);