Message179642
The fix introduced some refleaks:
$ ./python -m test -R3:2 test_xml_etree_c
test_xml_etree_c leaked [56, 56] references, sum=112
One seems to be in __getstate__:
diff --git a/Modules/_elementtree.c b/Modules/_elementtree.c
--- a/Modules/_elementtree.c
+++ b/Modules/_elementtree.c
@@ -859,8 +859,10 @@
PICKLED_ATTRIB, self->extra->attrib,
PICKLED_TEXT, self->text,
PICKLED_TAIL, self->tail);
- if (instancedict)
+ if (instancedict) {
+ Py_DECREF(children);
return instancedict;
+ }
else {
for (i = 0; i < PyList_GET_SIZE(children); i++)
Py_DECREF(PyList_GET_ITEM(children, i));
I'm still looking for the other. |
|
Date |
User |
Action |
Args |
2013-01-11 07:37:33 | ezio.melotti | set | recipients:
+ ezio.melotti, georg.brandl, jcea, larry, Arfrever, eli.bendersky, skrah, flox, santoso.wijaya, python-dev, einarfd, danielsh |
2013-01-11 07:37:32 | ezio.melotti | set | messageid: <1357889852.95.0.387817475438.issue16076@psf.upfronthosting.co.za> |
2013-01-11 07:37:32 | ezio.melotti | link | issue16076 messages |
2013-01-11 07:37:32 | ezio.melotti | create | |
|