Stowage/cpython
2
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-10-31 21:51:50 +00:00
Code Issues Projects Releases Packages Wiki Activity

bpo-27535: Fix memory leak with warnings ignore (#4489)

Browse source 
The warnings module doesn't leak memory anymore in the hidden
warnings registry for the "ignore" action of warnings filters.

The warn_explicit() function doesn't add the warning key to the
registry anymore for the "ignore" action.
This commit is contained in:
Victor Stinner 2017-11-27 16:57:07 +01:00 • committed by GitHub
parent 21c7730761
commit c9758784eb
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 14 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

1
Lib/test/test_warnings/__init__.py
View file

@ -125,6 +125,7 @@ def test_ignore(self):
self.module.filterwarnings("ignore", category=UserWarning)
self.module.warn("FilterTests.test_ignore", UserWarning)
self.assertEqual(len(w), 0)
self.assertEqual(list(__warningregistry__), ['version'])
def test_ignore_after_default(self):
with original_warnings.catch_warnings(record=True,

1
Lib/warnings.py
View file

@ -364,7 +364,6 @@ def warn_explicit(message, category, filename, lineno,
action = defaultaction
# Early exit actions
if action == "ignore":
registry[key] = 1
return
# Prime the linecache for formatting, in case the

4
Misc/NEWS.d/next/Library/2017-11-21-16-05-35.bpo-27535.JLhcNz.rst Normal file
View file

@ -0,0 +1,4 @@
The warnings module doesn't leak memory anymore in the hidden warnings
registry for the "ignore" action of warnings filters. warn_explicit()
function doesn't add the warning key to the registry anymore for the
"ignore" action.

13
Python/_warnings.c
View file

@ -528,16 +528,21 @@ warn_explicit(PyObject *category, PyObject *message,
goto cleanup;
}
if (_PyUnicode_EqualToASCIIString(action, "ignore")) {
goto return_none;
}
/* Store in the registry that we've been here, *except* when the action
is "always". */
rc = 0;
if (!_PyUnicode_EqualToASCIIString(action, "always")) {
if (registry != NULL && registry != Py_None &&
PyDict_SetItem(registry, key, Py_True) < 0)
PyDict_SetItem(registry, key, Py_True) < 0)
{
goto cleanup;
else if (_PyUnicode_EqualToASCIIString(action, "ignore"))
goto return_none;
else if (_PyUnicode_EqualToASCIIString(action, "once")) {
}
if (_PyUnicode_EqualToASCIIString(action, "once")) {
if (registry == NULL || registry == Py_None) {
registry = get_once_registry();
if (registry == NULL)