Skip to content

Conversation

@hartwork
Copy link
Contributor

@hartworkhartwork commented Oct 2, 2025

Expose the XML Expat 2.7.2 mitigation APIs to disallow use of disproportional amounts of dynamic memory from within an Expat parser (see CVE-2025-59375 for instance).

The exposed APIs are available on Expat parsers, that is, parsers created by xml.parsers.expat.ParserCreate(), as:

  • parser.SetAllocTrackerActivationThreshold(threshold), and
  • parser.SetAllocTrackerMaximumAmplification(max_factor).

(cherry picked from commit f04bea4)

CC @picnixz


📚 Documentation preview 📚: https://cpython-previews--139527.org.readthedocs.build/

picnixzand others added 4 commits October 2, 2025 23:47
CVE-2025-59375) (pythonGH-139234) Expose the XML Expat 2.7.2 mitigation APIs to disallow use of disproportional amounts of dynamic memory from within an Expat parser (see CVE-2025-59375 for instance). The exposed APIs are available on Expat parsers, that is, parsers created by `xml.parsers.expat.ParserCreate()`, as: - `parser.SetAllocTrackerActivationThreshold(threshold)`, and - `parser.SetAllocTrackerMaximumAmplification(max_factor)`. (cherry picked from commit f04bea4) Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
…on API (python#139366) Fix some typos left in f04bea4, and simplify some internal functions to ease maintenance of future mitigation APIs. (cherry picked from commit 68a1778)
@picnixz
Copy link
Member

To have a good synchronization, we'll also delay 3.10 to 3.13 backports for their next release cycle (see #139359 (comment)).

@picnixzpicnixz self-assigned this Oct 7, 2025
@ambv
Copy link
Contributor

ambv commented Oct 8, 2025

I set DO-NOT-MERGE to avoid confusion. Unset that when you think we should be releasing this.

Copy link
Member

@picnixzpicnixz left a comment

Choose a reason for hiding this comment

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

The 22 additional lines are in the clinic file so all good.

@picnixzpicnixz requested review from Yhg1s and removed request for Yhg1sNovember 8, 2025 13:47
@hartwork
Copy link
ContributorAuthor

@Yhg1s do you have a minute for this? 🙏

@hartwork
Copy link
ContributorAuthor

@Yhg1s do you have a minute? 🙏

@Yhg1sYhg1s merged commit 0e4cd89 into python:3.12Dec 17, 2025
30 checks passed
@hartwork
Copy link
ContributorAuthor

Thanks! 🙏

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@hartwork@picnixz@ambv@Yhg1s