[issue2222] Memory leak in os.rename?

Alexander Belopolsky report at bugs.python.org
Tue Mar 4 18:14:23 CET 2008


Alexander Belopolsky added the comment:

On the test_os patch: I would rename LeakTest to ErrorTest and leave it 
on for all platforms.  BTW, I cannot produce a definitive proof that 
posix_2str leaks when second conversion fails (regrtest -R does not 
catch it because the leaked buffer is not a PyObject).  Nevertheless, it 
would be a good idea to add tests that exercise os.rename("foo", 0), 
os.link("foo", 0) and os.symlink("foo", 0) errors on all platforms.

__________________________________
Tracker <report at bugs.python.org>
<http://bugs.python.org/issue2222>
__________________________________


More information about the Python-bugs-list mailing list