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/[email protected]/
Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Suggested-by: Jan Beulich <[email protected]>
Signed-off-by: Ariel Otilibili <[email protected]>
--
Cc: [email protected]
Cc: Andrew Morton <[email protected]>
Cc: [email protected]
Cc: Andrew Cooper <[email protected]>
Cc: Anthony PERARD <[email protected]>
Cc: Michal Orzel <[email protected]>
Cc: Julien Grall <[email protected]>
Cc: =?utf-8?q?Roger_Pau_Monn=C3=A9?= <[email protected]>
Cc: Stefano Stabellini <[email protected]>
Cc: [email protected]
---
 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