]> xenbits.xensource.com Git - people/royger/xen.git/commitdiff
xz: move s->lzma.len = 0 initialization to lzma_reset()
authorLasse Collin <lasse.collin@tukaani.org>
Mon, 7 Mar 2022 08:09:26 +0000 (09:09 +0100)
committerJan Beulich <jbeulich@suse.com>
Mon, 7 Mar 2022 08:09:26 +0000 (09:09 +0100)
It's a more logical place even if the resetting needs to be done
only once per LZMA2 stream (if lzma_reset() called in the middle
of an LZMA2 stream, .len will already be 0).

Link: https://lore.kernel.org/r/20211010213145.17462-4-xiang@kernel.org
Signed-off-by: Lasse Collin <lasse.collin@tukaani.org>
Signed-off-by: Gao Xiang <hsiangkao@linux.alibaba.com>
Origin: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git a98a25408b0e
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Luca Fancellu <luca.fancellu@arm.com>
Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
xen/common/xz/dec_lzma2.c

index 3b60191ee63b77da8ef248c02a3ddf55a637b948..027ca93ce90d3974292e6b793583ae8be1e3895e 100644 (file)
@@ -791,6 +791,7 @@ static void __init lzma_reset(struct xz_dec_lzma2 *s)
        s->lzma.rep1 = 0;
        s->lzma.rep2 = 0;
        s->lzma.rep3 = 0;
+       s->lzma.len = 0;
 
        /*
         * All probabilities are initialized to the same value. This hack
@@ -1174,8 +1175,6 @@ XZ_EXTERN enum xz_ret __init xz_dec_lzma2_reset(struct xz_dec_lzma2 *s, uint8_t
                }
        }
 
-       s->lzma.len = 0;
-
        s->lzma2.sequence = SEQ_CONTROL;
        s->lzma2.need_dict_reset = true;