Mailing-List: contact apache-bugdb-help@apache.org; run by ezmlm
Precedence: bulk
Reply-To: apache-bugdb@apache.org
Errors-To: <wrowe@rowe-clan.net>
From: "William A. Rowe, Jr." <wrowe@rowe-clan.net>
To: <apache-bugdb@apache.org>, <petter.mahlen@chello.se>
Subject: RE: config/6771: Solution suggestion to problem 6705
Date: Tue, 31 Oct 2000 17:18:39 -0600
Message-ID: <002501c04390$e6c621f0$92c0b0d0@roweclan.net>
MIME-Version: 1.0
Content-Type: text/plain;
	charset="iso-8859-1"
Content-Transfer-Encoding: 8bit
Importance: Normal
In-Reply-To: <20001031225509.32576.qmail@locus.apache.org>

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