summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2014-11-12 16:24:16 +0000
committerJoseph Myers <joseph@codesourcery.com>2014-11-12 16:24:16 +0000
commit2f5c1b000aaeb2c25b1df52453bba8de7f9453e2 (patch)
tree523dac74393fcf0967b6b7ec683848b4dfbed877
parent5a6fa4d7ed465c2f7da9d73004c972519dc2100e (diff)
downloadglibc-2f5c1b000aaeb2c25b1df52453bba8de7f9453e2.tar.gz
Fix __getcwd rewinddir namespace (bug 17584).
__getcwd is called from dcigettext.o (brought in by various ISO C functionality), but calls rewinddir, which is not an ISO C function. This patch makes __getcwd call __rewinddir instead and makes rewinddir a weak alias for __rewinddir. Since getcwd.c is shared with gnulib (albeit not merged in either direction for a long time, and omitted from gnulib's config/srclist.txt list of shared files) I put in a #ifndef _LIBC define of __rewinddir to rewinddir, although a future merged version of getcwd could end up looking significantly different. Tested for x86_64 (testsuite, and that disassembly of installed shared libraries is unchanged by this patch). [BZ #17584] * dirent/rewinddir.c (rewinddir): Rename to __rewinddir and define as weak alias of __rewinddir. Don't use libc_hidden_def. (__rewinddir): Use libc_hidden_def. * sysdeps/mach/hurd/rewinddir.c: Rename to __rewinddir and define as weak alias of __rewinddir. Don't use libc_hidden_def. (__rewinddir): Use libc_hidden_def. * sysdeps/posix/rewinddir.c: Rename to __rewinddir and define as weak alias of __rewinddir. Don't use libc_hidden_def. (__rewinddir): Use libc_hidden_def. * include/dirent.h (rewinddir): Don't use libc_hidden_proto. (__rewinddir): Use libc_hidden_proto. * sysdeps/posix/getcwd.c [!_LIBC] (__rewinddir): Define to rewinddir. (__getcwd): Use __rewinddir instead of rewinddir.
-rw-r--r--ChangeLog16
-rw-r--r--NEWS2
-rw-r--r--dirent/rewinddir.c5
-rw-r--r--include/dirent.h3
-rw-r--r--sysdeps/mach/hurd/rewinddir.c5
-rw-r--r--sysdeps/posix/getcwd.c6
-rw-r--r--sysdeps/posix/rewinddir.c5
7 files changed, 33 insertions, 9 deletions
diff --git a/ChangeLog b/ChangeLog
index ee9f3120df..42a3e828fc 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,21 @@
2014-11-12 Joseph Myers <joseph@codesourcery.com>
+ [BZ #17584]
+ * dirent/rewinddir.c (rewinddir): Rename to __rewinddir and define
+ as weak alias of __rewinddir. Don't use libc_hidden_def.
+ (__rewinddir): Use libc_hidden_def.
+ * sysdeps/mach/hurd/rewinddir.c: Rename to __rewinddir and define
+ as weak alias of __rewinddir. Don't use libc_hidden_def.
+ (__rewinddir): Use libc_hidden_def.
+ * sysdeps/posix/rewinddir.c: Rename to __rewinddir and define as
+ weak alias of __rewinddir. Don't use libc_hidden_def.
+ (__rewinddir): Use libc_hidden_def.
+ * include/dirent.h (rewinddir): Don't use libc_hidden_proto.
+ (__rewinddir): Use libc_hidden_proto.
+ * sysdeps/posix/getcwd.c [!_LIBC] (__rewinddir): Define to
+ rewinddir.
+ (__getcwd): Use __rewinddir instead of rewinddir.
+
[BZ #17583]
* libio/fileno.c (fileno): Rename to __fileno and define as weak
alias of __fileno. Use libc_hidden_weak.
diff --git a/NEWS b/NEWS
index 4727ea24cc..63b0a86fe6 100644
--- a/NEWS
+++ b/NEWS
@@ -11,7 +11,7 @@ Version 2.21
6652, 12926, 14132, 14138, 14171, 15215, 15884, 17266, 17344, 17363,
17370, 17371, 17411, 17460, 17475, 17485, 17501, 17506, 17508, 17522,
- 17555, 17583.
+ 17555, 17583, 17584.
* New locales: tu_IN, bh_IN.
diff --git a/dirent/rewinddir.c b/dirent/rewinddir.c
index 86d9fbd720..ba5b002ce0 100644
--- a/dirent/rewinddir.c
+++ b/dirent/rewinddir.c
@@ -22,13 +22,14 @@
/* Rewind DIRP to the beginning of the directory. */
void
-rewinddir (dirp)
+__rewinddir (dirp)
DIR *dirp;
{
__set_errno (ENOSYS);
/* No way to indicate failure. */
}
-libc_hidden_def (rewinddir)
+libc_hidden_def (__rewinddir)
+weak_alias (__rewinddir, rewinddir)
stub_warning (rewinddir)
diff --git a/include/dirent.h b/include/dirent.h
index 096a9771a1..e8e9e42de6 100644
--- a/include/dirent.h
+++ b/include/dirent.h
@@ -47,8 +47,9 @@ extern DIR *__alloc_dir (int fd, bool close_fd, int flags,
const struct stat64 *statp)
internal_function;
extern void __scandir_cancel_handler (void *arg);
+extern __typeof (rewinddir) __rewinddir;
-libc_hidden_proto (rewinddir)
+libc_hidden_proto (__rewinddir)
libc_hidden_proto (scandirat)
libc_hidden_proto (scandirat64)
# endif
diff --git a/sysdeps/mach/hurd/rewinddir.c b/sysdeps/mach/hurd/rewinddir.c
index bccd54909b..284a48bf1f 100644
--- a/sysdeps/mach/hurd/rewinddir.c
+++ b/sysdeps/mach/hurd/rewinddir.c
@@ -22,9 +22,10 @@
/* Rewind DIRP to the beginning of the directory. */
void
-rewinddir (dirp)
+__rewinddir (dirp)
DIR *dirp;
{
seekdir (dirp, (off_t) 0L);
}
-libc_hidden_def (rewinddir)
+libc_hidden_def (__rewinddir)
+weak_alias (__rewinddir, rewinddir)
diff --git a/sysdeps/posix/getcwd.c b/sysdeps/posix/getcwd.c
index 6201916712..92cb22ef33 100644
--- a/sysdeps/posix/getcwd.c
+++ b/sysdeps/posix/getcwd.c
@@ -195,6 +195,10 @@ extern char *alloca ();
#ifndef __GNU_LIBRARY__
# define __lstat64 stat64
#endif
+
+#ifndef _LIBC
+# define __rewinddir rewinddir
+#endif
#ifndef _LIBC
# define __getcwd getcwd
@@ -390,7 +394,7 @@ __getcwd (buf, size)
if (use_d_ino)
{
use_d_ino = false;
- rewinddir (dirstream);
+ __rewinddir (dirstream);
continue;
}
diff --git a/sysdeps/posix/rewinddir.c b/sysdeps/posix/rewinddir.c
index 5a4a7154ab..e9bc78d073 100644
--- a/sysdeps/posix/rewinddir.c
+++ b/sysdeps/posix/rewinddir.c
@@ -23,7 +23,7 @@
/* Rewind DIRP to the beginning of the directory. */
void
-rewinddir (dirp)
+__rewinddir (dirp)
DIR *dirp;
{
#ifndef NOT_IN_libc
@@ -38,4 +38,5 @@ rewinddir (dirp)
__libc_lock_unlock (dirp->lock);
#endif
}
-libc_hidden_def (rewinddir)
+libc_hidden_def (__rewinddir)
+weak_alias (__rewinddir, rewinddir)