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
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 6 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

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`.
10 changes: 9 additions & 1 deletion Modules/_tkinter.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3389,6 +3389,14 @@ DisableEventHook(void)
#endif
}

static void
module_free(void *mod)
{
Py_DECREF(Tkinter_TclError);
Py_DECREF(Tkapp_Type);
Py_DECREF(Tktt_Type);
Py_DECREF(PyTclObject_Type);
}

static struct PyModuleDef _tkintermodule = {
PyModuleDef_HEAD_INIT,
Expand All @@ -3399,7 +3407,7 @@ static struct PyModuleDef _tkintermodule = {
NULL,
NULL,
NULL,
NULL
module_free
};

PyMODINIT_FUNC
Expand Down