Skip to content
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.

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

python-3.11/CVE-2024-7592 patch implementation #27288

Closed
wants to merge 1 commit into from

Conversation

jamie-albert
Copy link
Contributor

Cherry picking in the git-checkout block causes a out of memory error as the buffer size is exceeded when pulling in the two massive branches of 3.11 and 3.12, instead of adding in the logic of increasing buffer size and the subsequent increased build time, I created a patch of the backported changes instead. The patch is identical between all python versions and has been merged in 3.13 and 3.12, the only reason this remains an open PR is the fact that it requires a special level of privilege to merge on these branches. All tests pass on 3.11 commit that will implement this eventually

@jamie-albert
Copy link
Contributor Author

Will put out advisory with the updated CVE info

Copy link
Contributor

@xnox xnox left a comment

Choose a reason for hiding this comment

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

Why would we ship unreviewed backport ahead of cpython upstream shipping it for python3.11 for a low severity CVE? Or any other distro?

Risks being, unreproducible behaviour change not present in any other python 3.11 distribution.

Example status from Debian - https://security-tracker.debian.org/tracker/CVE-2024-7592
[bookworm] - python3.11 (Minor issue, wait until merged into 3.11 branch)

@jamie-albert
Copy link
Contributor Author

I agree with your points, the only way it would make sense would maybe be for a critical CVE approaching SLA. Here is the advisory

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.

2 participants