[Python-checkins] cpython (2.7): Issue #12467: warnings: fix a race condition if a warning is emitted at
victor.stinner
python-checkins at python.org
Mon Jul 4 03:05:45 CEST 2011
http://hg.python.org/cpython/rev/fc46acf7a645
changeset: 71175:fc46acf7a645
branch: 2.7
parent: 71170:c9f69b28c4d1
user: Victor Stinner <victor.stinner at haypocalc.com>
date: Mon Jul 04 03:05:37 2011 +0200
summary:
Issue #12467: warnings: fix a race condition if a warning is emitted at
shutdown, if globals()['__file__'] is None.
files:
Lib/test/test_warnings.py | 12 ++++++++++++
Misc/NEWS | 3 +++
Python/_warnings.c | 2 +-
3 files changed, 16 insertions(+), 1 deletions(-)
diff --git a/Lib/test/test_warnings.py b/Lib/test/test_warnings.py
--- a/Lib/test/test_warnings.py
+++ b/Lib/test/test_warnings.py
@@ -530,6 +530,18 @@
assert expected_line
self.assertEqual(second_line, expected_line)
+ def test_filename_none(self):
+ # issue #12467: race condition if a warning is emitted at shutdown
+ globals_dict = globals()
+ oldfile = globals_dict['__file__']
+ try:
+ with original_warnings.catch_warnings(module=self.module) as w:
+ self.module.filterwarnings("always", category=UserWarning)
+ globals_dict['__file__'] = None
+ self.module.warn('test', UserWarning)
+ finally:
+ globals_dict['__file__'] = oldfile
+
class WarningsDisplayTests(unittest.TestCase):
diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -18,6 +18,9 @@
Library
-------
+- Issue #12467: warnings: fix a race condition if a warning is emitted at
+ shutdown, if globals()['__file__'] is None.
+
- Issue #12352: Fix a deadlock in multiprocessing.Heap when a block is freed by
the garbage collector while the Heap lock is held.
diff --git a/Python/_warnings.c b/Python/_warnings.c
--- a/Python/_warnings.c
+++ b/Python/_warnings.c
@@ -491,7 +491,7 @@
/* Setup filename. */
*filename = PyDict_GetItemString(globals, "__file__");
- if (*filename != NULL) {
+ if (*filename != NULL && PyString_Check(*filename)) {
Py_ssize_t len = PyString_Size(*filename);
const char *file_str = PyString_AsString(*filename);
if (file_str == NULL || (len < 0 && PyErr_Occurred()))
--
Repository URL: http://hg.python.org/cpython
More information about the Python-checkins
mailing list