[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Gracefully handle incompatible locale data
From: |
Ondřej Bílka |
Subject: |
Re: [PATCH] Gracefully handle incompatible locale data |
Date: |
Tue, 22 Sep 2015 23:50:22 +0200 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
On Tue, Sep 22, 2015 at 11:22:40PM +0200, Ludovic Courtès wrote:
> Ondřej Bílka <address@hidden> skribis:
>
> > On Tue, Sep 22, 2015 at 05:27:55PM +0200, Ludovic Courtès wrote:
> >> With libc 2.22 people are starting to realize that libc does not
> >> guarantee that it can load locale data built with another libc version,
> >> but they learn it the hard way:
> >>
> >> loadlocale.c:130: _nl_intern_locale_data: Assertion `cnt < (sizeof
> >> (_nl_value_type_LC_COLLATE) / sizeof (_nl_value_type_LC_COLLATE[0]))'
> >> failed.
> >>
> >> This patch changes such conditions to return EINVAL instead of aborting.
> >>
> >> WDYT?
> >>
> > While that assert is quite cryptic I dont see why just returning EINVAL is
> > better. How do you distinguish that its wrong locale version versus not
> > installed?
>
> The rest of this function already returns EINVAL when something is
> fishy. This patch makes the behavior more consistent.
>
Then I take that back. But I don't see how this is reliable assertion to
detect different libc version. So could you as followup patch add
version field and check that instead this assert?
- [PATCH] Gracefully handle incompatible locale data, Ludovic Courtès, 2015/09/22
- Re: [PATCH] Gracefully handle incompatible locale data, Roland McGrath, 2015/09/22
- Re: [PATCH] Gracefully handle incompatible locale data, Ondřej Bílka, 2015/09/22
- Re: [PATCH] Gracefully handle incompatible locale data, Ludovic Courtès, 2015/09/22
- Re: [PATCH] Gracefully handle incompatible locale data,
Ondřej Bílka <=
- Re: [PATCH] Gracefully handle incompatible locale data, Ludovic Courtès, 2015/09/23
- Re: [PATCH] Gracefully handle incompatible locale data, Ondřej Bílka, 2015/09/24
- Re: [PATCH] Gracefully handle incompatible locale data, Ludovic Courtès, 2015/09/24
- Re: [PATCH] Gracefully handle incompatible locale data, Carlos O'Donell, 2015/09/25
- Re: [PATCH] Gracefully handle incompatible locale data, Ludovic Courtès, 2015/09/26
- Re: [PATCH] Gracefully handle incompatible locale data, Carlos O'Donell, 2015/09/29
Re: [PATCH] Gracefully handle incompatible locale data, Andreas Schwab, 2015/09/23
Re: [PATCH] Gracefully handle incompatible locale data, Mark H Weaver, 2015/09/23