Skip to content

[3.14] gh-140593: Fix a memory leak in function my_ElementDeclHandler of pyexpat (GH-140602)#140624

Closed
miss-islington wants to merge 1 commit into
python:3.14from
miss-islington:backport-e34a5e3-3.14
Closed

[3.14] gh-140593: Fix a memory leak in function my_ElementDeclHandler of pyexpat (GH-140602)#140624
miss-islington wants to merge 1 commit into
python:3.14from
miss-islington:backport-e34a5e3-3.14

Conversation

@miss-islington

@miss-islington miss-islington commented Oct 26, 2025

Copy link
Copy Markdown
Contributor

Ensure that the memory allocated for the content model
passed to my_ElementDeclHandler is freed in all error
paths.
(cherry picked from commit e34a5e3)

Co-authored-by: Sebastian Pipping sebastian@pipping.org

…` of `pyexpat` (pythonGH-140602)

Ensure that the memory allocated for the content model
passed to `my_ElementDeclHandler` is freed in all error
paths.
(cherry picked from commit e34a5e3)

Co-authored-by: Sebastian Pipping <sebastian@pipping.org>
@picnixz

picnixz commented Oct 26, 2025

Copy link
Copy Markdown
Member

@hartwork Can you take care of adding the import please? You can take over the PR since it's a bot-generated PR.

@hartwork

Copy link
Copy Markdown
Contributor

@hartwork Can you take care of adding the import please? You can take over the PR since it's a bot-generated PR.

@picnixz I would but I have no idea how to take over PRs without being a CPython member. Do you have a link on docs how to do that if it is possible?

@hartwork

Copy link
Copy Markdown
Contributor

@picnixz …or did you mean just to create a new one? That sure I can do.

@picnixz

picnixz commented Oct 26, 2025

Copy link
Copy Markdown
Member

Ah right, sorry. Yes, please create a fresh PR

@hartwork

Copy link
Copy Markdown
Contributor

Ah right, sorry. Yes, please create a fresh PR

@picnixz deal, I'll go for both 3.13 and 3.14 if things work out as expected 🤝

@picnixz picnixz closed this Oct 26, 2025
@miss-islington miss-islington deleted the backport-e34a5e3-3.14 branch January 2, 2026 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants