This is a follow up from a discussion in Xen: The if-statement tests `res` is non-zero; meaning the case zero is never reached. Link: https://lore.kernel.org/all/7587b503-b2ca-4476-8dc9-e9683d4ca5f0@xxxxxxxx/ Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2") Suggested-by: Jan Beulich <jbeulich@xxxxxxxx> Signed-off-by: Ariel Otilibili <ariel.otilibili-anieli@xxxxxxxxxx> -- Cc: stable@xxxxxxxxxxxxxxx Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> Cc: linux-kernel@xxxxxxxxxxxxxxx Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> Cc: Anthony PERARD <anthony.perard@xxxxxxxxxx> Cc: Michal Orzel <michal.orzel@xxxxxxx> Cc: Julien Grall <julien@xxxxxxx> Cc: =?utf-8?q?Roger_Pau_Monn=C3=A9?= <roger.pau@xxxxxxxxxx> Cc: Stefano Stabellini <sstabellini@xxxxxxxxxx> Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx --- lib/inflate.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/lib/inflate.c b/lib/inflate.c index fbaf03c1748d..eab886baa1b4 100644 --- a/lib/inflate.c +++ b/lib/inflate.c @@ -1257,8 +1257,6 @@ static int INIT gunzip(void) /* Decompress */ if ((res = inflate())) { switch (res) { - case 0: - break; case 1: error("invalid compressed format (err=1)"); break; -- 2.47.1