[issue10383] test_os leaks under Windows

2010-11-27 Thread Stefan Krah
Stefan Krah added the comment: Fixed original leaks in r86804, r86806 and r86804. Fixed additional leaks in r86829. -- components: +Extension Modules -Library (Lib), Windows resolution: -> fixed stage: -> committed/rejected status: open -> closed versions: +Python 2.7, Python 3.1 __

[issue10383] test_os leaks under Windows

2010-11-14 Thread Stefan Krah
Stefan Krah added the comment: I agree that the buffer should be released. The patch fixes this leak and another one. All tests pass. -- keywords: +patch nosy: +skrah Added file: http://bugs.python.org/file19606/posixmodule_leak.patch ___ Python trac

[issue10383] test_os leaks under Windows

2010-11-11 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc added the comment: This looks suspect (in function posix_write):: if (!PyArg_ParseTuple(args, "iy*:write", &fd, &pbuf)) return NULL; if (!_PyVerify_fd(fd)) return posix_error(); I'd prefer a "PyBuffer_Release(&pbuf);" before returning... Many other

[issue10383] test_os leaks under Windows

2010-11-11 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc added the comment: This leaks one reference every time: os.write(123456, b"b") -- ___ Python tracker ___ ___

[issue10383] test_os leaks under Windows

2010-11-11 Thread Antoine Pitrou
New submission from Antoine Pitrou : Y:\>py3k\__svn__\PCbuild\python_d.exe -m test.regrtest -R 3:2 test_os [1/1] test_os [33558 refs] beginning 5 repetitions 12345 [33558 refs] .[33558 refs] .[33558 refs] .[33558 refs] .[33558 refs] . test_os leaked [3, 3] references, sum=6 1 test failed: tes