diff options
author | Serhiy Storchaka <storchaka@gmail.com> | 2013-02-18 10:22:05 (GMT) |
---|---|---|
committer | Serhiy Storchaka <storchaka@gmail.com> | 2013-02-18 10:22:05 (GMT) |
commit | 2a47954895b8a8cb0f63e4dbf957cd3f48e84d16 (patch) | |
tree | 0035bdd0229324286b353a3f479f02471d12fdf9 | |
parent | 580e2c1a65334cf702290e9a789670828e4673ee (diff) | |
parent | 407aa2df2aa81180df611148fbfe7a7f57673a74 (diff) | |
download | cpython-2a47954895b8a8cb0f63e4dbf957cd3f48e84d16.zip cpython-2a47954895b8a8cb0f63e4dbf957cd3f48e84d16.tar.gz cpython-2a47954895b8a8cb0f63e4dbf957cd3f48e84d16.tar.bz2 |
Fix posixpath.realpath() for multiple pardirs (fixes issue #6975).
-rw-r--r-- | Lib/posixpath.py | 6 | ||||
-rw-r--r-- | Lib/test/test_posixpath.py | 18 |
2 files changed, 22 insertions, 2 deletions
diff --git a/Lib/posixpath.py b/Lib/posixpath.py index 6390a865..e1d59b6 100644 --- a/Lib/posixpath.py +++ b/Lib/posixpath.py @@ -390,9 +390,11 @@ def _joinrealpath(path, rest, seen): if name == pardir: # parent dir if path: - path = dirname(path) + path, name = split(path) + if name == pardir: + path = join(path, pardir, pardir) else: - path = name + path = pardir continue newpath = join(path, name) if not islink(newpath): diff --git a/Lib/test/test_posixpath.py b/Lib/test/test_posixpath.py index 0d85727..ed56c51 100644 --- a/Lib/test/test_posixpath.py +++ b/Lib/test/test_posixpath.py @@ -283,6 +283,24 @@ class PosixPathTest(unittest.TestCase): self.assertEqual(posixpath.normpath(b"///..//./foo/.//bar"), b"/foo/bar") + def test_realpath_curdir(self): + self.assertEqual(realpath('.'), os.getcwd()) + self.assertEqual(realpath('./.'), os.getcwd()) + self.assertEqual(realpath('/'.join(['.'] * 100)), os.getcwd()) + + self.assertEqual(realpath(b'.'), os.getcwdb()) + self.assertEqual(realpath(b'./.'), os.getcwdb()) + self.assertEqual(realpath(b'/'.join([b'.'] * 100)), os.getcwdb()) + + def test_realpath_pardir(self): + self.assertEqual(realpath('..'), dirname(os.getcwd())) + self.assertEqual(realpath('../..'), dirname(dirname(os.getcwd()))) + self.assertEqual(realpath('/'.join(['..'] * 100)), '/') + + self.assertEqual(realpath(b'..'), dirname(os.getcwdb())) + self.assertEqual(realpath(b'../..'), dirname(dirname(os.getcwdb()))) + self.assertEqual(realpath(b'/'.join([b'..'] * 100)), b'/') + @unittest.skipUnless(hasattr(os, "symlink"), "Missing symlink implementation") @skip_if_ABSTFN_contains_backslash |