commit ee3f7fee94915ab4f34c92d6c89b211de476660d
author Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> Fri Jun 12 10:02:45 2020 -0700
committer GitHub <noreply@github.com> Fri Jun 12 10:02:45 2020 -0700
tree 37e501199c16e43f1a4b868c41e89953edd8f476
parent 2c0a59bb3d2fb83312c965ea9d5ef9d0028a53bb

bpo-40955: Fix memory leak in subprocess module (GH-20825)


```
Direct leak of 8 byte(s) in 1 object(s) allocated from:
    GH-0 0x7f008bf19667 in __interceptor_malloc (/lib64/libasan.so.6+0xb0667)
    GH-1 0x7f007a0bee4a in subprocess_fork_exec /home/heimes/dev/python/cpython/Modules/_posixsubprocess.c:774
    GH-2 0xe0305b in cfunction_call Objects/methodobject.c:546
```

Signed-off-by: Christian Heimes <christian@python.org>
(cherry picked from commit 0d3350daa8123a3e16d4a534b6e873eb12c10d7c)

Co-authored-by: Christian Heimes <christian@python.org>

Modules/_posixsubprocess.c

diff --git a/Modules/_posixsubprocess.c b/Modules/_posixsubprocess.c
index add2962..5d1691a 100644
--- a/Modules/_posixsubprocess.c
+++ b/Modules/_posixsubprocess.c
@@ -893,6 +893,7 @@
     if (_enable_gc(need_to_reenable_gc, gc_module)) {
         pid = -1;
     }
+    PyMem_RawFree(groups);
     Py_XDECREF(preexec_fn_args_tuple);
     Py_XDECREF(gc_module);