Repository navigation
Update lz4 to 1.11.1 - #181
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates the
src/lz4submodule from v1.11.0 to v1.11.1.Library changes in
lz4.c/lz4hc.cbetween the two tags, and what they mean for lz4-java:LZ4_resetStreamHC()out-of-bounds write fix (levels >= 10, on a state containing garbage). Does not affect lz4-java: the JNI layer never callsLZ4_resetStreamHC(). HC states come fromLZ4_createStreamHC()(zero-allocated, extended size) and are reused viaLZ4_compress_HC_extStateHC_fastReset(), which usesLZ4_resetStreamHC_fast(). The one-shotLZ4_compress_HC()initializes its state withLZ4_initStreamHC().LZ4_FAST_DEC_LOOPandLZ4_wildCopy64enabled. Upstream reports +12–24% decompression speed on silesia; applies to the shippedlinux/ppc64lenative.LZ4_STATIC_LINKING_ONLY_DISABLE_MEMORY_ALLOCATIONis set. Not used by lz4-java.The remaining release changes are CLI, frame format and build-system fixes, none of which are compiled into lz4-java.
Tested locally on linux/amd64:
./mvnw verifywith the JNI library rebuilt from the new sources — 4105 tests, 0 failures.🤖 Generated with Claude Code