-
Notifications
You must be signed in to change notification settings - Fork 601
[AutoPR- Security] Patch moby-engine for CVE-2025-58183 [MEDIUM] #15102
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
[AutoPR- Security] Patch moby-engine for CVE-2025-58183 [MEDIUM] #15102
Conversation
|
@xordux Please check whether the patch fixes the problem logically and not introducing any possible regression and provide signoff. |
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.
We will verify CVE applicability once. --> As per this, #15098 (review), removing "Request changes". Will leave for above comment to be resolved.
|
I signoff this patch. It fixes the problem logically and not introducing any possible regression. One point that might cause confusion is: “Why doesn’t this CVE patch include the maxSpecialFileSize value?” |
kgodara912
left a comment
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 matches with upstream reference except test files. Buddy build is successful. LGTM.
Auto Patch moby-engine for CVE-2025-58183.
Autosec pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=983215&view=results
Merge Checklist
All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)
*-staticsubpackages, etc.) have had theirReleasetag 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.jsonfilessudo make go-tidy-allandsudo make go-test-coveragepassSummary
What does the PR accomplish, why was it needed?
Change Log
Does this affect the toolchain?
YES/NO
Associated issues
Links to CVEs
Test Methodology