ci(release): accept the unfixed arm64 Chromium CVEs in the image scan - #1434
Merged
Conversation
The v0.23.1 image scan failed on linux/arm64 with 33 HIGH findings: eleven Chromium ids across chromium, chromium-common and chromium-sandbox. amd64 was clean, and the asymmetry is structural rather than incidental. Chrome for Testing publishes no linux-arm64 build, so amd64 uses CfT while arm64 installs Debian's chromium; only the arm64 image carries the package the findings are against. The build is not stale. It fetched from bookworm-security with no cached apt layer and installed 151.0.7922.137-1~deb12u1, the newest that exists. Trivy names 151.0.7922.169-1~deb12u1 as fixed, and the security mirror's own package index still serves .137 for both architectures, so there is nothing to upgrade to and a rebuild changes nothing. Eight of the eleven are code execution in the renderer and none are neutralised by running with --no-sandbox, unlike the earlier batch. The note in the file states that plainly rather than talking it down. Drop all eleven once bookworm ships 151.0.7922.169 and re-run the scan without them.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The v0.23.1 image scan failed on
linux/arm64with 33 HIGH findings, blocking promotion and the GitHub Release. This accepts them temporarily so the release can ship, with the same shape and the same honesty as the earlier chromium batch in this file.Why the build is not at fault
chromium,chromium-commonandchromium-sandbox, which is why eleven produce thirty-three findings.151.0.7922.137-1~deb12u1; Trivy names151.0.7922.169-1~deb12u1as fixed. The release build fetched frombookworm-securitywith no cached apt layer, visible as realGet:lines in the build log and noCACHEDmarker on the production stage, and installed the newest version that exists. I read the security mirror's own package index directly: it still serves.137for both architectures. There is nothing to upgrade to, and rebuilding today produces the identical image.Exposure, not talked down
Eight of the eleven are arbitrary or remote code execution in the renderer: v8 type confusion (76038, 76047), a v8 miscalculation (76043), Dawn (76036), a WebGL use-after-free (76045), a use-after-free (76040), a race condition (76044) and link following (76037). One is a site isolation bypass (76033) and two are information disclosure (76039, 76041).
Unlike the earlier batch, where two of four were sandbox escapes that
--no-sandboxmade irrelevant, none of these are neutralised by how the container runs. Chromium renders content the sender controls, so a crafted message is a plausible path to code execution as theopenwauser inside the arm64 container, with the container itself as the only confinement boundary. This is accepted deliberately to ship, not because it is harmless.Drop condition
All eleven come out the moment bookworm ships
chromium >= 151.0.7922.169, tracked at the Debian security tracker link in the file, and the scan is re-run without them before assuming they are gone. That is what happened to the previous batch, which was added and then removed once Debian caught up.Verification
.trivyignorenow holds fourteen ids with no duplicates against the three already there.check:auditand the docs lane (264 tests) pass.