diff options
| author | Lasse Collin <lasse.collin@tukaani.org> | 2015-02-21 23:40:26 +0200 |
|---|---|---|
| committer | Lasse Collin <lasse.collin@tukaani.org> | 2015-02-21 23:40:26 +0200 |
| commit | 117d962685c72682c63edc9bb765367189800202 (patch) | |
| tree | d34a3c2b7cc93f5d8367cbc13a0f4409d4de8d07 | |
| parent | xz: Fix the fcntl() usage when creating a pipe for the self-pipe trick. (diff) | |
| download | xz-117d962685c72682c63edc9bb765367189800202.tar.xz | |
liblzma: Fix a compression-ratio regression in LZMA1/2 in fast mode.
The bug was added in the commit
f48fce093b07aeda95c18850f5e086d9f2383380 and thus
affected 5.1.4beta and 5.2.0. Luckily the bug cannot
cause data corruption or other nasty things.
| -rw-r--r-- | src/liblzma/lzma/lzma_encoder_optimum_fast.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/liblzma/lzma/lzma_encoder_optimum_fast.c b/src/liblzma/lzma/lzma_encoder_optimum_fast.c index 8922cbd9..9b30347c 100644 --- a/src/liblzma/lzma/lzma_encoder_optimum_fast.c +++ b/src/liblzma/lzma/lzma_encoder_optimum_fast.c @@ -152,7 +152,7 @@ lzma_lzma_optimum_fast(lzma_coder *restrict coder, lzma_mf *restrict mf, // the old buf pointer instead of recalculating it with mf_ptr(). ++buf; - const uint32_t limit = len_main - 1; + const uint32_t limit = my_max(2, len_main - 1); for (uint32_t i = 0; i < REPS; ++i) { if (memcmp(buf, buf - coder->reps[i] - 1, limit) == 0) { |