From 20880caef4dc58e22bccac0494cc06c4f24155fe Mon Sep 17 00:00:00 2001 From: Victor Stinner Date: Wed, 26 Jul 2017 01:54:00 +0200 Subject: [PATCH] bpo-31019: Fix multiprocessing.Process.is_alive() 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 ce4ce43cfca6fbd..8e500dc93dc6bdb 100644 --- a/Lib/multiprocessing/process.py +++ b/Lib/multiprocessing/process.py @@ -148,10 +148,16 @@ def is_alive(self): 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): '''