From 667d4b577f576323628d97f7b2ed421a2a05322f Mon Sep 17 00:00:00 2001 From: Victor Stinner Date: Sat, 25 Dec 2010 22:40:32 +0000 Subject: [PATCH] Issue #10763: subprocess.communicate() closes stdout and stderr if both are pipes (bug specific to Windows). Improve also the unit test: write a portable unit test. --- Lib/subprocess.py | 1 + Lib/test/test_subprocess.py | 34 ++++++++++++++++++++-------------- Misc/NEWS | 3 +++ 3 files changed, 24 insertions(+), 14 deletions(-) diff --git a/Lib/subprocess.py b/Lib/subprocess.py index 452611c5a9..c12898de16 100644 --- a/Lib/subprocess.py +++ b/Lib/subprocess.py @@ -985,6 +985,7 @@ class Popen(object): def _readerthread(self, fh, buffer): buffer.append(fh.read()) + fh.close() def _communicate(self, input): diff --git a/Lib/test/test_subprocess.py b/Lib/test/test_subprocess.py index 8ec3d719dc..f1c91435b5 100644 --- a/Lib/test/test_subprocess.py +++ b/Lib/test/test_subprocess.py @@ -366,22 +366,28 @@ class ProcessTestCase(BaseTestCase): self.assertEqual(stdout, b"banana") self.assertStderrEqual(stderr, b"pineapple") - # This test is Linux specific for simplicity to at least have - # some coverage. It is not a platform specific bug. - @unittest.skipUnless(os.path.isdir('/proc/%d/fd' % os.getpid()), - "Linux specific") # Test for the fd leak reported in http://bugs.python.org/issue2791. def test_communicate_pipe_fd_leak(self): - fd_directory = '/proc/%d/fd' % os.getpid() - num_fds_before_popen = len(os.listdir(fd_directory)) - p = subprocess.Popen([sys.executable, "-c", "print()"], - stdout=subprocess.PIPE) - p.communicate() - num_fds_after_communicate = len(os.listdir(fd_directory)) - del p - num_fds_after_destruction = len(os.listdir(fd_directory)) - self.assertEqual(num_fds_before_popen, num_fds_after_destruction) - self.assertEqual(num_fds_before_popen, num_fds_after_communicate) + for stdin_pipe in (False, True): + for stdout_pipe in (False, True): + for stderr_pipe in (False, True): + options = {} + if stdin_pipe: + options['stdin'] = subprocess.PIPE + if stdout_pipe: + options['stdout'] = subprocess.PIPE + if stderr_pipe: + options['stderr'] = subprocess.PIPE + if not options: + continue + p = subprocess.Popen((sys.executable, "-c", "pass"), **options) + p.communicate() + if p.stdin is not None: + self.assertTrue(p.stdin.closed) + if p.stdout is not None: + self.assertTrue(p.stdout.closed) + if p.stderr is not None: + self.assertTrue(p.stderr.closed) def test_communicate_returns(self): # communicate() should return None if no redirection is active diff --git a/Misc/NEWS b/Misc/NEWS index 7b6d2d11c2..88dd83625c 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -11,6 +11,9 @@ Core and Builtins Library ------- +- Issue #10763: subprocess.communicate() closes stdout and stderr if both are + pipes (bug specific to Windows). + - Issue #1693546: fix email.message RFC 2231 parameter encoding to be in better compliance (no "s around encoded values). -- 2.40.0