diff options
author | Serhiy Storchaka <storchaka@gmail.com> | 2013-05-09 11:36:58 (GMT) |
---|---|---|
committer | Serhiy Storchaka <storchaka@gmail.com> | 2013-05-09 11:36:58 (GMT) |
commit | b3c5d8568030842c6fbd30a6f197c8b684e0ccc6 (patch) | |
tree | 517e83deb06792e056b50606483824d66db46f8f /Lib/tarfile.py | |
parent | f28dfdd07b57fa16f7d489e7ac99d9a15f1bef80 (diff) | |
parent | 263fab94ee1f32b53b709c25c76044719d0eb88e (diff) | |
download | cpython-b3c5d8568030842c6fbd30a6f197c8b684e0ccc6.zip cpython-b3c5d8568030842c6fbd30a6f197c8b684e0ccc6.tar.gz cpython-b3c5d8568030842c6fbd30a6f197c8b684e0ccc6.tar.bz2 |
Issue #16601: Restarting iteration over tarfile no more continues from where
it left off. Patch by Michael Birtwell.
Diffstat (limited to 'Lib/tarfile.py')
-rw-r--r-- | Lib/tarfile.py | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/Lib/tarfile.py b/Lib/tarfile.py index ffa7327..06df7df 100644 --- a/Lib/tarfile.py +++ b/Lib/tarfile.py @@ -2397,16 +2397,18 @@ class TarIter: # Fix for SF #1100429: Under rare circumstances it can # happen that getmembers() is called during iteration, # which will cause TarIter to stop prematurely. - if not self.tarfile._loaded: + + if self.index == 0 and self.tarfile.firstmember is not None: + tarinfo = self.tarfile.next() + elif self.index < len(self.tarfile.members): + tarinfo = self.tarfile.members[self.index] + elif not self.tarfile._loaded: tarinfo = self.tarfile.next() if not tarinfo: self.tarfile._loaded = True raise StopIteration else: - try: - tarinfo = self.tarfile.members[self.index] - except IndexError: - raise StopIteration + raise StopIteration self.index += 1 return tarinfo |