Skip to content

CVE-2021-28918: perlPackages.NetCIDRLite, backport upstream fix (no version bump) - #42

Draft
jasonodoom wants to merge 2 commits into
masterfrom
trace/CVE-2021-28918--perlPackages.NetCIDRLite--ad030e70e8cdb2b5
Draft

CVE-2021-28918: perlPackages.NetCIDRLite, backport upstream fix (no version bump)#42
jasonodoom wants to merge 2 commits into
masterfrom
trace/CVE-2021-28918--perlPackages.NetCIDRLite--ad030e70e8cdb2b5

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-2021-28918.

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


Evidence: CVE-2021-28918 (human-review-required) trace bundle ad030e70e8cdb2b5

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

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 (lib/Net/CIDR/Lite.pm) are identical to the upstream fix commit Net-CIDR-Lite 23b6ff05, 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 33976731276) built perlPackages.NetCIDRLite at the pinned revision with and without the patch: the unpatched arm accepted the leading-zero octet and fired the discriminator, the patched arm rejected it.

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: a different regex that rejects the same leading-zero octets. 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