From a3616c321e1f495f6608093b7b3204ce4971e928 Mon Sep 17 00:00:00 2001 From: Jelle Zijlstra Date: Tue, 20 Oct 2026 21:34:23 -0700 Subject: [PATCH] [3.14] gh-159739: Fix reference and error handling in warnings (GH-159758) Do not release the borrowed skip_file_prefixes argument, and stop if allocation of the fallback filename fails. (cherry picked from commit fab1405da8c93429ab1a1d3d3e1779d15b7223a2) --- Lib/test/test_warnings/__init__.py | 13 +++++++++++++ .../2026-10-19-19-33-35.gh-issue-159739.f81734.rst | 2 ++ Python/_warnings.c | 4 +++- 3 files changed, 18 insertions(+), 1 deletion(-) create mode 100644 Misc/NEWS.d/next/Core_and_Builtins/2026-10-19-19-33-35.gh-issue-159739.f81734.rst diff --git a/Lib/test/test_warnings/__init__.py b/Lib/test/test_warnings/__init__.py index 7d43bb6b3089025..5273baa065ac359 100644 --- a/Lib/test/test_warnings/__init__.py +++ b/Lib/test/test_warnings/__init__.py @@ -663,6 +663,19 @@ def test_skip_file_prefixes_type_errors(self): with self.assertRaises(TypeError): warn("msg", skip_file_prefixes="a sequence of strs") + def test_skip_file_prefixes_empty_tuple_subclass(self): + with warnings_state(self.module): + self.module.simplefilter("always") + warn = warning_tests.warnings.warn + class EmptyTuple(tuple): + pass + # Passing an empty tuple subclass used to lead to a crash. + for _ in range(3): + with self.module.catch_warnings(record=True) as w: + warn("msg", skip_file_prefixes=EmptyTuple()) + self.assertEqual(len(w), 1) + self.assertEqual(w[-1].filename, __file__) + def test_exec_filename(self): filename = "" codeobj = compile(("import warnings\n" diff --git a/Misc/NEWS.d/next/Core_and_Builtins/2026-10-19-19-33-35.gh-issue-159739.f81734.rst b/Misc/NEWS.d/next/Core_and_Builtins/2026-10-19-19-33-35.gh-issue-159739.f81734.rst new file mode 100644 index 000000000000000..0342bdef3e86994 --- /dev/null +++ b/Misc/NEWS.d/next/Core_and_Builtins/2026-10-19-19-33-35.gh-issue-159739.f81734.rst @@ -0,0 +1,2 @@ +Fix a crash in :func:`warnings.warn` when ``skip_file_prefixes`` is an empty tuple subclass. +Propagate allocation errors when creating the fallback filename. diff --git a/Python/_warnings.c b/Python/_warnings.c index 5b08e4aee09e396..69e69f3aabda006 100644 --- a/Python/_warnings.c +++ b/Python/_warnings.c @@ -1093,6 +1093,9 @@ setup_context(Py_ssize_t stack_level, if (f == NULL) { globals = interp->sysdict; *filename = PyUnicode_FromString(""); + if (*filename == NULL) { + return 0; + } *lineno = 0; } else { @@ -1237,7 +1240,6 @@ warnings_warn_impl(PyObject *module, PyObject *message, PyObject *category, stacklevel = 2; } } else { - Py_DECREF((PyObject *)skip_file_prefixes); skip_file_prefixes = NULL; } }