-
Notifications
You must be signed in to change notification settings - Fork 586
[MEDIUM] patch cmake for CVE-2025-4947 #13959
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
Conversation
2cd81ee
to
0db1349
Compare
24562e1
to
a378200
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code changes look good to me. Can you please attach the buddy build results.
Hi @suresh-thelkar, Log file from Local VM: Thank you! |
a378200
to
3caeca8
Compare
3caeca8
to
d839652
Compare
Buddy build link |
Hi @kgodara912, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Buddy Build
- patch applied during the build (check
rpm.log
) - patch include an upstream reference
- PR has security tag
@durgajagadeesh @Kanishk-Bansal @suresh-thelkar Toolchain change require full-build. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code changes look good to me. As Bala mentioned please make sure that you run full build and verify all the results.
Waiting for full build |
Merge Checklist
All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)
*-static
subpackages, etc.) have had theirRelease
tag incremented../cgmanifest.json
,./toolkit/scripts/toolchain/cgmanifest.json
,.github/workflows/cgmanifest.json
)./LICENSES-AND-NOTICES/SPECS/data/licenses.json
,./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md
,./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON
)*.signatures.json
filessudo make go-tidy-all
andsudo make go-test-coverage
passSummary
Change Log
Does this affect the toolchain?
NO
Associated issues
Links to CVEs
Test Methodology