-
Notifications
You must be signed in to change notification settings - Fork 590
Patch luajit for CVE-2024-25177[HIGH] #14436
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
Patch luajit for CVE-2024-25177[HIGH] #14436
Conversation
this link |
Yes, please use the github link. |
We can use |
The above has different signature compared to what we have already due to .gitignore in source tree, except that everything else is same. I will rename existing source tree and will update signature for the file. |
Updating new reference for LuaJit.
Updated source URL with valid upstream reference.
Updated signature file as well.
Updated signature values.
Updated buddy 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.
Patch Analysis (the patch applies cleanly wrt t upstream)
there was a check failing due to which we updated the signature and cgmanifest as the older link was giving a 404 error
- Buddy Build
- patch applied during the build (check
rpm.log
) - patch include an upstream reference
- PR has security tag
Co-authored-by: Kshitiz Godara <kgodara@microsoft.com> Co-authored-by: Kanishk Bansal <103916909+Kanishk-Bansal@users.noreply.github.com> (cherry picked from commit 40c76f2)
Auto cherry-pick results: Auto cherry-pick pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=900928&view=results |
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
Added patch for CVE-2024-25177
Change Log
Does this affect the toolchain?
NO
Associated issues
Links to CVEs
Test Methodology