diff options
-rw-r--r-- | Lib/test/test_warnings/__init__.py | 10 | ||||
-rw-r--r-- | Misc/NEWS.d/next/Core and Builtins/2017-09-24-09-57-04.bpo-31566.OxwINs.rst | 2 | ||||
-rw-r--r-- | Python/_warnings.c | 7 |
3 files changed, 16 insertions, 3 deletions
diff --git a/Lib/test/test_warnings/__init__.py b/Lib/test/test_warnings/__init__.py index 354da6b..a07886c 100644 --- a/Lib/test/test_warnings/__init__.py +++ b/Lib/test/test_warnings/__init__.py @@ -820,6 +820,16 @@ class _WarningsTests(BaseTest, unittest.TestCase): self.assertRaises(TypeError): wmod.warn_explicit('foo', Warning, 'bar', 1) + @support.cpython_only + def test_issue31566(self): + # warn() shouldn't cause an assertion failure in case of a bad + # __name__ global. + with original_warnings.catch_warnings(module=self.module): + self.module.filterwarnings('error', category=UserWarning) + with support.swap_item(globals(), '__name__', b'foo'), \ + support.swap_item(globals(), '__file__', None): + self.assertRaises(UserWarning, self.module.warn, 'bar') + class WarningsDisplayTests(BaseTest): diff --git a/Misc/NEWS.d/next/Core and Builtins/2017-09-24-09-57-04.bpo-31566.OxwINs.rst b/Misc/NEWS.d/next/Core and Builtins/2017-09-24-09-57-04.bpo-31566.OxwINs.rst new file mode 100644 index 0000000..d3ccfd7 --- /dev/null +++ b/Misc/NEWS.d/next/Core and Builtins/2017-09-24-09-57-04.bpo-31566.OxwINs.rst @@ -0,0 +1,2 @@ +Fix an assertion failure in `_warnings.warn()` in case of a bad +``__name__`` global. Patch by Oren Milman. diff --git a/Python/_warnings.c b/Python/_warnings.c index 7270d2c..b074788 100644 --- a/Python/_warnings.c +++ b/Python/_warnings.c @@ -694,13 +694,14 @@ setup_context(Py_ssize_t stack_level, PyObject **filename, int *lineno, /* Setup module. */ *module = PyDict_GetItemString(globals, "__name__"); - if (*module == NULL) { + if (*module == Py_None || (*module != NULL && PyUnicode_Check(*module))) { + Py_INCREF(*module); + } + else { *module = PyUnicode_FromString("<string>"); if (*module == NULL) goto handle_error; } - else - Py_INCREF(*module); /* Setup filename. */ *filename = PyDict_GetItemString(globals, "__file__"); |