From f73813a8bbb55ac725720f4d3839b51f992f1f3f Mon Sep 17 00:00:00 2001 From: Benjamin Peterson Date: Mon, 2 Apr 2012 11:15:17 -0400 Subject: [PATCH] prevent writing to stderr from messing up the exception state (closes #14474) --- Lib/test/test_thread.py | 24 ++++++++++++++++++++++++ Misc/NEWS | 3 +++ Modules/threadmodule.c | 3 +++ 3 files changed, 30 insertions(+) diff --git a/Lib/test/test_thread.py b/Lib/test/test_thread.py index 544e70d2fd..387fa030ce 100644 --- a/Lib/test/test_thread.py +++ b/Lib/test/test_thread.py @@ -130,6 +130,30 @@ class ThreadRunningTests(BasicThreadTest): time.sleep(0.01) self.assertEqual(thread._count(), orig) + def test_save_exception_state_on_error(self): + # See issue #14474 + def task(): + started.release() + sys.stderr = stderr + raise SyntaxError + def mywrite(self, *args): + try: + raise ValueError + except ValueError: + pass + real_write(self, *args) + c = thread._count() + started = thread.allocate_lock() + with test_support.captured_output("stderr") as stderr: + real_write = stderr.write + stderr.write = mywrite + started.acquire() + thread.start_new_thread(task, ()) + started.acquire() + while thread._count() > c: + pass + self.assertIn("Traceback", stderr.getvalue()) + class Barrier: def __init__(self, num_threads): diff --git a/Misc/NEWS b/Misc/NEWS index 15c943c1c0..8593a10b1c 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -9,6 +9,9 @@ What's New in Python 2.7.4 Core and Builtins ----------------- +- Issue #14474: Save and restore exception state in thread.start_new_thread() + while writing error message if the thread leaves a unhandled exception. + - Issue #13019: Fix potential reference leaks in bytearray.extend(). Patch by Suman Saha. diff --git a/Modules/threadmodule.c b/Modules/threadmodule.c index 4e41085ed9..73ee382df3 100644 --- a/Modules/threadmodule.c +++ b/Modules/threadmodule.c @@ -618,6 +618,8 @@ t_bootstrap(void *boot_raw) PyErr_Clear(); else { PyObject *file; + PyObject *exc, *value, *tb; + PyErr_Fetch(&exc, &value, &tb); PySys_WriteStderr( "Unhandled exception in thread started by "); file = PySys_GetObject("stderr"); @@ -625,6 +627,7 @@ t_bootstrap(void *boot_raw) PyFile_WriteObject(boot->func, file, 0); else PyObject_Print(boot->func, stderr, 0); + PyErr_Restore(exc, value, tb); PySys_WriteStderr("\n"); PyErr_PrintEx(0); } -- 2.50.0