Skip to content

CVE-2023-49502: ffmpeg_5, backport upstream fix (no version bump) - #43

Draft
jasonodoom wants to merge 2 commits into
masterfrom
trace/CVE-2023-49502--ffmpeg_5--f745077a5aeba1f0
Draft

CVE-2023-49502: ffmpeg_5, backport upstream fix (no version bump)#43
jasonodoom wants to merge 2 commits into
masterfrom
trace/CVE-2023-49502--ffmpeg_5--f745077a5aeba1f0

Conversation

@jasonodoom

Copy link
Copy Markdown

⚠️ EXPERIMENTAL source-backport — NOT a version bump

This PR authors the upstream fix as an in-tree patch against the current nixpkgs version (no version bump). An agent generated it; a cross-family reviewer verified it transcribes the upstream commit, and a cert proved the bundled reproducer goes red→green. A human maintainer must still confirm sufficiency before merging:

  • This applies the COMPLETE upstream fix, not one of several required commits.
  • The backported hunk matches the upstream fix commit(s):
  • Reviewed the cert evidence (reproducer red→green) for CVE-2023-49502.

Opened as a draft deliberately — mark Ready-for-review only after the boxes above are checked.


Evidence: CVE-2023-49502 (human-review-required) trace bundle f745077a5aeba1f0

Verify locally
gh run download <run-id> --name steward-smoke-<bundle> --dir ./artifact
trace bundle verify ./artifact  # bundle_id=f745077a5aeba1f0

Generated by trace.

@jasonodoom

Copy link
Copy Markdown
Author

Comparison against the upstream fix

Transcription lane (this PR's payload). The added and removed code lines of the embedded patch (libavfilter/vf_bwdif.c) are identical to the upstream fix commit FFmpeg 8e6c82ce, normalized for whitespace. The cross-family Reviewer attested this; the check here was recomputed independently against the bundle's upstream-fix.patch. The Linux cert (trace run 33975633515) built ffmpeg_5 at the pinned revision with and without the patch, a 37 minute two-arm build: the unpatched arm fired the discriminator on the crafted clip, the patched arm was clean.

Blind-authoring lane (separate measurement, not this payload). When the same CVE was given to claude-opus-4-8 with only the vulnerability description and the pre-fix source, never the upstream patch, the graded result against the human fix was GOLD: the same chroma-plane-aware dimension check, placed after the pixel format descriptor lookup. Report: trace data/reports/authoring-accuracy-vs-human-2026-09-04.json.

This is one of the cases where the blind lane got the fix right unaided. The PR ships the transcribed upstream fix because that is what the Reviewer can attest line by line.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant