Issue #12494: Close pipes and kill process on error in subprocess functions

On error, call(), check_call(), check_output() and getstatusoutput() functions
of the subprocess module now kill the process, read its status (to avoid
zombis) and close pipes.

Lib/subprocess.py

@@ -464,10 +464,10 @@ def call(*popenargs, timeout=None, **kwargs):
 
     retcode = call(["ls", "-l"])
     """
-    p = Popen(*popenargs, **kwargs)
+    with Popen(*popenargs, **kwargs) as p:
         try:
             return p.wait(timeout=timeout)
-    except TimeoutExpired:
+        except:
             p.kill()
             p.wait()
             raise
@@ -514,13 +514,17 @@ def check_output(*popenargs, timeout=None, **kwargs):
     """
     if 'stdout' in kwargs:
         raise ValueError('stdout argument not allowed, it will be overridden.')
-    process = Popen(*popenargs, stdout=PIPE, **kwargs)
+    with Popen(*popenargs, stdout=PIPE, **kwargs) as process:
         try:
             output, unused_err = process.communicate(timeout=timeout)
         except TimeoutExpired:
             process.kill()
             output, unused_err = process.communicate()
             raise TimeoutExpired(process.args, timeout, output=output)
+        except:
+            process.kill()
+            process.wait()
+            raise
         retcode = process.poll()
         if retcode:
             raise CalledProcessError(retcode, process.args, output=output)
@@ -618,11 +622,19 @@ def getstatusoutput(cmd):
     >>> subprocess.getstatusoutput('/bin/junk')
     (256, 'sh: /bin/junk: not found')
     """
-    pipe = os.popen('{ ' + cmd + '; } 2>&1', 'r')
+    with os.popen('{ ' + cmd + '; } 2>&1', 'r') as pipe:
+        try:
             text = pipe.read()
             sts = pipe.close()
-    if sts is None: sts = 0
-    if text[-1:] == '\n': text = text[:-1]
+        except:
+            process = pipe._proc
+            process.kill()
+            process.wait()
+            raise
+    if sts is None:
+        sts = 0
+    if text[-1:] == '\n':
+        text = text[:-1]
     return sts, text
 
 

Misc/NEWS

@@ -271,6 +271,10 @@ Core and Builtins
 Library
 -------
 
+- Issue #12494: On error, call(), check_call(), check_output() and
+  getstatusoutput() functions of the subprocess module now kill the process,
+  read its status (to avoid zombis) and close pipes.
+
 - Issue #12720: Expose low-level Linux extended file attribute functions in os.
 
 - Issue #10946: The distutils commands bdist_dumb, bdist_wininst and bdist_msi