Re: bug#10472: [PATCH] canonicalize: fix // handling

2012-02-08 Thread Eric Blake
On 02/08/2012 03:13 AM, Pádraig Brady wrote: >> From d1f3998942236194f1894c45804ec947d07ed134 Mon Sep 17 00:00:00 2001 >> From: Eric Blake >> Date: Sat, 4 Feb 2012 11:11:40 -0700 >> Subject: [PATCH] canonicalize: avoid uninitialized memory use >> >> When DOUBLE_SLASH_IS_DISTINCT_ROOT is non-zero,

Re: bug#10472: [PATCH] canonicalize: fix // handling

2012-02-08 Thread Pádraig Brady
On 02/04/2012 06:38 PM, Eric Blake wrote: > On 02/04/2012 10:59 AM, Eric Blake wrote: >> On 02/04/2012 09:56 AM, Eric Blake wrote: >>> On Cygwin, and other platforms where // is detected as distinct >>> from / at configure time, the canonicalize routines were incorrectly >>> treating all instances

Re: bug#10472: [PATCH] canonicalize: fix // handling

2012-02-04 Thread Eric Blake
On 02/04/2012 10:59 AM, Eric Blake wrote: > On 02/04/2012 09:56 AM, Eric Blake wrote: >> On Cygwin, and other platforms where // is detected as distinct >> from / at configure time, the canonicalize routines were incorrectly >> treating all instances of multiple leading slashes as //. >> See also c

Re: bug#10472: [PATCH] canonicalize: fix // handling

2012-02-04 Thread Eric Blake
On 02/04/2012 09:56 AM, Eric Blake wrote: > On Cygwin, and other platforms where // is detected as distinct > from / at configure time, the canonicalize routines were incorrectly > treating all instances of multiple leading slashes as //. > See also coreutils bug http://debbugs.gnu.org/10472 > > *

Re: bug#10472: [PATCH] canonicalize: fix // handling

2012-02-04 Thread Bruno Haible
Eric Blake wrote: > canonicalize-lgpl had the same bug Please, feel free to fix it there as well. Bruno

Re: bug#10472: [PATCH] canonicalize: fix // handling

2012-02-04 Thread Eric Blake
On 02/04/2012 09:56 AM, Eric Blake wrote: > On Cygwin, and other platforms where // is detected as distinct > from / at configure time, the canonicalize routines were incorrectly > treating all instances of multiple leading slashes as //. > See also coreutils bug http://debbugs.gnu.org/10472 > > *

[PATCH] canonicalize: fix // handling

2012-02-04 Thread Eric Blake
On Cygwin, and other platforms where // is detected as distinct from / at configure time, the canonicalize routines were incorrectly treating all instances of multiple leading slashes as //. See also coreutils bug http://debbugs.gnu.org/10472 * lib/canonicalize.c (canonicalize_filename_mode): Don'