On 10/27/2015 11:30 AM, Samuel Thibault wrote:
> Hello,
>
> Ludovic Courtès, le Tue 22 Sep 2015 17:27:55 +0200, a écrit :
>> 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 chang
On 10/13/2015 10:45 AM, Ludovic Courtès wrote:
>> - What does localedef --list-archive return?
>>
>> - The new LC_COLLATE format will make it's way into the binary locale
>> archive
>> and that means glibc can't read the locale-archive? Does it fail? exit
>> code?
>
> The patch does not ch
Samuel Thibault skribis:
> Ludovic Courtès, le Tue 22 Sep 2015 17:27:55 +0200, a écrit :
>> 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 EI
Hello,
Ludovic Courtès, le Tue 22 Sep 2015 17:27:55 +0200, a écrit :
> 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.
"Carlos O'Donell" skribis:
> On 09/29/2015 04:08 AM, Ludovic Courtès wrote:
>> "Carlos O'Donell" skribis:
[...]
>> Apart from that, ‘localedef --list-archive’ simply opens the locale
>> archive (typically /usr/lib/locale/locale-archive, regardless of the
>> ‘LOCPATH’ environment variable value
On 09/29/2015 04:08 AM, Ludovic Courtès wrote:
> "Carlos O'Donell" skribis:
>
>> On 09/26/2015 06:24 AM, Ludovic Courtès wrote:
>>> Furthermore, the function in question returns EINVAL in other similar
>>> cases–e.g., when libc 2.22 loads LC_COLLATE data from libc 2.21.
>>
>> If you change this p
On 10/12/2015 08:49 PM, Allan McRae wrote:
> On 29/09/15 06:54, Carlos O'Donell wrote:
>> On 09/26/2015 06:24 AM, Ludovic Courtès wrote:
>>> Furthermore, the function in question returns EINVAL in other similar
>>> cases–e.g., when libc 2.22 loads LC_COLLATE data from libc 2.21.
>>
>> If you change
Allan McRae skribis:
> On 29/09/15 06:54, Carlos O'Donell wrote:
>> On 09/26/2015 06:24 AM, Ludovic Courtès wrote:
>>> Furthermore, the function in question returns EINVAL in other similar
>>> cases–e.g., when libc 2.22 loads LC_COLLATE data from libc 2.21.
>>
>> If you change this particular ca
On 29/09/15 06:54, Carlos O'Donell wrote:
> On 09/26/2015 06:24 AM, Ludovic Courtès wrote:
>> Furthermore, the function in question returns EINVAL in other similar
>> cases–e.g., when libc 2.22 loads LC_COLLATE data from libc 2.21.
>
> If you change this particular case to EINVAL, what does the us
Hello!
A friendly ping so the discussion does not die out. :-)
https://sourceware.org/ml/libc-alpha/2015-09/threads.html#00575
https://sourceware.org/ml/libc-alpha/2015-09/msg00727.html
TIA,
Ludo’.
"Carlos O'Donell" skribis:
> On 09/26/2015 06:24 AM, Ludovic Courtès wrote:
>> Furthermore, the function in question returns EINVAL in other similar
>> cases–e.g., when libc 2.22 loads LC_COLLATE data from libc 2.21.
>
> If you change this particular case to EINVAL, what does the user see
> as a
On 09/26/2015 06:24 AM, Ludovic Courtès wrote:
> Furthermore, the function in question returns EINVAL in other similar
> cases–e.g., when libc 2.22 loads LC_COLLATE data from libc 2.21.
If you change this particular case to EINVAL, what does the user see
as a result of this change? Do they get a n
"Carlos O'Donell" skribis:
> Despite Roland saying "LGTM", I think this is not a good change.
>
> Firstly, it's not the community consensus as Ondrej is pointing out.
>
> https://sourceware.org/glibc/wiki/Style_and_Conventions#Error_Handling
“Assertions are for internal consistency checking only
On 09/24/2015 12:12 PM, Ludovic Courtès wrote:
> Ondřej, I think we have been miscommunicating.
>
> I noticed that a program linked against 2.21 or earlier would abort with
> an assertion failure when it stumbles upon 2.22 locale data.
>
> All the patch tries to do is change the abort to EINVAL (
Mark H Weaver skribis:
> [Removed libc-alpha from the CC list]
>
> l...@gnu.org (Ludovic Courtès) writes:
>
>> 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:
>>
>> loa
Ondřej, I think we have been miscommunicating.
I noticed that a program linked against 2.21 or earlier would abort with
an assertion failure when it stumbles upon 2.22 locale data.
All the patch tries to do is change the abort to EINVAL (and skip locale
data) when that happens.
I’m not claiming
On Wed, Sep 23, 2015 at 11:45:52PM +0200, Ludovic Courtès wrote:
> Ondřej Bílka skribis:
>
> > On Tue, Sep 22, 2015 at 11:22:40PM +0200, Ludovic Courtès wrote:
> >> Ondřej Bílka skribis:
> >>
> >> > On Tue, Sep 22, 2015 at 05:27:55PM +0200, Ludovic Courtès wrote:
> >> >> With libc 2.22 people a
[Removed libc-alpha from the CC list]
l...@gnu.org (Ludovic Courtès) writes:
> 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_dat
Ondřej Bílka skribis:
> On Tue, Sep 22, 2015 at 11:22:40PM +0200, Ludovic Courtès wrote:
>> Ondřej Bílka 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 loca
Andreas Schwab writes:
> What about the other assertion?
Ignore that. This one is a real internal consistency check.
Andreas.
--
Andreas Schwab, SUSE Labs, sch...@suse.de
GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7
"And now for something completely different."
l...@gnu.org (Ludovic Courtès) writes:
> diff --git a/locale/loadlocale.c b/locale/loadlocale.c
> index fdba6e9..e04e720 100644
> --- a/locale/loadlocale.c
> +++ b/locale/loadlocale.c
> @@ -122,8 +122,9 @@ _nl_intern_locale_data (int category, const void *data,
> size_t datasize)
> {
> #de
On Tue, Sep 22, 2015 at 11:22:40PM +0200, Ludovic Courtès wrote:
> Ondřej Bílka 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
Ondřej Bílka 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
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
Looks OK to me.
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_COL
26 matches
Mail list logo