summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMike Frysinger <vapier@gentoo.org>2011-01-09 00:55:11 +0000
committerMike Frysinger <vapier@gentoo.org>2011-01-09 00:55:11 +0000
commit13f68101133242f45ffde389343ec7dee4f00cf3 (patch)
tree4f234047cd40366005145bf5d87219f37e05203c /src/patchsets/glibc/2.12.2/1095_all_glibc-2.9-assume-pipe2.patch
parentinitial 2.11.3 patchset based on last 2.11.2 patchset (diff)
downloadgentoo-13f68101133242f45ffde389343ec7dee4f00cf3.tar.gz
gentoo-13f68101133242f45ffde389343ec7dee4f00cf3.tar.bz2
gentoo-13f68101133242f45ffde389343ec7dee4f00cf3.zip
initial 2.12.2 patchset based on last 2.12.1 patchset
Diffstat (limited to 'src/patchsets/glibc/2.12.2/1095_all_glibc-2.9-assume-pipe2.patch')
-rw-r--r--src/patchsets/glibc/2.12.2/1095_all_glibc-2.9-assume-pipe2.patch52
1 files changed, 52 insertions, 0 deletions
diff --git a/src/patchsets/glibc/2.12.2/1095_all_glibc-2.9-assume-pipe2.patch b/src/patchsets/glibc/2.12.2/1095_all_glibc-2.9-assume-pipe2.patch
new file mode 100644
index 0000000000..7390378b54
--- /dev/null
+++ b/src/patchsets/glibc/2.12.2/1095_all_glibc-2.9-assume-pipe2.patch
@@ -0,0 +1,52 @@
+http://bugs.gentoo.org/250342
+http://sources.redhat.com/bugzilla/show_bug.cgi?id=9685
+
+we cant 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:
+
+#include <grp.h>
+#include <stdio.h>
+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.
+
+--- a/include/unistd.h
++++ b/include/unistd.h
+@@ -167,9 +167,6 @@
+ extern int __pause_nocancel (void) attribute_hidden;
+
+ extern int __have_sock_cloexec;
+-/* At lot of other functionality became available at the same time as
+- SOCK_CLOEXEC. Avoid defining separate variables for all of them
+- unless it is really necessary. */
+-#define __have_pipe2 __have_sock_cloexec
++extern int __have_pipe2;
+
+ #endif
+--- a/socket/have_sock_cloexec.c
++++ b/socket/have_sock_cloexec.c
+@@ -16,9 +16,14 @@
+ Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA
+ 02111-1307 USA. */
+
++#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