public inbox for glibc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug libc/17585] New: locale memmem namespace
@ 2014-11-11 22:01 jsm28 at gcc dot gnu.org
2014-11-12 22:42 ` [Bug libc/17585] " cvs-commit at gcc dot gnu.org
2014-11-12 22:52 ` jsm28 at gcc dot gnu.org
0 siblings, 2 replies; 3+ messages in thread
From: jsm28 at gcc dot gnu.org @ 2014-11-11 22:01 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=17585
Bug ID: 17585
Summary: locale memmem namespace
Product: glibc
Version: 2.21
Status: NEW
Severity: normal
Priority: P2
Component: libc
Assignee: unassigned at sourceware dot org
Reporter: jsm28 at gcc dot gnu.org
CC: drepper.fsp at gmail dot com
Locale code, brought in by ISO C functions, calls memmem, which is not an ISO C
function. This isn't an ISO C conformance bug, because all mem* names are
reserved, but glibc practice is not to rely on that reservation (thus, memmem
is only declared in string.h if __USE_GNU even though ISO C would allow it to
be declared unconditionally, for example).
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Bug libc/17585] locale memmem namespace
2014-11-11 22:01 [Bug libc/17585] New: locale memmem namespace jsm28 at gcc dot gnu.org
@ 2014-11-12 22:42 ` cvs-commit at gcc dot gnu.org
2014-11-12 22:52 ` jsm28 at gcc dot gnu.org
1 sibling, 0 replies; 3+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2014-11-12 22:42 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=17585
--- Comment #1 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".
The branch, master has been updated
via 9975e3d37d8585b41db075321bade3495cf58cdb (commit)
from c4eae75271734f820a7477dbce33f8752af6f003 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=9975e3d37d8585b41db075321bade3495cf58cdb
commit 9975e3d37d8585b41db075321bade3495cf58cdb
Author: Joseph Myers <joseph@codesourcery.com>
Date: Wed Nov 12 22:41:03 2014 +0000
Fix locale memmem namespace (bug 17585).
Locale code, brought in by ISO C functions, calls memmem, which is not
an ISO C function. This isn't an ISO C conformance bug, because all
mem* names are reserved, but glibc practice is not to rely on that
reservation (thus, memmem is only declared in string.h if __USE_GNU
even though ISO C would allow it to be declared unconditionally, for
example). This patch changes that code to use __memmem.
Note: there are uses of memmem elsewhere in glibc that I didn't
change, although it may turn out some of those also need to use
__memmem.
Tested for x86_64 (testsuite, and that disassembly of installed shared
libraries is unchanged by this patch).
[BZ #17585]
* string/memmem.c [!_LIBC] (__memmem): Define to memmem.
(memmem): Rename to __memmem and define as weak alias of
__memmem. Use libc_hidden_weak.
(__memmem): Use libc_hidden_def.
* include/string.h (__memmem): Declare. Use libc_hidden_proto.
* locale/findlocale.c (valid_locale_name): Use __memmem instead of
memmem.
-----------------------------------------------------------------------
Summary of changes:
ChangeLog | 9 +++++++++
NEWS | 2 +-
include/string.h | 2 ++
locale/findlocale.c | 4 ++--
string/memmem.c | 9 ++++++---
5 files changed, 20 insertions(+), 6 deletions(-)
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Bug libc/17585] locale memmem namespace
2014-11-11 22:01 [Bug libc/17585] New: locale memmem namespace jsm28 at gcc dot gnu.org
2014-11-12 22:42 ` [Bug libc/17585] " cvs-commit at gcc dot gnu.org
@ 2014-11-12 22:52 ` jsm28 at gcc dot gnu.org
1 sibling, 0 replies; 3+ messages in thread
From: jsm28 at gcc dot gnu.org @ 2014-11-12 22:52 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=17585
Joseph Myers <jsm28 at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--- Comment #2 from Joseph Myers <jsm28 at gcc dot gnu.org> ---
Fixed for 2.21.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-11-12 22:52 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-11-11 22:01 [Bug libc/17585] New: locale memmem namespace jsm28 at gcc dot gnu.org
2014-11-12 22:42 ` [Bug libc/17585] " cvs-commit at gcc dot gnu.org
2014-11-12 22:52 ` jsm28 at gcc dot gnu.org
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).