Correct a memory leak: the range() object was not properly freed.

amauryfa · amauryfa · commit a1e9ec4e5515 · 2007-11-15T19:16:08.000Z
diff --git a/Objects/object.c b/Objects/object.c
@@ -1509,6 +1509,9 @@ _Py_ReadyTypes(void)
 
 	if (PyType_Ready(&PyStdPrinter_Type) < 0)
 		Py_FatalError("Can't initialize StdPrinter");
+
+	if (PyType_Ready(&PyRange_Type) < 0)
+		Py_FatalError("Can't initialize 'range'");
 }
 
 
diff --git a/Objects/rangeobject.c b/Objects/rangeobject.c
@@ -107,6 +107,7 @@ range_dealloc(rangeobject *r)
     Py_DECREF(r->start);
     Py_DECREF(r->stop);
     Py_DECREF(r->step);
+    Py_Type(r)->tp_free(r);
 }
 
 /* Return number of items in range (lo, hi, step), when arguments are

Original file line number	Diff line number	Diff line change
`@@ -1509,6 +1509,9 @@ _Py_ReadyTypes(void)`
`1509`	`1509`
`1510`	`1510`	`if (PyType_Ready(&PyStdPrinter_Type) < 0)`
`1511`	`1511`	`Py_FatalError("Can't initialize StdPrinter");`
	`1512`	`+`
	`1513`	`+ if (PyType_Ready(&PyRange_Type) < 0)`
	`1514`	`+ Py_FatalError("Can't initialize 'range'");`
`1512`	`1515`	`}`
`1513`	`1516`
`1514`	`1517`
Original file line number	Diff line number	Diff line change
`@@ -107,6 +107,7 @@ range_dealloc(rangeobject *r)`
`107`	`107`	`Py_DECREF(r->start);`
`108`	`108`	`Py_DECREF(r->stop);`
`109`	`109`	`Py_DECREF(r->step);`
	`110`	`+ Py_Type(r)->tp_free(r);`
`110`	`111`	`}`
`111`	`112`
`112`	`113`	`/* Return number of items in range (lo, hi, step), when arguments are`