From 3017a7bbbf2046b93ac89fcbabadaf0a528e5ae1 Mon Sep 17 00:00:00 2001 From: Vinay Sajip Date: Mon, 22 Mar 2010 12:33:08 +0000 Subject: [PATCH] Issue #8200: logging: Handle errors when multiprocessing is not fully loaded when logging occurs. --- Lib/logging/__init__.py | 14 +++++++++++--- Misc/NEWS | 3 +++ 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/Lib/logging/__init__.py b/Lib/logging/__init__.py index b1810d68e53..088c1fc9ca0 100644 --- a/Lib/logging/__init__.py +++ b/Lib/logging/__init__.py @@ -287,10 +287,18 @@ class LogRecord(object): self.threadName = None if not logMultiprocessing: self.processName = None - elif 'multiprocessing' not in sys.modules: - self.processName = 'MainProcess' else: - self.processName = sys.modules['multiprocessing'].current_process().name + self.processName = 'MainProcess' + mp = sys.modules.get('multiprocessing') + if mp is not None: + # Errors may occur if multiprocessing has not finished loading + # yet - e.g. if a custom import hook causes third-party code + # to run when multiprocessing calls import. See issue 8200 + # for an example + try: + self.processName = mp.current_process().name + except StandardError: + pass if logProcesses and hasattr(os, 'getpid'): self.process = os.getpid() else: diff --git a/Misc/NEWS b/Misc/NEWS index dd61f4f64c2..12db4af60fb 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -29,6 +29,9 @@ Core and Builtins Library ------- +- Issue #8200: logging: Handle errors when multiprocessing is not + fully loaded when logging occurs. + - Issue #3890: Fix recv() and recv_into() on non-blocking SSL sockets. - Issue #8179: Fix macpath.realpath() on a non-existing path.