aboutsummaryrefslogtreecommitdiff
path: root/socket
diff options
context:
space:
mode:
authorMike Frysinger <vapier@gentoo.org>2012-08-17 22:03:56 -0400
committerMike Frysinger <vapier@gentoo.org>2012-08-18 00:35:47 -0400
commita277af22ea038ff963355b603ade8d0a8a98eb5d (patch)
tree2c15ae9829a1f5a9d949e4bc48b8a77abe4ece3e /socket
parentfdab8fd3351de83f0d4a513552318f337e14c4cb (diff)
downloadglibc-a277af22ea038ff963355b603ade8d0a8a98eb5d.zip
glibc-a277af22ea038ff963355b603ade8d0a8a98eb5d.tar.gz
glibc-a277af22ea038ff963355b603ade8d0a8a98eb5d.tar.bz2
split assume pipe2/dup3/sock_cloexec knobs
We can't assume sock_cloexec and pipe2 are bound together as the former defines are found in glibc only while the latter are a combo of kernel headers and glibc. So if we do a runtime detection of SOCK_CLOEXEC, but pipe2() is a stub inside of glibc, we hit a problem. For example: main() { getgrnam("portage"); if (!popen("ls", "r")) perror("popen()"); } getgrnam() will detect that the kernel supports SOCK_CLOEXEC and then set both __have_sock_cloexec and __have_pipe2 to true. But if glibc was built against older kernel headers where __NR_pipe2 does not exist, glibc will have a ENOSYS stub for it. So popen() will always fail as glibc assumes pipe2() works. While this isn't too much of an issue for some arches as they added the functionality to the kernel at the same time, not all arches are that lucky. Since the code already has dedicated names for each feature, delete the defines wiring these three features together and make each one a proper dedicated knob. We've been carrying this in Gentoo since glibc-2.9. Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Diffstat (limited to 'socket')
-rw-r--r--socket/have_sock_cloexec.c11
1 files changed, 10 insertions, 1 deletions
diff --git a/socket/have_sock_cloexec.c b/socket/have_sock_cloexec.c
index d57cbcf..8cf0959 100644
--- a/socket/have_sock_cloexec.c
+++ b/socket/have_sock_cloexec.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 2008 Free Software Foundation, Inc.
+/* Copyright (C) 2008-2012 Free Software Foundation, Inc.
This file is part of the GNU C Library.
The GNU C Library is free software; you can redistribute it and/or
@@ -15,9 +15,18 @@
License along with the GNU C Library; if not, see
<http://www.gnu.org/licenses/>. */
+#include <fcntl.h>
#include <sys/socket.h>
#include <kernel-features.h>
#if defined SOCK_CLOEXEC && !defined __ASSUME_SOCK_CLOEXEC
int __have_sock_cloexec;
#endif
+
+#if defined O_CLOEXEC && !defined __ASSUME_PIPE2
+int __have_pipe2;
+#endif
+
+#if defined O_CLOEXEC && !defined __ASSUME_DUP3
+int __have_dup3;
+#endif