[MEDIUM] Patch libvirt for CVE-2024-4418#13811
Conversation
kgodara912
left a comment
There was a problem hiding this comment.
Buddy build. Patch exactly matches with upstream reference.
Kanishk-Bansal
left a comment
There was a problem hiding this comment.
- Buddy Build
- patch applied during the build (check
rpm.log)Patch applies cleanly - patch include an upstream reference
- PR has security tag
@Kanishk-Bansal I didn't quite get it, could you please throw some light, I don't see any license check failures. Thanks! |
352dbf6 to
d7f1101
Compare
|
Waiting for @kgodara912 's review requested by @Kanishk-Bansal |
| %files docs | ||
| %doc AUTHORS.rst NEWS.rst README.rst | ||
| %doc libvirt-docs/* | ||
| %license libvirt-docs/html/fonts/LICENSE.rst |
There was a problem hiding this comment.
This change is not correct as it will remove all the docs from docs rpm (size reduced from 2MB to 100kb), we will see how to use exclude and packages to properly get license and docs both.
kgodara912
left a comment
There was a problem hiding this comment.
Please wait for a suggestion for a proper fix.
d7f1101 to
38a2ab4
Compare
Kanishk-Bansal
left a comment
There was a problem hiding this comment.
Fixing the license issue seems non-trivial for now. Let's keep this PR moving forward.
38a2ab4 to
f8476e8
Compare
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
Addresses libvirt CVE-2024-4418
Patch file: https://gitlab.com/libvirt/libvirt/-/commit/8074d64dc2eca846d6a61efe1a9b7428a0ce1dd1
Change Log
Does this affect the toolchain?
NO
Associated issues
Links to CVEs
Test Methodology