diff options
author | Daniel Edgecumbe <git@esotericnonsense.com> | 2019-09-02 22:03:14 +0100 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2019-09-05 13:26:58 +0200 |
commit | de82f0b764de4720fed3378f2e5f938a3f4b9d18 (patch) | |
tree | a3f2cc7ad9e8b25160018bf951c6aa3027ec86dc | |
parent | 10509a70ee5c28800d23bf891b4f72603447e364 (diff) | |
download | busybox-w32-de82f0b764de4720fed3378f2e5f938a3f4b9d18.tar.gz busybox-w32-de82f0b764de4720fed3378f2e5f938a3f4b9d18.tar.bz2 busybox-w32-de82f0b764de4720fed3378f2e5f938a3f4b9d18.zip |
gzip: default level with ENABLE_FEATURE_GZIP_LEVELS should be 6
Fixes an off-by-one that actually resulted in level 7 being used
Signed-off-by: Daniel Edgecumbe <git@esotericnonsense.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r-- | archival/gzip.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/archival/gzip.c b/archival/gzip.c index 17341de45..37db347b8 100644 --- a/archival/gzip.c +++ b/archival/gzip.c | |||
@@ -2222,7 +2222,7 @@ int gzip_main(int argc UNUSED_PARAM, char **argv) | |||
2222 | #if ENABLE_FEATURE_GZIP_LEVELS | 2222 | #if ENABLE_FEATURE_GZIP_LEVELS |
2223 | opt >>= (BBUNPK_OPTSTRLEN IF_FEATURE_GZIP_DECOMPRESS(+ 2) + 1); /* drop cfkvq[dt]n bits */ | 2223 | opt >>= (BBUNPK_OPTSTRLEN IF_FEATURE_GZIP_DECOMPRESS(+ 2) + 1); /* drop cfkvq[dt]n bits */ |
2224 | if (opt == 0) | 2224 | if (opt == 0) |
2225 | opt = 1 << 6; /* default: 6 */ | 2225 | opt = 1 << 5; /* default: 6 */ |
2226 | opt = ffs(opt >> 4); /* Maps -1..-4 to [0], -5 to [1] ... -9 to [5] */ | 2226 | opt = ffs(opt >> 4); /* Maps -1..-4 to [0], -5 to [1] ... -9 to [5] */ |
2227 | max_chain_length = 1 << gzip_level_config[opt].chain_shift; | 2227 | max_chain_length = 1 << gzip_level_config[opt].chain_shift; |
2228 | good_match = gzip_level_config[opt].good; | 2228 | good_match = gzip_level_config[opt].good; |