diff -r 09a8ee0f7585 Lib/test/test_xml_etree_c.py --- a/Lib/test/test_xml_etree_c.py Wed Dec 14 19:56:53 2016 +0200 +++ b/Lib/test/test_xml_etree_c.py Wed Dec 14 20:46:55 2016 +0200 @@ -11,6 +11,7 @@ cET_alias = import_fresh_module('xml.etr fresh=['_elementtree', 'xml.etree']) +@unittest.skipUnless(cET, 'requires _elementtree') class MiscTests(unittest.TestCase): # Issue #8651. @support.bigmemtest(size=support._2G + 100, memuse=1, dry_run=False) @@ -54,6 +55,15 @@ class MiscTests(unittest.TestCase): del element.attrib self.assertEqual(element.attrib, {'A': 'B', 'C': 'D'}) + def test_trashcan(self): + # If this test fails, it will most likely die via segfault. + e = root = cET.Element('root') + for i in range(200000): + e = cET.SubElement(e, 'x') + del e + del root + support.gc_collect() + @unittest.skipUnless(cET, 'requires _elementtree') class TestAliasWorking(unittest.TestCase): diff -r 09a8ee0f7585 Modules/_elementtree.c --- a/Modules/_elementtree.c Wed Dec 14 19:56:53 2016 +0200 +++ b/Modules/_elementtree.c Wed Dec 14 20:46:55 2016 +0200 @@ -626,6 +626,7 @@ static void element_dealloc(ElementObject* self) { PyObject_GC_UnTrack(self); + Py_TRASHCAN_SAFE_BEGIN(self) if (self->weakreflist != NULL) PyObject_ClearWeakRefs((PyObject *) self); @@ -636,6 +637,7 @@ element_dealloc(ElementObject* self) RELEASE(sizeof(ElementObject), "destroy element"); Py_TYPE(self)->tp_free((PyObject *)self); + Py_TRASHCAN_SAFE_END(self) } /* -------------------------------------------------------------------- */