diff options
| author | Gregory P. Smith <greg@krypto.org> | 2013-12-01 03:02:57 (GMT) |
|---|---|---|
| committer | Gregory P. Smith <greg@krypto.org> | 2013-12-01 03:02:57 (GMT) |
| commit | 1eda9e7c3074bfe37674c8140b3d02bf94caf4f3 (patch) | |
| tree | 66e0ccc1c1d085cfdfb6335f07b48e57dac0d4e2 /Lib/subprocess.py | |
| parent | 65846c6c5190a92446176f318fd837c6005cfa29 (diff) | |
| download | cpython-1eda9e7c3074bfe37674c8140b3d02bf94caf4f3.zip cpython-1eda9e7c3074bfe37674c8140b3d02bf94caf4f3.tar.gz cpython-1eda9e7c3074bfe37674c8140b3d02bf94caf4f3.tar.bz2 | |
Fixes Issue #15798 - subprocess.Popen() no longer fails if file
descriptor 0, 1 or 2 is closed.
Diffstat (limited to 'Lib/subprocess.py')
| -rw-r--r-- | Lib/subprocess.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/Lib/subprocess.py b/Lib/subprocess.py index c3a2788..a659087 100644 --- a/Lib/subprocess.py +++ b/Lib/subprocess.py @@ -1368,7 +1368,10 @@ class Popen(object): executable_list = tuple( os.path.join(os.fsencode(dir), executable) for dir in os.get_exec_path(env)) - fds_to_keep = set(pass_fds) + # Never close stdin, stdout and stderr for the child. + fds_to_keep = {0,1,2} + fds_to_keep.update(pass_fds) + # Our child uses this one to signal error before exec(). fds_to_keep.add(errpipe_write) self.pid = _posixsubprocess.fork_exec( args, executable_list, |