diff options
| author | Victor Stinner <victor.stinner@gmail.com> | 2017-05-04 11:21:16 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-05-04 11:21:16 (GMT) |
| commit | 8a1c71053139f20348ea487c0c464694ed3c88c5 (patch) | |
| tree | 7bccb5f42ce5ce45a0c4ee0654beff9eaee2a500 /Python/pylifecycle.c | |
| parent | 943861f09ab6bffcd1d97efcd0dd6c87c7f26800 (diff) | |
| download | cpython-8a1c71053139f20348ea487c0c464694ed3c88c5.zip cpython-8a1c71053139f20348ea487c0c464694ed3c88c5.tar.gz cpython-8a1c71053139f20348ea487c0c464694ed3c88c5.tar.bz2 | |
bpo-30225: Fix is_valid_fd() on macOS Tiger (#1443) (#1450)
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.
(cherry picked from commit 1c4670ea0cc3d208121af11b9b973e6bb268e570)
Diffstat (limited to 'Python/pylifecycle.c')
| -rw-r--r-- | Python/pylifecycle.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/Python/pylifecycle.c b/Python/pylifecycle.c index 87dadc1..422454c 100644 --- a/Python/pylifecycle.c +++ b/Python/pylifecycle.c @@ -968,6 +968,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 || !_PyVerify_fd(fd)) return 0; @@ -977,6 +985,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 */ |