Fix strtoll / strtoull namespace for 32-bit (bug 17594).
authorJoseph Myers <joseph@codesourcery.com>
Thu, 13 Nov 2014 19:50:55 +0000 (19:50 +0000)
committerJoseph Myers <joseph@codesourcery.com>
Thu, 13 Nov 2014 19:50:55 +0000 (19:50 +0000)
commit2a1cfd94a26e0fa96d8f284897213b65c60142d6
tree4ea3d0832469c75ddf3cc5d147f96a15e6c98868
parent20ed0d266f2c04f5612e2cedf02ede0d0c152e73
Fix strtoll / strtoull namespace for 32-bit (bug 17594).

For 32-bit platforms, strtoll and strtoull are strong symbols in libc,
but they are not in ISO C90, and are brought in by references to
__strtoll_internal / __strtoull_internal from scanf.  (For 64-bit
platforms, they are properly weak.)  This patch makes them weak for
32-bit (it has a side-effect of making other symbols weak that don't
need to be weak, such as strtol, but that's harmless).

Tested for x86 (testsuite, and that the disassembly of installed
shared libraries is unchanged by the patch).  This fixes all 120
unXFAILed FAILs of the new linknamespace tests seen for x86 (in fact,
there are now seven XPASSes of those tests for x86

XPASS: conform/POSIX2008/fcntl.h/linknamespace
XPASS: conform/UNIX98/libgen.h/linknamespace
XPASS: conform/XOPEN2K/fcntl.h/linknamespace
XPASS: conform/XOPEN2K/libgen.h/linknamespace
XPASS: conform/XOPEN2K8/fcntl.h/linknamespace
XPASS: conform/XOPEN2K8/libgen.h/linknamespace
XPASS: conform/XPG4/libgen.h/linknamespace

so suggesting that the failures seen for those on x86_64 are in some
way architecture-specific or 64-bit-specific).

[BZ #17594]
* stdlib/strtol.c (SYM__): New macro.
(SYM__1): Likewise.
(__strtol): Likewise.
(strtol): Rename to __strtol and define as weak alias of
__strtol.  Use libc_hidden_weak.
ChangeLog
NEWS
stdlib/strtol.c