diff options
| -rw-r--r-- | Lib/tarfile.py | 8 | ||||
| -rw-r--r-- | Lib/test/test_tarfile.py | 17 | ||||
| -rw-r--r-- | Misc/NEWS | 3 |
3 files changed, 24 insertions, 4 deletions
diff --git a/Lib/tarfile.py b/Lib/tarfile.py index ae242912e9..7143f0b5e2 100644 --- a/Lib/tarfile.py +++ b/Lib/tarfile.py @@ -2037,11 +2037,11 @@ class TarFile(object): # Set correct owner, mtime and filemode on directories. for tarinfo in directories: - path = os.path.join(path, tarinfo.name) + dirpath = os.path.join(path, tarinfo.name) try: - self.chown(tarinfo, path) - self.utime(tarinfo, path) - self.chmod(tarinfo, path) + self.chown(tarinfo, dirpath) + self.utime(tarinfo, dirpath) + self.chmod(tarinfo, dirpath) except ExtractError, e: if self.errorlevel > 1: raise diff --git a/Lib/test/test_tarfile.py b/Lib/test/test_tarfile.py index a280bdda82..d446d5fa2d 100644 --- a/Lib/test/test_tarfile.py +++ b/Lib/test/test_tarfile.py @@ -244,6 +244,23 @@ class MiscReadTest(ReadTest): data = open(os.path.join(TEMPDIR, "ustar/symtype"), "rb").read() self.assertEqual(md5sum(data), md5_regtype) + def test_extractall(self): + # Test if extractall() correctly restores directory permissions + # and times (see issue1735). + if sys.platform == "win32": + # Win32 has no support for utime() on directories or + # fine grained permissions. + return + + tar = tarfile.open(tarname, encoding="iso8859-1") + directories = [t for t in tar if t.isdir()] + tar.extractall(TEMPDIR, directories) + for tarinfo in directories: + path = os.path.join(TEMPDIR, tarinfo.name) + self.assertEqual(tarinfo.mode & 0777, os.stat(path).st_mode & 0777) + self.assertEqual(tarinfo.mtime, os.path.getmtime(path)) + tar.close() + class StreamReadTest(ReadTest): @@ -342,6 +342,9 @@ Core and builtins Library ------- +- Issue #1735: TarFile.extractall() now correctly sets directory permissions + and times. + - Bug #1713: posixpath.ismount() claims symlink to a mountpoint is a mountpoint. - Bug #1687: Fxed plistlib.py restricts <integer> to Python int when writing |