Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 2 Dec 2015 23:37:43 +0100
From:      Baptiste Daroussin <bapt@FreeBSD.org>
To:        Dag-Erling =?iso-8859-1?Q?Sm=F8rgrav?= <des@des.no>
Cc:        svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, svn-src-stable-10@freebsd.org
Subject:   Re: svn commit: r287084 - in stable/10/usr.sbin/pw: . tests
Message-ID:  <20151202223743.GE20169@ivaldir.etoilebsd.net>
In-Reply-To: <20151202215958.GD20169@ivaldir.etoilebsd.net>
References:  <201508232142.t7NLgSXX033227@repo.freebsd.org> <867fkxcbq9.fsf@desk.des.no> <20151202215958.GD20169@ivaldir.etoilebsd.net>

next in thread | previous in thread | raw e-mail | index | archive | help

[-- Attachment #1 --]
On Wed, Dec 02, 2015 at 10:59:58PM +0100, Baptiste Daroussin wrote:
> On Wed, Dec 02, 2015 at 12:48:46PM +0100, Dag-Erling Smørgrav wrote:
> > Baptiste Daroussin <bapt@FreeBSD.org> writes:
> > > Log:
> > >   MFC: r285133,r285136,r285137,r285156,r285157,r285158,r285256,r285318,r285395,
> > >   r285396,r285398,r285401,r285403,r285405,r285406,r285408,r285409,r285411,
> > >   r285412,r285413,r285415,r285418,r285430,r285433,r285434,r285442,r285948,
> > >   r285984,r285985,r285989,r285996,r285997,r286045,r286047,r286066,r286150,
> > >   r286151,r286152,r286154,r286155,r286156,r286157,r286173,r286196,r286197,
> > >   r286198,r286199,r286200,r286201,r286202,r286203,r286204,r286210,r286211,
> > >   r286217,r286218,r286258,r286259,r286341,r286775,r286982,r286986,r286991,
> > >   r286993
> > >   
> > >   Validate most pw inputs.
> > 
> > This is a serious regression as the validation code is not locale-aware.
> > I would like to see it reverted and an EN issued.  Fixing it correctly
> > will be a major undertaking since it would needs to support multibyte
> > locales.  In the meantime, I will have to replace pw on my 10.2 systems
> > with the 10.1 binary.
> 
> Or a fix can be made, if you provide an example of the failing case, I would be
> able to fix it and add it to the regression tests.
> 
> Bapt

If that is what you were speaking about, it is fixed.
https://svnweb.freebsd.org/base?view=revision&revision=291658

Best regards,
Bapt

[-- Attachment #2 --]
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iEYEARECAAYFAlZfcrcACgkQ8kTtMUmk6Ex+pACfb+RciUJgii7U5/8xMaZA9GQo
IpAAnihYRbJ2puJHX0tpaHA5r+Zk7n4I
=PKDz
-----END PGP SIGNATURE-----

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20151202223743.GE20169>