win_readlink inModules/posixmodule.c mistakenly treats the PrintNameOffset field of the reparse data buffer as a number of characters instead of bytes. Thus, if the offset is non-zero, the value returned is incorrect stack garbage. For example, the following should return "C:\\ProgramData":    >>> os.readlink(r'C:\Users\All Users')    '\u6c20\u3012\u041f\x01\u2768\u60b2\u031b\x02\x05\x00\u031e\x06\u8c01\u4012'Craig Holmquist found this bug, as detailed in message 277385. He included a fix in the patch forissue 23407, but this should be addressed in the next maintenance release of 3.5 and 3.6.

New patch with test.I'm not sure if C:\Users\All Users and C:\ProgramData exist with those names on non-English installations of Windows.  I set the test to skip if they aren't found.

Thanks for the patch. I think it would be better to use the existing framework in Win32SymlinkTests to create a test file instead of using a hardcoded path like 'C:\Users\All Users' (e.g. use Win32SymlinkTests.filelink and Win32SymlinkTests.filelink_target to create a link)

os.symlink calls CreateSymbolicLink, which creates the reparse data buffer with the print name stored first, so the offset is always 0. Otherwise we would have noticed this problem already. For example:    >>> os.symlink('C:\\', 'link', True)    >>> os.system('fsutil reparsepoint query link')    Reparse Tag Value : 0xa000000c    Tag value: Microsoft    Tag value: Name Surrogate    Tag value: Symbolic Link    Reparse Data Length: 0x00000020    Reparse Data:    0000:  06 00 0e 00 00 00 06 00  00 00 00 00 43 00 3a 00  ............C.:.    0010:  5c 00 5c 00 3f 00 3f 00  5c 00 43 00 3a 00 5c 00  \.\.?.?.\.C.:.\.As you can see above, CreateSymbolicLink stores the "PrintName" DOS path (e.g. "C:\") first, with an offset of 0, followed by the "SubstituteName" NT path (e.g. "\??\C:\").The "All Users" link, on the other hand, seems to have been created manually with an inverted order. I have ctypes code to manually create a similar link (calling OpenProcessToken/AdjustTokenPrivileges to enable the required privilege and CreateFile/DeviceIoControl to set the reparse point), but I doubt it's worth adding it just to test this bug.

opened PRhttps://github.com/python/cpython/pull/5577

New changeset3c34aad4e7a95913ec7db8e5e948a8fc69047bf7 by Berker Peksag (SSE4) in branch 'master':bpo-29248: Fix os.readlink() on Windows (GH-5577)https://github.com/python/cpython/commit/3c34aad4e7a95913ec7db8e5e948a8fc69047bf7

New changeseta1d33f742515dc70ae99bc3ea1c851729522afc3 by Berker Peksag (Miss Islington (bot)) in branch '3.6':bpo-29248: Fix os.readlink() on Windows (GH-5577)https://github.com/python/cpython/commit/a1d33f742515dc70ae99bc3ea1c851729522afc3

New changeset74ebbaeb566dc10031756430ec5c896e56d0e491 by Miss Islington (bot) in branch '3.7':bpo-29248: Fix os.readlink() on Windows (GH-5577)https://github.com/python/cpython/commit/74ebbaeb566dc10031756430ec5c896e56d0e491

Thank you, Craig and SSE4.