httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From wr...@apache.org
Subject cvs commit: httpd-2.0/modules/mappers mod_rewrite.c
Date Sun, 21 Oct 2001 17:56:36 GMT
wrowe       01/10/21 10:56:36

  Modified:    modules/mappers mod_rewrite.c
  Log:
    Reading commit log messages can be very helpful for noting inconsistent
    comments... no code is hurt in this commit
  
  Revision  Changes    Path
  1.85      +6 -6      httpd-2.0/modules/mappers/mod_rewrite.c
  
  Index: mod_rewrite.c
  ===================================================================
  RCS file: /home/cvs/httpd-2.0/modules/mappers/mod_rewrite.c,v
  retrieving revision 1.84
  retrieving revision 1.85
  diff -u -r1.84 -r1.85
  --- mod_rewrite.c	2001/10/21 17:48:21	1.84
  +++ mod_rewrite.c	2001/10/21 17:56:36	1.85
  @@ -1507,7 +1507,7 @@
                   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 @@
       /*
        *   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 @@
       /*
        *  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)) {
  
  
  

Mime
View raw message