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.13 + 3.12: Add tests for CVE-2025-0938 #41308

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

techalchemy
Copy link
Contributor

- Ensure we don't regress CVE-2025-0938

Signed-off-by: Dan Ryan <daniel.ryan@chainguard.dev>
imjasonh
imjasonh previously approved these changes Feb 4, 2025
Signed-off-by: Jason Hall <jason@chainguard.dev>
@imjasonh imjasonh closed this Feb 4, 2025
@imjasonh imjasonh reopened this Feb 4, 2025
@imjasonh imjasonh added the approved-to-run A repo member has approved this external contribution label Feb 4, 2025
@octo-sts octo-sts bot added the bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. label Feb 4, 2025
@imjasonh
Copy link
Member

imjasonh commented Feb 5, 2025

ci-cve-scan says

└── 📄 /.PKGINFO
        📦 python-3.12 3.12.8-r4 (apk)
            Unknown CVE-2025-0938

This indicates that NVD/grype still thinks .8 is vulnerable, which just means we need to make sure to file an advisory that we've got the fix in .8.

Separately, we have these update PRs, which are also not being detected as fixed:

We could merge both this PR and those, or just those, up to you. We'll need advisories either way.

@techalchemy
Copy link
Contributor Author

which just means we need to make sure to file an advisory that we've got the fix in .8.

I did open an advisory PR, wolfi-dev/advisories#11921 -- but we opted to close this for now in favor of giving a bit of time to see if grype updates.

I am still getting my bearings so I will probably defer to @ajayk and @smoser as to whether we still want to merge these tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved-to-run A repo member has approved this external contribution bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants