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 &&