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.
This commit is contained in:
Neal Norwitz 2006-07-10 00:23:17 +00:00
parent 8440483fea
commit 4a5fbda66d
1 changed files with 0 additions and 7 deletions

View File

@ -1750,13 +1750,6 @@ class TarFile(object):
try: try:
tarinfo = TarInfo.frombuf(buf) 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 # Set the TarInfo object's offset to the current position of the
# TarFile and set self.offset to the position where the data blocks # TarFile and set self.offset to the position where the data blocks
# should begin. # should begin.