fix: FastLZ match length calculation to avoid overflow and bad lengths #3211

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

anim001k wants to merge 1 commit into bluealloy:main from anim001k:patch-3

+6 −3

Contributor

anim001k commented Dec 7, 2025

Description:

Fix cmp to avoid unsigned underflow when q > r and stop on first mismatch.
Guard flz_match against l == 0 to preserve minimal encoded length.
Ensures FastLZ length estimates stay in sync with reference and no panics on crafted input.


          Update fast_lz.rs

a22f9c1

codspeed-hq bot commented Dec 7, 2025

CodSpeed Performance Report

Merging #3211 will not alter performance

_{Comparing anim001k:patch-3 (a22f9c1) with main (eb37a15)}

Summary

✅ 173 untouched

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

Labels

None yet