]> granicus.if.org Git - python/commit
bpo-32777: Fix _Py_set_inheritable async-safety in subprocess (GH-5560)
authorAlexey Izbyshev <izbyshev@users.noreply.github.com>
Tue, 6 Feb 2018 06:09:34 +0000 (09:09 +0300)
committerGregory P. Smith <greg@krypto.org>
Tue, 6 Feb 2018 06:09:34 +0000 (22:09 -0800)
commitc1e46e94de38a92f98736af9a42d89c3975a9919
tree90fdbdc92f2b0e90d5bb889b6f921995b51dd43d
parent22864bc8e4a076bbac748ccda6c27f1ec41b53e7
bpo-32777: Fix _Py_set_inheritable async-safety in subprocess (GH-5560)

Fix a rare but potential pre-exec child process deadlock in subprocess on POSIX systems when marking file descriptors inheritable on exec in the child process.  This bug appears to have been introduced in 3.4 with the inheritable file descriptors support.

This also changes Python/fileutils.c `set_inheritable` to use the "slow" two `fcntl` syscall path instead of the "fast" single `ioctl` syscall path when asked to be async signal safe (by way of being asked not to raise exceptions).  `ioctl` is not a POSIX async-signal-safe approved function.

ref: http://pubs.opengroup.org/onlinepubs/9699919799/functions/V2_chap02.html
Include/fileutils.h
Misc/NEWS.d/next/Library/2018-02-05-21-28-28.bpo-32777.C-wIXF.rst [new file with mode: 0644]
Modules/_posixsubprocess.c
Python/fileutils.c