You are viewing a plain text version of this content. The canonical link for it is here.
Posted to apache-bugdb@apache.org by "William A. Rowe, Jr." <wr...@rowe-clan.net> on 2000/11/01 00:18:39 UTC

RE: config/6771: Solution suggestion to problem 6705

> From: Petter Måhlén [mailto:petter.mahlen@chello.se]
> Sent: Tuesday, October 31, 2000 4:55 PM
>
> >Fix:
> two options:
> 1. change the ap_os_is_path_absolute() function to:
>   return file && (file[0] == '/' || (file[1] == ':' && (file[2] == '/' || file[2] == '\')));
> In fact, even better would be something like
> isDirectorySeparator(file[2]).

Unacceptable, since Apache uses ap_os_is_path_absolute -within- the server,
as well as to parse the httpd.conf.

> 2. fix the JServ installation program to use / instead of \

A very good idea, IMHO.

The real fix, which will I will commit this week, is to add a call through
ap_canonicalize_filename to all of the places where the path is parsed from
httpd.conf.  Thank you very much, for your analysis and the suggestions!