From: William A. Rowe Jr <wrowe@apache.org>
Date: Sun, 21 Oct 2001 17:56:36 +0000 (+0000)
Subject:   Reading commit log messages can be very helpful for noting inconsistent
X-Git-Tag: 2.0.27~52
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=61f021ac41528c883ed83e1b3faa69e2198a1da6;p=apache

  Reading commit log messages can be very helpful for noting inconsistent
  comments... no code is hurt in this commit


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@91623 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/modules/mappers/mod_rewrite.c b/modules/mappers/mod_rewrite.c
index e35319a7d9..556c73f5bc 100644
--- a/modules/mappers/mod_rewrite.c
+++ b/modules/mappers/mod_rewrite.c
@@ -1507,7 +1507,7 @@ static int hook_fixup(request_rec *r)
                 r->filename = apr_pstrdup(r->pool, r->filename+12);
             }
 
-            /* the filename has to start with a slash! */
+            /* the filename must be an absolute path! */
             if (!ap_os_is_path_absolute(r->pool, r->filename)) {
                 return HTTP_BAD_REQUEST;
             }
@@ -1987,11 +1987,11 @@ static int apply_rewrite_rule(request_rec *r, rewriterule_entry *p,
     /*
      *   Again add the previously stripped per-directory location
      *   prefix if the new URI is not a new one for this
-     *   location, i.e. if it's not starting with either a slash
-     *   or a fully qualified URL scheme.
+     *   location, i.e. if it's not an absolute path nor
+     *   a fully qualified URL scheme.
      */
     if (prefixstrip && !ap_os_is_path_absolute(r->pool, r->filename)
-	&& !is_absolute_uri(r->filename)) {
+                    && !is_absolute_uri(r->filename)) {
         rewritelog(r, 3, "[per-dir %s] add per-dir prefix: %s -> %s%s",
                    perdir, r->filename, perdir, r->filename);
         r->filename = apr_pstrcat(r->pool, perdir, r->filename, NULL);
@@ -2073,8 +2073,8 @@ static int apply_rewrite_rule(request_rec *r, rewriterule_entry *p,
     /*
      *  Now we are sure it is not a fully qualified URL.  But
      *  there is still one special case left: A local rewrite in
-     *  per-directory context, i.e. a substitution URL which does
-     *  not start with a slash. Here we add again the initially
+     *  per-directory context, i.e. a substitution URL which is not
+     *  an absolute path. Here we add again the initially
      *  stripped per-directory prefix.
      */
     if (prefixstrip && !ap_os_is_path_absolute(r->pool, r->filename)) {