You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by wr...@apache.org on 2007/12/11 15:23:22 UTC

svn commit: r603253 - /httpd/mod_ftp/trunk/modules/ftp/ftp_util.c

Author: wrowe
Date: Tue Dec 11 06:23:14 2007
New Revision: 603253

URL: http://svn.apache.org/viewvc?rev=603253&view=rev
Log:
Jim and I both reviewed this code, it's a remnant that's uninteresting
today now that we use fnmatch.


Modified:
    httpd/mod_ftp/trunk/modules/ftp/ftp_util.c

Modified: httpd/mod_ftp/trunk/modules/ftp/ftp_util.c
URL: http://svn.apache.org/viewvc/httpd/mod_ftp/trunk/modules/ftp/ftp_util.c?rev=603253&r1=603252&r2=603253&view=diff
==============================================================================
--- httpd/mod_ftp/trunk/modules/ftp/ftp_util.c (original)
+++ httpd/mod_ftp/trunk/modules/ftp/ftp_util.c Tue Dec 11 06:23:14 2007
@@ -190,15 +190,6 @@
 
     rr = ap_sub_req_lookup_file(name, r, NULL);
 
-    /* Another hack.  With BSD glob it is possible to return a file
-     * outside the current directory. (or document root for that matter).
-     * This was an oversight made when adding the GLOB_NOPERIOD option
-     * to BSD glob.  Parsing the uri and checking if NULL is returned
-     * will allow us to check if the current file is outside the docroot.
-     */
-    ap_parse_uri(rr, name);
-    ap_getparents(rr->uri);
-
     if ((rr->finfo.filetype != 0) && 
         ((rr->status == HTTP_OK) || (rr->status == HTTP_MOVED_PERMANENTLY) ||
          (rr->status == HTTP_UNAUTHORIZED &&