From: Andrew M. Kuchling <amk@amk.ca>
Date: Wed, 5 Feb 2003 21:35:07 +0000 (+0000)
Subject: [Patch #654421 from Matthew Mueller]
X-Git-Tag: v2.3c1~2017
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=64edd6ac1ddb29f7817ee44e9b376b096ffa1d55;p=python

[Patch #654421 from Matthew Mueller]
  gzip shouldn't raise ValueError on corrupt files

  Currently the gzip module will raise a ValueError if the file was
  corrupt (bad crc or bad size).  I can't see how that applies to
  reading a corrupt file.  IOError seems better, and it's what code
  will likely be looking for.
---

diff --git a/Lib/gzip.py b/Lib/gzip.py
index 36f9c36fd3..761d94157b 100644
--- a/Lib/gzip.py
+++ b/Lib/gzip.py
@@ -305,9 +305,9 @@ class GzipFile:
         crc32 = read32(self.fileobj)
         isize = U32(read32(self.fileobj))   # may exceed 2GB
         if U32(crc32) != U32(self.crc):
-            raise ValueError, "CRC check failed"
+            raise IOError, "CRC check failed"
         elif isize != LOWU32(self.size):
-            raise ValueError, "Incorrect length of data produced"
+            raise IOError, "Incorrect length of data produced"
 
     def close(self):
         if self.mode == WRITE: