From: Victor Stinner <victor.stinner@gmail.com>
Date: Wed, 3 May 2017 22:45:56 +0000 (+0200)
Subject: bpo-30225: Fix is_valid_fd() on macOS Tiger (#1443)
X-Git-Tag: v3.7.0a1~864
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=1c4670ea0cc3d208121af11b9b973e6bb268e570;p=thirdparty%2FPython%2Fcpython.git

bpo-30225: Fix is_valid_fd() on macOS Tiger (#1443)

is_valid_fd() now uses fstat() instead of dup() on macOS to return 0
on a pipe when the other side of the pipe is closed. fstat() fails
with EBADF in that case, whereas dup() succeed.
---

diff --git a/Python/pylifecycle.c b/Python/pylifecycle.c
index afd4eb8f55d7..c0f41b3ca734 100644
--- a/Python/pylifecycle.c
+++ b/Python/pylifecycle.c
@@ -1045,6 +1045,14 @@ initsite(void)
 static int
 is_valid_fd(int fd)
 {
+#ifdef __APPLE__
+    /* bpo-30225: On macOS Tiger, when stdout is redirected to a pipe
+       and the other side of the pipe is closed, dup(1) succeed, whereas
+       fstat(1, &st) fails with EBADF. Prefer fstat() over dup() to detect
+       such error. */
+    struct stat st;
+    return (fstat(fd, &st) == 0);
+#else
     int fd2;
     if (fd < 0)
         return 0;
@@ -1057,6 +1065,7 @@ is_valid_fd(int fd)
         close(fd2);
     _Py_END_SUPPRESS_IPH
     return fd2 >= 0;
+#endif
 }
 
 /* returns Py_None if the fd is not valid */