Skip to content
Merged
40 changes: 39 additions & 1 deletion Lib/test/test_atexit.py
Original file line number Diff line number Diff line change
@@ -1,9 +1,11 @@
import atexit
import os
import subprocess
import tempfile
import textwrap
import unittest
from test import support
from test.support import script_helper
from test.support import SuppressCrashReport, script_helper
from test.support import threading_helper

class GeneralTest(unittest.TestCase):
Expand Down Expand Up @@ -189,6 +191,42 @@ def callback():
self.assertEqual(os.read(r, len(expected)), expected)
os.close(r)

# Python built with Py_TRACE_REFS fail with a fatal error in
# _PyRefchain_Trace() on memory allocation error.
@unittest.skipIf(support.Py_TRACE_REFS, 'cannot test Py_TRACE_REFS build')
def test_atexit_with_low_memory(self):
# gh-140080: Test that setting low memory after registering an atexit
# callback doesn't cause an infinite loop during finalization.
code = textwrap.dedent("""
import atexit
import _testcapi

def callback():
print("hello")

atexit.register(callback)
# Simulate low memory condition
_testcapi.set_nomemory(0)
""")

with tempfile.NamedTemporaryFile(mode='w', suffix='.py', delete=False) as f:
f.write(code)
script = f.name

try:
with SuppressCrashReport():
with script_helper.spawn_python(script,
stderr=subprocess.PIPE) as proc:
proc.wait()
stdout = proc.stdout.read()
stderr = proc.stderr.read()
finally:
os.unlink(script)

self.assertIn(proc.returncode, (0, 1))
self.assertNotIn(b"hello", stdout)
self.assertIn(b"MemoryError", stderr)


if __name__ == "__main__":
unittest.main()
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Fix hang during finalization when attempting to call :mod:`atexit` handlers under no memory.
1 change: 1 addition & 0 deletions Modules/atexitmodule.c
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,7 @@ atexit_callfuncs(struct atexit_state *state)
{
PyErr_FormatUnraisable("Exception ignored while "
"copying atexit callbacks");
atexit_cleanup(state);
return;
}

Expand Down
Loading