[issue30614] [EASY][2.7] testInitNonExistentFile() of test_bz2 leaks references

2017-06-10 Thread STINNER Victor
STINNER Victor added the comment: The bug should now be fixed. -- resolution: -> fixed stage: -> resolved status: open -> closed ___ Python tracker ___

[issue30614] [EASY][2.7] testInitNonExistentFile() of test_bz2 leaks references

2017-06-10 Thread STINNER Victor
STINNER Victor added the comment: New changeset 28288bebaf61269d1e26bb0795c2de6b481e1cf0 by Victor Stinner (Stéphane Wirtel) in branch '2.7': bpo-30614: testInitNonExistentFile() of test_bz2 leaks references (#2033) https://github.com/python/cpython/commit/28288bebaf61269d1e26bb0795c2de6b481e1c

[issue30614] [EASY][2.7] testInitNonExistentFile() of test_bz2 leaks references

2017-06-09 Thread Stéphane Wirtel
Changes by Stéphane Wirtel : -- pull_requests: +2098 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://ma

[issue30614] [EASY][2.7] testInitNonExistentFile() of test_bz2 leaks references

2017-06-09 Thread STINNER Victor
STINNER Victor added the comment: The leaking unit test was added to fix a bug found by fuzzing: see bpo-19878. To fix the bug, I suggest to: 1) Add a clear method: extract most of BZ2File_dealloc() code except of code to clear the lock and the tp_free. Create a new BZ2File_clear() function wh

[issue30614] [EASY][2.7] testInitNonExistentFile() of test_bz2 leaks references

2017-06-09 Thread Stéphane Wirtel
Changes by Stéphane Wirtel : -- pull_requests: +2097 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://ma

[issue30614] [EASY][2.7] testInitNonExistentFile() of test_bz2 leaks references

2017-06-09 Thread STINNER Victor
New submission from STINNER Victor: haypo@selma$ ./python -m test -R 3:3 -m testInitNonExistentFile test_bz2 Run tests sequentially 0:00:00 [1/1] test_bz2 beginning 6 repetitions 123456 .. test_bz2 leaked [10, 10, 10] references, sum=30 1 test failed: test_bz2 Total duration: 81 ms Tests