[Python-checkins] [3.6] bpo-30817: Fix PyErr_PrintEx() when no memory (GH-2526). (#4107)
xdegaye
webhook-mailer at python.org
Tue Oct 24 10:42:39 EDT 2017
https://github.com/python/cpython/commit/d5d79545b73110b2f4c2b66d150409514e2ca8e0
commit: d5d79545b73110b2f4c2b66d150409514e2ca8e0
branch: 3.6
author: xdegaye <xdegaye at gmail.com>
committer: GitHub <noreply at github.com>
date: 2017-10-24T16:42:33+02:00
summary:
[3.6] bpo-30817: Fix PyErr_PrintEx() when no memory (GH-2526). (#4107)
(cherry picked from commit 66caacf2f0d6213b049a3097556e28e30440b900)
files:
A Misc/NEWS.d/next/Core and Builtins/2017-07-01-15-11-13.bpo-30817.j7ZvN_.rst
M Lib/test/test_exceptions.py
M Python/pythonrun.c
diff --git a/Lib/test/test_exceptions.py b/Lib/test/test_exceptions.py
index 53851cb6cfd..7eb08b746c0 100644
--- a/Lib/test/test_exceptions.py
+++ b/Lib/test/test_exceptions.py
@@ -10,7 +10,7 @@
from test.support import (TESTFN, captured_stderr, check_impl_detail,
check_warnings, cpython_only, gc_collect, run_unittest,
- no_tracing, unlink, import_module)
+ no_tracing, unlink, import_module, script_helper)
class NaiveException(Exception):
def __init__(self, x):
@@ -1104,6 +1104,23 @@ def test_unhandled(self):
self.assertIn("test message", report)
self.assertTrue(report.endswith("\n"))
+ @cpython_only
+ def test_memory_error_in_PyErr_PrintEx(self):
+ code = """if 1:
+ import _testcapi
+ class C(): pass
+ _testcapi.set_nomemory(0, %d)
+ C()
+ """
+
+ # Issue #30817: Abort in PyErr_PrintEx() when no memory.
+ # Span a large range of tests as the CPython code always evolves with
+ # changes that add or remove memory allocations.
+ for i in range(1, 20):
+ rc, out, err = script_helper.assert_python_failure("-c", code % i)
+ self.assertIn(rc, (1, 120))
+ self.assertIn(b'MemoryError', err)
+
class ImportErrorTests(unittest.TestCase):
diff --git a/Misc/NEWS.d/next/Core and Builtins/2017-07-01-15-11-13.bpo-30817.j7ZvN_.rst b/Misc/NEWS.d/next/Core and Builtins/2017-07-01-15-11-13.bpo-30817.j7ZvN_.rst
new file mode 100644
index 00000000000..f50aeefa847
--- /dev/null
+++ b/Misc/NEWS.d/next/Core and Builtins/2017-07-01-15-11-13.bpo-30817.j7ZvN_.rst
@@ -0,0 +1,2 @@
+`PyErr_PrintEx()` clears now the ignored exception that may be raised by
+`_PySys_SetObjectId()`, for example when no memory.
diff --git a/Python/pythonrun.c b/Python/pythonrun.c
index bf651d0ff8d..6a9722bee5f 100644
--- a/Python/pythonrun.c
+++ b/Python/pythonrun.c
@@ -624,9 +624,15 @@ PyErr_PrintEx(int set_sys_last_vars)
return;
/* Now we know v != NULL too */
if (set_sys_last_vars) {
- _PySys_SetObjectId(&PyId_last_type, exception);
- _PySys_SetObjectId(&PyId_last_value, v);
- _PySys_SetObjectId(&PyId_last_traceback, tb);
+ if (_PySys_SetObjectId(&PyId_last_type, exception) < 0) {
+ PyErr_Clear();
+ }
+ if (_PySys_SetObjectId(&PyId_last_value, v) < 0) {
+ PyErr_Clear();
+ }
+ if (_PySys_SetObjectId(&PyId_last_traceback, tb) < 0) {
+ PyErr_Clear();
+ }
}
hook = _PySys_GetObjectId(&PyId_excepthook);
if (hook) {
More information about the Python-checkins
mailing list