From 2db64823c20538a6cfc6033661fab5711d2d4585 Mon Sep 17 00:00:00 2001 From: Victor Stinner Date: Wed, 26 Jul 2017 02:32:42 +0200 Subject: bpo-31019: Fix multiprocessing.Process.is_alive() (#2875) multiprocessing.Process.is_alive() now removes the process from the _children set if the process completed. The change prevents leaking "dangling" processes. --- 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 ce4ce43..8e500dc 100644 --- a/Lib/multiprocessing/process.py +++ b/Lib/multiprocessing/process.py @@ -148,10 +148,16 @@ class BaseProcess(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: + _children.discard(self) + return False def close(self): ''' -- cgit v0.12