Skip to content

gh-139400: Make sure that parent parsers outlive their subparsers in pyexpat #139403

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
picnixz merged 22 commits into from
Oct 5, 2025
Merged

gh-139400: Make sure that parent parsers outlive their subparsers in pyexpat #139403

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
e1c0470
pyexpat.c: Disallow collection of in-use parent parsers
hartwork Sep 28, 2025
3b0afda
Add a regression test for issue 139400
hartwork Sep 28, 2025
8f70991
Add news item for issue 139400
hartwork Sep 28, 2025
a63bd34
Improve test's docstring
hartwork Sep 28, 2025
22417ec
Simplify xmlparseobject_ to xmlparseobject
hartwork Sep 28, 2025
20caacd
Leverage Py_CLEAR
hartwork Sep 28, 2025
f8c58a3
Simplify call to Py_CLEAR
hartwork Sep 28, 2025
f1e3e36
Streamline docsting of ParentParserLifetimeTest
hartwork Sep 28, 2025
1b2937d
pyexpat.c: Adjust whitespace around pointer
hartwork Sep 28, 2025
8c5a4e8
Re-write news item
hartwork Sep 29, 2025
a6f5520
Turn parent into a PyObject pointer
hartwork Sep 30, 2025
4470cea
Add missing parent visit to xmlparse_traverse
hartwork Sep 30, 2025
3759ac7
Re-write comment about mission of "PyObject *parent"
hartwork Sep 30, 2025
30598a9
Add whitespace as requested
hartwork Oct 2, 2025
6d3c424
Add more tests
hartwork Oct 2, 2025
66b77e0
Add comment about Py_CLEAR(self->parent) to xmlparse_clear
hartwork Oct 5, 2025
455787f
Add missing full stops to docstrings
hartwork Oct 5, 2025
a669454
Drop reference to XML_ExternalEntityParserCreate
hartwork Oct 5, 2025
a91ed6b
Make test test_cycle waterproof
hartwork Oct 5, 2025
c056cfe
Full stops and sentences
hartwork Oct 5, 2025
12b8989
Drop test test_cycle
hartwork Oct 5, 2025
0541d6f
Drop now-unused import
hartwork Oct 5, 2025
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
Prev Previous commit
Next Next commit
Simplify call to Py_CLEAR 
Idea by @picnixz
  • Loading branch information
@hartwork
hartwork committed Sep 28, 2025
commit f8c58a3040b2deb9b8d37196a4697c9ebf04abd6
4 changes: 1 addition & 3 deletions Modules/pyexpat.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1549,9 +1549,7 @@ xmlparse_dealloc(PyObject *op)
XML_ParserFree(self->itself);
}
self->itself = NULL;
if (self->parent != NULL) {
Py_CLEAR(self->parent);
}
Py_CLEAR(self->parent);
Comment thread
picnixz marked this conversation as resolved.

if (self->handlers != NULL) {
PyMem_Free(self->handlers);
Expand Down