aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2016-05-03 23:36:18 +0000
committerJoseph Myers <joseph@codesourcery.com>2016-05-03 23:36:18 +0000
commit2faba597eca15666ce46cc721041747e96c8b942 (patch)
treeeb65e6ce835cb591d5a7f683eb4456a8ead83238
parentcbc06bc486635347ee0da51d04a82eedf51602d5 (diff)
downloadglibc-2faba597eca15666ce46cc721041747e96c8b942.zip
glibc-2faba597eca15666ce46cc721041747e96c8b942.tar.gz
glibc-2faba597eca15666ce46cc721041747e96c8b942.tar.bz2
Fix sys/time.h timespec namespace (bug 20041).
For UNIX98 and older standards, sys/time.h should not define struct timespec, but does so via the inclusion of sys/select.h (which is a new header in the 2001 edition of POSIX, and defines struct timespec because of the declaration of pselect, a new function in the 2001 edition of POSIX). In turn, this affects some other headers that themselves include sys/time.h. This patch fixes this by conditioning the __need_timespec definition in sys/select.h on __USE_XOPEN2K, the same condition used there for the declaration of pselect (this has no effect on direct uses of sys/select.h with feature test macros for any standard that includes that header, since such standards result in __USE_XOPEN2K being defined). Tested for x86_64 and x86 (testsuite, and that installed stripped shared libraries are unchanged by the patch). [BZ #20041] * misc/sys/select.h (__need_timespec): Only define if [__USE_XOPEN2K]. * conform/Makefile (test-xfail-XPG4/sys/time.h/conform): Remove variable. (test-xfail-XPG4/utmpx.h/conform): Likewise. (test-xfail-UNIX98/sys/time.h/conform): Likewise. (test-xfail-UNIX98/utmpx.h/conform): Likewise.
-rw-r--r--ChangeLog11
-rw-r--r--conform/Makefile4
-rw-r--r--misc/sys/select.h4
3 files changed, 14 insertions, 5 deletions
diff --git a/ChangeLog b/ChangeLog
index 64d1bc7..35b1270 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2016-05-03 Joseph Myers <joseph@codesourcery.com>
+
+ [BZ #20041]
+ * misc/sys/select.h (__need_timespec): Only define if
+ [__USE_XOPEN2K].
+ * conform/Makefile (test-xfail-XPG4/sys/time.h/conform): Remove
+ variable.
+ (test-xfail-XPG4/utmpx.h/conform): Likewise.
+ (test-xfail-UNIX98/sys/time.h/conform): Likewise.
+ (test-xfail-UNIX98/utmpx.h/conform): Likewise.
+
2016-05-03 Paul E. Murphy <murphyp@linux.vnet.ibm.com>
Segher Boessenkool <segher@gcc.gnu.org>
diff --git a/conform/Makefile b/conform/Makefile
index 55cada5..d5273c1 100644
--- a/conform/Makefile
+++ b/conform/Makefile
@@ -184,24 +184,20 @@ test-xfail-XPG4/netinet/in.h/conform = yes
test-xfail-XPG4/signal.h/conform = yes
test-xfail-XPG4/stdlib.h/conform = yes
test-xfail-XPG4/sys/stat.h/conform = yes
-test-xfail-XPG4/sys/time.h/conform = yes
test-xfail-XPG4/sys/types.h/conform = yes
test-xfail-XPG4/sys/wait.h/conform = yes
test-xfail-XPG4/termios.h/conform = yes
test-xfail-XPG4/ucontext.h/conform = yes
test-xfail-XPG4/unistd.h/conform = yes
-test-xfail-XPG4/utmpx.h/conform = yes
test-xfail-POSIX/sys/wait.h/conform = yes
test-xfail-UNIX98/arpa/inet.h/conform = yes
test-xfail-UNIX98/netdb.h/conform = yes
test-xfail-UNIX98/netinet/in.h/conform = yes
test-xfail-UNIX98/signal.h/conform = yes
test-xfail-UNIX98/stdlib.h/conform = yes
-test-xfail-UNIX98/sys/time.h/conform = yes
test-xfail-UNIX98/sys/wait.h/conform = yes
test-xfail-UNIX98/ucontext.h/conform = yes
test-xfail-UNIX98/unistd.h/conform = yes
-test-xfail-UNIX98/utmpx.h/conform = yes
test-xfail-XOPEN2K/semaphore.h/conform = yes
test-xfail-XOPEN2K/signal.h/conform = yes
test-xfail-XOPEN2K/sys/wait.h/conform = yes
diff --git a/misc/sys/select.h b/misc/sys/select.h
index b852dac..52dd94f 100644
--- a/misc/sys/select.h
+++ b/misc/sys/select.h
@@ -39,7 +39,9 @@ typedef __sigset_t sigset_t;
/* Get definition of timer specification structures. */
#define __need_time_t
-#define __need_timespec
+#ifdef __USE_XOPEN2K
+# define __need_timespec
+#endif
#include <time.h>
#define __need_timeval
#include <bits/time.h>