Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[3.10] gh-139400: Make sure that parent parsers outlive their subparsers in pyexpat (GH-139403)#139613

Merged
pablogsal merged 2 commits into
python:3.10python/cpython:3.10from
hartwork:backport-6edb2dd-3.10hartwork/cpython:backport-6edb2dd-3.10Copy head branch name to clipboard
Oct 6, 2025
Merged

[3.10] gh-139400: Make sure that parent parsers outlive their subparsers in pyexpat (GH-139403)#139613
pablogsal merged 2 commits into
python:3.10python/cpython:3.10from
hartwork:backport-6edb2dd-3.10hartwork/cpython:backport-6edb2dd-3.10Copy head branch name to clipboard

Conversation

@hartwork

@hartwork hartwork commented Oct 5, 2025

Copy link
Copy Markdown
Contributor

…rs in `pyexpat` (python#139403)

* Modules/pyexpat.c: Disallow collection of in-use parent parsers.

Within libexpat, a parser created via `XML_ExternalEntityParserCreate`
is relying on its parent parser throughout its entire lifetime.
Prior to this fix, is was possible for the parent parser to be
garbage-collected too early.

(cherry picked from commit 6edb2dd)
@picnixz picnixz added the 🔨 test-with-refleak-buildbots Test PR w/ refleak buildbots; report in status section label Oct 5, 2025
@bedevere-bot

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @picnixz for commit 135a66e 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F139613%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-refleak-buildbots label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-refleak-buildbots Test PR w/ refleak buildbots; report in status section label Oct 5, 2025
@picnixz

picnixz commented Oct 5, 2025

Copy link
Copy Markdown
Member

I'll just run the refleak bots on the 3.10 and 3.13 branches as I think 3.11/3.12/3.13 are similar right (it's just to make sure the change of the GC protocol will not surprise us)

@hartwork

hartwork commented Oct 5, 2025

Copy link
Copy Markdown
Contributor Author

3.11/3.12/3.13 are similar right

@picnixz about as similar as it gets

@picnixz

picnixz commented Oct 5, 2025

Copy link
Copy Markdown
Member

@pablogsal pablogsal merged commit 8ea678d into python:3.10 Oct 6, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.