gzip: default level with ENABLE_FEATURE_GZIP_LEVELS should be 6
authorDaniel Edgecumbe <git@esotericnonsense.com>
Mon, 2 Sep 2019 21:03:14 +0000 (22:03 +0100)
committerDenys Vlasenko <vda.linux@googlemail.com>
Thu, 5 Sep 2019 11:26:58 +0000 (13:26 +0200)
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>
archival/gzip.c

index 17341de45790c49565b259ff35bd883ef1a3b780..37db347b873801ce5b6063ec04f9a88b5d1b386f 100644 (file)
@@ -2222,7 +2222,7 @@ int gzip_main(int argc UNUSED_PARAM, char **argv)
 #if ENABLE_FEATURE_GZIP_LEVELS
        opt >>= (BBUNPK_OPTSTRLEN IF_FEATURE_GZIP_DECOMPRESS(+ 2) + 1); /* drop cfkvq[dt]n bits */
        if (opt == 0)
-               opt = 1 << 6; /* default: 6 */
+               opt = 1 << 5; /* default: 6 */
        opt = ffs(opt >> 4); /* Maps -1..-4 to [0], -5 to [1] ... -9 to [5] */
        max_chain_length = 1 << gzip_level_config[opt].chain_shift;
        good_match       = gzip_level_config[opt].good;