]> git.ipfire.org Git - thirdparty/xz.git/commit
xz: Fix decompressor behavior if input uses an unsupported check type.
authorLasse Collin <lasse.collin@tukaani.org>
Tue, 25 Oct 2022 16:07:17 +0000 (19:07 +0300)
committerLasse Collin <lasse.collin@tukaani.org>
Fri, 11 Nov 2022 11:38:34 +0000 (13:38 +0200)
commita3e4606134058241bf1c396243bebcec42c2a780
treedb8aa84d91560e222d3d0c9ed84a2a8678953fcf
parent0b5e8c7e073bce07c59112af1e8a7495e3576e6c
xz: Fix decompressor behavior if input uses an unsupported check type.

Now files with unsupported check will make xz display
a warning, set the exit status to 2 (unless --no-warn is used),
and then decompress the file normally. This is how it was
supposed to work since the beginning but this was broken by
the commit 231c3c7098f1099a56abb8afece76fc9b8699f05, that is,
a little before 5.0.0 was released. The buggy behavior displayed
a message, set exit status 1 (error), and xz didn't attempt to
to decompress the file.

This doesn't matter today except for special builds that disable
CRC64 or SHA-256 at build time (but such builds should be used
in special situations only). The bug matters if new check type
is added in the future and an old xz version is used to decompress
such a file; however, it's likely that such files would use a new
filter too and an old xz wouldn't be able to decompress the file
anyway.

The first hunk in the commit is the actual fix. The second hunk
is a cleanup since LZMA_TELL_ANY_CHECK isn't used in xz.

There is a test file for unsupported check type but it wasn't
used by test_files.sh, perhaps due to different behavior between
xz and the simpler xzdec.
src/xz/coder.c