[issue11397] os.path.realpath() may produce incorrect results

Serhiy Storchaka added the comment: Already fixed in issue6975. -- nosy: +serhiy.storchaka resolution: -> out of date stage: needs patch -> committed/rejected status: open -> closed ___ Python tracker

[issue11397] os.path.realpath() may produce incorrect results

Changes by Konstantin Nikitin : -- nosy: +stromsund ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail

[issue11397] os.path.realpath() may produce incorrect results

Changes by Vadim Fint : -- nosy: +Vadim.Fint ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python

[issue11397] os.path.realpath() may produce incorrect results

Martin Panter added the comment: Another infinite loop that isn't caught in Python 3.2.1: With the symbolic link link => link/inside a readlink("link") call will keep looping. Anyhow, the proposed solution in issue11397_py32_2.patch does not account for paths with multiple independent refe

[issue11397] os.path.realpath() may produce incorrect results

Changes by Santoso Wijaya : Added file: http://bugs.python.org/file21824/issue11397_py31_2.patch ___ Python tracker ___ ___ Python-bugs-list m

[issue11397] os.path.realpath() may produce incorrect results

Changes by Santoso Wijaya : Added file: http://bugs.python.org/file21823/issue11397_py27_2.patch ___ Python tracker ___ ___ Python-bugs-list m

[issue11397] os.path.realpath() may produce incorrect results

Changes by Santoso Wijaya : Added file: http://bugs.python.org/file21822/issue11397_py32_2.patch ___ Python tracker ___ ___ Python-bugs-list m

[issue11397] os.path.realpath() may produce incorrect results

Changes by Santoso Wijaya : Removed file: http://bugs.python.org/file21820/issue11397_py32_2.patch ___ Python tracker ___ ___ Python-bugs-list

[issue11397] os.path.realpath() may produce incorrect results

Santoso Wijaya added the comment: (For 3.2) Patch v2. Added some more corner case tests. -- Added file: http://bugs.python.org/file21820/issue11397_py32_2.patch ___ Python tracker _

[issue11397] os.path.realpath() may produce incorrect results

STINNER Victor added the comment: I tested issue11397_py32.patch: - you should use os.fsencode(sep) instead of sep.encode() - os.path.realpath('../'*10) raises IndexError('pop from empty list') instead of giving '/' -- ___ Python tracker

[issue11397] os.path.realpath() may produce incorrect results

Santoso Wijaya added the comment: Oh, I understand. I'm just wondering especially for this one because it has no assigned dev. -- ___ Python tracker ___ ___

[issue11397] os.path.realpath() may produce incorrect results

Terry J. Reedy added the comment: Sorry, patches sometimes sit awhile before a developer who can do something does do something. -- ___ Python tracker ___ _

[issue11397] os.path.realpath() may produce incorrect results

Santoso Wijaya added the comment: ping? -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.

[issue11397] os.path.realpath() may produce incorrect results

Changes by Santoso Wijaya : Added file: http://bugs.python.org/file21440/issue11397_py32.patch ___ Python tracker ___ ___ Python-bugs-list mai

[issue11397] os.path.realpath() may produce incorrect results

Santoso Wijaya added the comment: Attaching patches against 3.1 and 3.2 (due to unicode and slightly different unittest layout). -- Added file: http://bugs.python.org/file21439/issue11397_py31.patch ___ Python tracker

[issue11397] os.path.realpath() may produce incorrect results

Changes by Santoso Wijaya : -- components: +Library (Lib) -Extension Modules ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue11397] os.path.realpath() may produce incorrect results

Changes by STINNER Victor : -- nosy: +haypo ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.

[issue11397] os.path.realpath() may produce incorrect results

Santoso Wijaya added the comment: I'm attaching a patch that fixes this bug. I'm using a different algorithm altogether than the currently existing one, to allow a stack-based approach that lends itself to fixing this bug (as inspired by Wojciech). Hopefully it won't expose new, subtler bugs.

[issue11397] os.path.realpath() may produce incorrect results

Changes by Terry J. Reedy : -- stage: test needed -> needs patch ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue11397] os.path.realpath() may produce incorrect results

Santoso Wijaya added the comment: Attaching a unittest that will manifest this bug. -- keywords: +patch Added file: http://bugs.python.org/file21309/test_issue11397.patch ___ Python tracker ___

[issue11397] os.path.realpath() may produce incorrect results

Terry J. Reedy added the comment: 2.5 and 2.6 are only open for security issues. I am guessing that 2.7, 3.1, and 3.3 are affected. I cannot test as I only have Windows currently. -- nosy: +terry.reedy stage: -> test needed versions: +Python 2.7, Python 3.1, Python 3.3 -Python 2.5, Pyt

[issue11397] os.path.realpath() may produce incorrect results

Changes by Santoso Wijaya : -- nosy: +santa4nt ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyth

[issue11397] os.path.realpath() may produce incorrect results

Wojciech Muła added the comment: Confirmed in python3.2. Lib/posixpath.py/_resolve_link: path returned by readlink is normalized by normpath. In this case readlink('zlink') returns 'xlink/../blink', then normpath is called, and returns 'blink'. This ends processing because 'blink' isn't a link.

[issue11397] os.path.realpath() may produce incorrect results

Changes by Giampaolo Rodola' : -- nosy: +giampaolo.rodola ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http:

[issue11397] os.path.realpath() may produce incorrect results

New submission from Mikhail Kovtun : This bug appears in Python 2.4, 2.5, 2.6; not tested in Python 2.7. How to reproduce on Linux: {{{ $ mkdir ~/testsymlinks $ cd ~/testsymlinks $ mkdir adir $ ln -s ../adir adir/blink $ mkdir -p adir/cdir/ddir $ ln -s adir/cdir/ddir/.. xlink $ ln -s xlink/../b