From b65cb8a35641675d44af84c9b18c40c6094f03ef Mon Sep 17 00:00:00 2001 From: Victor Stinner Date: Wed, 26 Jul 2017 17:54:42 +0200 Subject: bpo-31019: Fix multiprocessing.Process.is_alive() (#2875) (#2882) multiprocessing.Process.is_alive() now removes the process from the _children set if the process completed. The change prevents leaking "dangling" processes. (cherry picked from commit 2db64823c20538a6cfc6033661fab5711d2d4585) --- Lib/multiprocessing/process.py | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/Lib/multiprocessing/process.py b/Lib/multiprocessing/process.py index 16c4e1e..11c8fca 100644 --- a/Lib/multiprocessing/process.py +++ b/Lib/multiprocessing/process.py @@ -156,10 +156,16 @@ class Process(object): if self is _current_process: return True assert self._parent_pid == os.getpid(), 'can only test a child process' + if self._popen is None: return False - self._popen.poll() - return self._popen.returncode is None + + returncode = self._popen.poll() + if returncode is None: + return True + else: + _current_process._children.discard(self) + return False @property def name(self): -- cgit v0.12