Skip to content

Sync zinflate follow-ups from upstream - #70

Merged
Coralesoft merged 1 commit into
mainfrom
sync/zinflate-upstream-followups
Aug 5, 2026
Merged

Sync zinflate follow-ups from upstream#70
Coralesoft merged 1 commit into
mainfrom
sync/zinflate-upstream-followups

Conversation

@Coralesoft

Copy link
Copy Markdown
Collaborator

Mirrors the follow-up changes made upstream after merging our DEFLATE HLIT fix in weidai11/cryptopp#1371, keeping our inflator aligned with upstream.

  • Add a debug assertion for the rejected HLIT range alongside the existing runtime guard.
  • Add const qualifiers to the header fields.
  • Size codeLengths to zlib's 320 entries.
  • Recognise -DCRYPTOPP_DEBUG in the GNUmakefile debug filter.

Release behaviour is unchanged because the runtime guard already rejects invalid HLIT values.

Upstream merged the HLIT fix in weidai11/cryptopp#1371, then added a debug assertion for the rejected range, `const` qualifiers, and a 320-entry `codeLengths` buffer matching zlib.

Mirror those changes and recognise `-DCRYPTOPP_DEBUG` in the GNUmakefile debug filter to keep our inflator aligned with upstream.

The runtime guard already rejects invalid HLIT values, so release behaviour is unchanged.
@Coralesoft
Coralesoft merged commit 2fdab13 into main Aug 5, 2026
47 checks passed
@Coralesoft
Coralesoft deleted the sync/zinflate-upstream-followups branch August 5, 2026 09:55
@Coralesoft Coralesoft mentioned this pull request Aug 5, 2026
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