builtin_map(): Nailed memory leak. PyList_Append() borrows a

reference, so you have to DECREF the appended value.  This was a fun
one!

diff --git a/Python/bltinmodule.c b/Python/bltinmodule.c
index df6b98b42c401c572633f4bd1c16f33d969e888c..1929ae92ba7f3ca375f15f2105a97f971a12239e 100644 (file)
--- a/Python/bltinmodule.c
+++ b/Python/bltinmodule.c
@@ -903,6 +903,7 @@ builtin_map(self, args)
                if (i >= len) {
                        if (PyList_Append(result, value) < 0)
                                goto Fail_1;
+                       Py_DECREF(value);
                }
                else {
                        if (PyList_SetItem(result, i, value) < 0)