[Bug libc/11009] New: `nl_langinfo_l (item, LC_GLOBAL_LOCALE)' crashes

[Bug libc/11009] New: `nl_langinfo_l (item, LC_GLOBAL_LOCALE)' crashes

[ludo at gnu dot org]

As a followup to #10968, the attached example crashes with glibc <= 2.11.

-- 
           Summary: `nl_langinfo_l (item, LC_GLOBAL_LOCALE)' crashes
           Product: glibc
           Version: 2.9
            Status: NEW
          Severity: normal
          Priority: P2
         Component: libc
        AssignedTo: drepper at redhat dot com
        ReportedBy: ludo at gnu dot org
                CC: bruno at clisp dot org,glibc-bugs at sources dot redhat
                    dot com
 GCC build triplet: x86_64-unknown-linux-gnu
  GCC host triplet: x86_64-unknown-linux-gnu
GCC target triplet: x86_64-unknown-linux-gnu


http://sourceware.org/bugzilla/show_bug.cgi?id=11009

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/11009] `nl_langinfo_l (item, LC_GLOBAL_LOCALE)' crashes

[ludo at gnu dot org]

------- Additional Comments From ludo at gnu dot org  2009-11-23 22:41 -------
Created an attachment (id=4412)
 --> (http://sourceware.org/bugzilla/attachment.cgi?id=4412&action=view)
Test case.


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=11009

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/11009] `nl_langinfo_l (item, LC_GLOBAL_LOCALE)' crashes

[drepper at redhat dot com]

------- Additional Comments From drepper at redhat dot com  2009-11-23 22:47 -------
Not everything that is possible to do should be done.  Why on earth is this needed?

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |WAITING


http://sourceware.org/bugzilla/show_bug.cgi?id=11009

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/11009] `nl_langinfo_l (item, LC_GLOBAL_LOCALE)' crashes

[jakub at redhat dot com]

------- Additional Comments From jakub at redhat dot com  2009-11-23 23:01 -------
Note that locale.h description says that LC_GLOBAL_LOCALE is a special value for
use by the uselocale function.  There is nothing that says it can be used in
other functions.  If you want to query the global locale, why don't you just use
nl_langinfo instead of nl_langinfo_l?
You'd be requesting that isdigit_l (x, LC_GLOBAL_LOCALE) works next time, all
those macros/inclines then would need to special case that value.

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=11009

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/11009] `nl_langinfo_l (item, LC_GLOBAL_LOCALE)' crashes

[ludo at gnu dot org]

------- Additional Comments From ludo at gnu dot org  2009-11-23 23:44 -------
It's sometimes useful to have code that treats `LC_GLOBAL_LOCALE' like any other
locale object.

The wording at
http://www.opengroup.org/onlinepubs/9699919799/basedefs/locale.h.html suggests
that it's a "locale object descriptor", which should be usable with any function
that takes a locale object descriptor, not just uselocale(3).

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=11009

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/11009] `nl_langinfo_l (item, LC_GLOBAL_LOCALE)' crashes

[drepper at redhat dot com]

------- Additional Comments From drepper at redhat dot com  2009-11-23 23:48 -------
(In reply to comment #4)
> It's sometimes useful to have code that treats `LC_GLOBAL_LOCALE' like any other
> locale object.

It's complete nonsense in this case and prohibitively expensive in some other
cases.  I won't change this.  And since the specification is written after the
glibc implementation it has to be adjusted.



-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |RESOLVED
         Resolution|                            |WONTFIX


http://sourceware.org/bugzilla/show_bug.cgi?id=11009

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/11009] `nl_langinfo_l (item, LC_GLOBAL_LOCALE)' crashes

[bruno at clisp dot org]

------- Additional Comments From bruno at clisp dot org  2009-11-24 01:12 -------
(In reply to comment #3)
> If you want to query the global locale, why don't you just use
> nl_langinfo instead of nl_langinfo_l?

Note that the proposed

  nl_langinfo_l (item, LC_GLOBAL_LOCALE)

is not equivalent to

  nl_langinfo (item)

but rather to

  ({ locale_t old_thread_locale = uselocale (LC_GLOBAL_LOCALE);
     const char *value = nl_langinfo (item);
     uselocale (old_thread_locale);
     value;
  })

or to

  ({ locale_t global_locale = duplocale (LC_GLOBAL_LOCALE);
     const char *value = nl_langinfo_l (item, global_locale);
     freelocale (global_locale);
     value;
  })

But I agree with your evaluation that it is not essential to support
LC_GLOBAL_LOCALE arguments here (unlike in duplocale, see bug10969).


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=11009

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/11009] `nl_langinfo_l (item, LC_GLOBAL_LOCALE)' crashes

[fweimer at redhat dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=11009

Florian Weimer <fweimer at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |security-

-- 
You are receiving this mail because:
You are on the CC list for the bug.