Fixed!  (test_site and test_functools are still leaking sporadically,
but it looks unrelated to the import.c leak).

-eric

On Thu, May 29, 2014 at 10:34 AM, Eric Snow <ericsnowcurren...@gmail.com> wrote:
> Good catch.  I'll look into it.
>
> -eric
>
> On Thu, May 29, 2014 at 8:49 AM, Brett Cannon <bcan...@gmail.com> wrote:
>> I think the memory leak was caused by
>> http://hg.python.org/cpython/rev/7d20e30bd540 because
>> http://hg.python.org/cpython/file/0eedac3d0b0a/Python/import.c#l903 sets the
>> 'res' variable and then overwrites it unconditionally w/o PY_DECREF
>> beforehand.
>>
>>
>> On Thu May 29 2014 at 4:02:17 AM, <solip...@pitrou.net> wrote:
>>>
>>> results for 8391f99208f6 on branch "default"
>>> --------------------------------------------
>>>
>>> test_collections leaked [0, 2, 4] references, sum=6
>>> test_collections leaked [0, 1, 2] memory blocks, sum=3
>>> test_functools leaked [0, 0, 3] memory blocks, sum=3
>>> test_importlib leaked [5, 5, 5] references, sum=15
>>> test_pkgutil leaked [1, 1, 1] references, sum=3
>>> test_site leaked [0, 0, 2] references, sum=2
>>> test_site leaked [0, 0, 2] memory blocks, sum=2
>>> test_zipimport leaked [47, 47, 47] references, sum=141
>>> test_zipimport_support leaked [2, 2, 2] references, sum=6
>>>
>>>
>>> Command line was: ['./python', '-m', 'test.regrtest', '-uall', '-R',
>>> '3:3:/home/antoine/cpython/refleaks/reflogDdbYzY', '-x']
>>> _______________________________________________
>>> Python-checkins mailing list
>>> python-check...@python.org
>>> https://mail.python.org/mailman/listinfo/python-checkins
_______________________________________________
Python-Dev mailing list
Python-Dev@python.org
https://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: 
https://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com

Reply via email to