Message-ID: <35AB94EF.5433C030@workfire.com>
Date: Tue, 14 Jul 1998 10:27:11 -0700
From: snyland@workfire.com (snyland@workfire.com)
MIME-Version: 1.0
To: new-httpd@apache.org
Subject: Re: [BUG] Win32 assert: sub_canonical_filename() called with
 szFile==""
References: <Pine.BSF.3.95q.980714104039.21195F-100000@valis.worldgate.com>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Sender: new-httpd-owner@apache.org
Precedence: bulk
Reply-To: new-httpd@apache.org


Marc Slemko wrote:

> On Tue, 14 Jul 1998, Martin Kraemer wrote:
>
> > A colleague of mine just tried a compilation of today's apache-1.3.1-dev
> > under WinNT. He reports no major compilation problems, but
> > runs into an assertion in os/win32/util_win32.c line 24 because
> > sub_canonical_filename() -after being called several times successfully-
> > is finally called with an argument of szFile == "" (empty string).
>
> The problem is due to having a path in the form "/" or "c:/" in your
> config file in a directory section or something.  Remove that and you
> should be happier.
>

Confirmed.  The default access.conf has a directory section <Directory />.
This runs into problems in ap_os_canonical_filename where the code now blows
away trailing slashes.

// Finally, a trailing slash(es) screws thing, so blow them away
    for(nSlashes=0 ; d > b2 && d[-1] == '\\' ; --d,++nSlashes)
 ;
    /* XXXX this breaks '/' and 'c:/' cases */
    *d='\0';

> Yes, it is in the default config file.  That is one of the things that has
> to be fixed before 1.3.1.

 Yes please.  This will seem very serious & incomprehensible to a lot of Win32
users.