(Merge 3.2) Issue #12310: finalize the old process after _run_after_forkers()

Victor Stinner · Victor Stinner · commit 3bcc0170bdd5 · 2011-06-17T12:36:26.000+02:00
multiprocessing: Process._bootstrap() keeps a reference to the old process to
delay its finalization until after _run_after_forkers() as been executed. This
change should fix a crash on Mac OS X Tiger when a lock is released after a
fork.

Patch written by Charles-François Nataliv and Antoine Pitrou.
diff --git a/Lib/multiprocessing/process.py b/Lib/multiprocessing/process.py
@@ -267,9 +267,15 @@ def _bootstrap(self):
                     sys.stdin = open(os.devnull)
                 except (OSError, ValueError):
                     pass
+            old_process = _current_process
             _current_process = self
-            util._finalizer_registry.clear()
-            util._run_after_forkers()
+            try:
+                util._finalizer_registry.clear()
+                util._run_after_forkers()
+            finally:
+                # delay finalization of the old process object until after
+                # _run_after_forkers() is executed
+                del old_process
             util.info('child process calling self.run()')
             try:
                 self.run()
