Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 10 Jun 2016 15:19:43 +0300
From:      Andrey Chernov <ache@freebsd.org>
To:        Pedro Giffuni <pfg@FreeBSD.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r301777 - head/lib/libc/stdio
Message-ID:  <52d3a4b3-0b52-abd3-8633-909885f12781@freebsd.org>
In-Reply-To: <cd74f302-f089-3208-a6ca-89eb7460d41b@FreeBSD.org>
References:  <201606100521.u5A5Lqa6035072@repo.freebsd.org> <3346cab5-d713-c1ad-5e50-ea8d50ee47b9@freebsd.org> <cd74f302-f089-3208-a6ca-89eb7460d41b@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 10.06.2016 15:03, Pedro Giffuni wrote:
> (Sorry I have been offline/sleep for a while)
> 
> Hello;
> 
> I cannot back this out because of this report:
> 
> http://docs.freebsd.org/cgi/mid.cgi?20160608132224.GA15454
> 
> This change only re-states what has been working before the ABI breakage
> fix in r301461.

Because now strcoll_l() is used as before and it is broken in wide char
converting process. It is broken for _all_, not just for vfscanf()
ranges. It should never fail with any args.
Real fix should be in strcoll_l().

> The fix to regex is completely independent.

It is dependent. vfscanf() and regcomp() use the same code for range
collation. If we remove range collation from everywhere, we save
vfscanf(), but strcoll_l() remains broken.




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?52d3a4b3-0b52-abd3-8633-909885f12781>