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-e9683d4ca...@suse.com/
Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Suggested-by: Jan Beulich <jbeul...@suse.com>
Signed-off-by: Ariel Otilibili <ariel.otilibili-ani...@eurecom.fr>
--
Cc: Andrew Morton <a...@linux-foundation.org>
Cc: linux-ker...@vger.kernel.org
Cc: Andrew Cooper <andrew.coop...@citrix.com>
Cc: Anthony PERARD <anthony.per...@vates.tech>
Cc: Michal Orzel <michal.or...@amd.com>
Cc: Julien Grall <jul...@xen.org>
Cc: =?utf-8?q?Roger_Pau_Monn=C3=A9?= <roger....@citrix.com>
Cc: Stefano Stabellini <sstabell...@kernel.org>
Cc: xen-devel@lists.xenproject.org
---
 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


Reply via email to