[Arthur031221]

SerenityOS unzip aborts on valid source archives. In a ZIP made with git archive --format=zip 80baeb29708a, 460 entries fail, starting at Base/home/anon/Source/js/object-expression.js.

The Huffman decoder requires the full lookup width at EOF even when fewer bits contain a complete symbol. This change uses an explicit relaxed peek, then requires the matched code length. Strict streams still reject truncated symbols and propagate read errors.

Fixes #25005 and #22130.

Added tests for short symbols, truncated codes, and stream errors. Tested with ctest --test-dir Build/lagom/Tests/LibCompress --output-on-failure.

Assisted by Claude/Codex.

[ADKaster]

How does this targeted fix compare to the comprehensive refactor by @DanShaders in #24567 ?

Nico and Tim seem to have discussed using the method you are using in this patch in the referenced issues, so it might be reasonable to use it -- I didn't look very closely

[Arthur031221]

This fixes the synchronous path unzip uses. Both peek past EOF, then read the matched symbol's actual bit length. #24567 adds async DEFLATE/zlib decoders and integrates async zlib into HTTP, but leaves the existing synchronous Huffman overload unchanged. On this PR's current head, TestDeflate and TestBitStream pass (21 and 12 cases), including the short-tail input 4b4c4a4ec48600, truncated-symbol cases, and read-error propagation.

[Arthur031221]

Is there anything else you need from me on this one?

[DanShaders]

You should probably advertise this on #code-review in Discord.

(Not sure what exactly Andrew wanted me to look at here)