[issue9927] Leak around GetFinalPathNameByHandle (Windows)
Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp added the comment: I'll close this entry because #12084 contains this fix. -- status: open - closed superseder: - os.stat() on windows doesn't consider relative symlink ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9927 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9927] Leak around GetFinalPathNameByHandle (Windows)
Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp added the comment: This is updated version. Can you test this? (I only fixed leak, deferred other fixes to future) -- Added file: http://bugs.python.org/file19985/py3k_fix_leak_around_GetFinalPathNameByHandle_v2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9927 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9927] Leak around GetFinalPathNameByHandle (Windows)
Changes by Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp: Removed file: http://bugs.python.org/file18979/py3k_fix_leak_around_GetFinalPathNameByHandle.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9927 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9927] Leak around GetFinalPathNameByHandle (Windows)
New submission from Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp: I've found there are leaks around GetFinalPathNameByHandle in Modules/posixmodule.c. (Leaks when function fails) I hope attached patch will fix this. -- components: Windows files: py3k_fix_leak_around_GetFinalPathNameByHandle.patch keywords: patch messages: 117197 nosy: ocean-city priority: normal severity: normal status: open title: Leak around GetFinalPathNameByHandle (Windows) versions: Python 3.2 Added file: http://bugs.python.org/file18979/py3k_fix_leak_around_GetFinalPathNameByHandle.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9927 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9927] Leak around GetFinalPathNameByHandle (Windows)
Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp added the comment: About buffer size, GetFinalPathNameByHandle may return the length or string or the size of required buffer size (the length of striing + 1) depending on platforms and ANSI/WIDE version. If function returns the length of string buffer size, buf_size + 1 is buffer size. MSDN saids cchFilePath [in] The size of lpszFilePath, in TCHARs. This value does not include a NULL termination character. so we must pass buf_size + 1 for GetFinalPathNameByHandle. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9927 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9927] Leak around GetFinalPathNameByHandle (Windows)
Changes by Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp: -- Removed message: http://bugs.python.org/msg117199 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9927 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9927] Leak around GetFinalPathNameByHandle (Windows)
Changes by Brian Curtin cur...@acm.org: -- components: +Extension Modules nosy: +brian.curtin, jaraco stage: - patch review type: - resource usage ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9927 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9927] Leak around GetFinalPathNameByHandle (Windows)
Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp added the comment: I cannot test this directly. Thank you. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9927 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9927] Leak around GetFinalPathNameByHandle (Windows)
Brian Curtin cur...@acm.org added the comment: The tests pass here, but I haven't thoroughly reviewed the patch yet. After a quick glance it generally looks ok, although I wish we didn't have so much duplication. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9927 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com