summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSerhiy Storchaka <storchaka@gmail.com>2013-02-18 10:22:05 (GMT)
committerSerhiy Storchaka <storchaka@gmail.com>2013-02-18 10:22:05 (GMT)
commit2a47954895b8a8cb0f63e4dbf957cd3f48e84d16 (patch)
tree0035bdd0229324286b353a3f479f02471d12fdf9
parent580e2c1a65334cf702290e9a789670828e4673ee (diff)
parent407aa2df2aa81180df611148fbfe7a7f57673a74 (diff)
downloadcpython-2a47954895b8a8cb0f63e4dbf957cd3f48e84d16.zip
cpython-2a47954895b8a8cb0f63e4dbf957cd3f48e84d16.tar.gz
cpython-2a47954895b8a8cb0f63e4dbf957cd3f48e84d16.tar.bz2
Fix posixpath.realpath() for multiple pardirs (fixes issue #6975).
-rw-r--r--Lib/posixpath.py6
-rw-r--r--Lib/test/test_posixpath.py18
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