Skip to content

gh-123504: Fix reference leak in initialization of _tkinter #123505

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
vstinner merged 9 commits into from
Sep 3, 2024
Merged

gh-123504: Fix reference leak in initialization of _tkinter #123505

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
cddd021
Fix reference leak in initialization of _tkinter
ZeroIntensity Aug 30, 2024
a9eb482
Add NEWS entry.
ZeroIntensity Aug 30, 2024
36c65af
Switch to `PyModule_AddObjectRef`
ZeroIntensity Aug 30, 2024
cc291e4
Switch to a module clear function.
ZeroIntensity Aug 30, 2024
df9265f
Add back missing newline.
ZeroIntensity Aug 30, 2024
4f1f4e4
Fix formatting.
ZeroIntensity Aug 30, 2024
a77abba
Switch to `Py_CLEAR`
ZeroIntensity Aug 30, 2024
f35f2a0
Update Misc/NEWS.d/next/Library/2024-08-30-09-01-35.gh-issue-123504.l…
ZeroIntensity Sep 3, 2024
fc34562
Use dot notation for module initializer.
ZeroIntensity Sep 3, 2024
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions Misc/NEWS.d/next/Library/2024-08-30-09-01-35.gh-issue-123504.lJ9_BB.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Fixed reference leak in the initalization of :mod:`tkinter`.
4 changes: 4 additions & 0 deletions Modules/_tkinter.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3421,6 +3421,7 @@ PyInit__tkinter(void)
Tkinter_TclError = PyErr_NewException("_tkinter.TclError", NULL, NULL);
if (PyModule_AddObjectRef(m, "TclError", Tkinter_TclError)) {
Py_DECREF(m);
Py_DECREF(Tkinter_TclError);
return NULL;
}

Expand Down Expand Up @@ -3474,18 +3475,21 @@ PyInit__tkinter(void)
Py_DECREF(m);
return NULL;
}
Py_DECREF(Tkapp_Type);
Copy link
Member

@serhiy-storchaka serhiy-storchaka Aug 30, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is the same for all other global references (otherwise why would they be global references?).

Yes, if you want to release these references, you need a module clearing function. This is not a big deal, because usually when you unload the module, you quit the program, so who cares about few dangling references?

Tktt_Type = PyType_FromSpec(&Tktt_Type_spec);
if (PyModule_AddObjectRef(m, "TkttType", Tktt_Type)) {
Py_DECREF(m);
return NULL;
}
Py_DECREF(Tktt_Type);

PyTclObject_Type = PyType_FromSpec(&PyTclObject_Type_spec);
if (PyModule_AddObjectRef(m, "Tcl_Obj", PyTclObject_Type)) {
Py_DECREF(m);
return NULL;
}
Py_DECREF(PyTclObject_Type);


/* This helps the dynamic loader; in Unicode aware Tcl versions
Expand Down