From 4a5fbda66d3132af761e32e164ed398977f51694 Mon Sep 17 00:00:00 2001 From: Neal Norwitz Date: Mon, 10 Jul 2006 00:23:17 +0000 Subject: Part of SF patch #1484695. This removes dead code. The chksum was already verified in .frombuf() on the lines above. If there was a problem an exception is raised, so there was no way this condition could have been true. --- Lib/tarfile.py | 7 ------- 1 file changed, 7 deletions(-) diff --git a/Lib/tarfile.py b/Lib/tarfile.py index 061d0f5..47bc871 100644 --- a/Lib/tarfile.py +++ b/Lib/tarfile.py @@ -1750,13 +1750,6 @@ class TarFile(object): try: tarinfo = TarInfo.frombuf(buf) - # We shouldn't rely on this checksum, because some tar programs - # calculate it differently and it is merely validating the - # header block. We could just as well skip this part, which would - # have a slight effect on performance... - if tarinfo.chksum not in calc_chksums(buf): - self._dbg(1, "tarfile: Bad Checksum %r" % tarinfo.name) - # Set the TarInfo object's offset to the current position of the # TarFile and set self.offset to the position where the data blocks # should begin. -- cgit v0.12