Message259224
Left a review.
Squidevil: You say if do_mkvalue() fails, the N object is not released. But looking at the code, I think it gets stored in the tuple, and then the tuple is released: <https://hg.python.org/cpython/file/v3.5.1/Python/modsupport.c#l182>. So the only leak I can see is if the tuple construction fails.
Serhiy’s patch fixes do_mktuple(), but I think the same problem would affect do_mkdict() and do_mklist() as well. |
|
Date |
User |
Action |
Args |
2016-01-29 23:57:13 | martin.panter | set | recipients:
+ martin.panter, serhiy.storchaka, josh.r, squidevil |
2016-01-29 23:57:13 | martin.panter | set | messageid: <1454111833.41.0.878498086836.issue26168@psf.upfronthosting.co.za> |
2016-01-29 23:57:13 | martin.panter | link | issue26168 messages |
2016-01-29 23:57:13 | martin.panter | create | |
|