2011-10-20 Phil Muldoon <pmuldoon@redhat.com>
authorPhil Muldoon <pmuldoon@redhat.com>
Thu, 20 Oct 2011 13:34:17 +0000 (13:34 +0000)
committerPhil Muldoon <pmuldoon@redhat.com>
Thu, 20 Oct 2011 13:34:17 +0000 (13:34 +0000)
        PR python/13308
        PR python/13309

* python/py-breakpoint.c (gdbpy_breakpoints): Fix List reference
leak.
* python/py-inferior.c (gdbpy_inferiors): Fix List reference
leak.  Delete unused variables.

gdb/ChangeLog
gdb/python/py-breakpoint.c
gdb/python/py-inferior.c

index 154aee5..c9d5b26 100644 (file)
@@ -1,5 +1,15 @@
 2011-10-20  Phil Muldoon  <pmuldoon@redhat.com>
 
+        PR python/13308
+        PR python/13309
+
+       * python/py-breakpoint.c (gdbpy_breakpoints): Fix List reference
+       leak.
+       * python/py-inferior.c (gdbpy_inferiors): Fix List reference
+       leak.  Delete unused variables.
+
+2011-10-20  Phil Muldoon  <pmuldoon@redhat.com>
+
        PR python/12656
 
        * python/py-frame.c (frapy_read_var): Use const struct *block.
index 87265e2..daceb99 100644 (file)
@@ -697,7 +697,7 @@ build_bp_list (struct breakpoint *b, void *arg)
 PyObject *
 gdbpy_breakpoints (PyObject *self, PyObject *args)
 {
-  PyObject *list;
+  PyObject *list, *tuple;
 
   if (bppy_live == 0)
     Py_RETURN_NONE;
@@ -715,7 +715,10 @@ gdbpy_breakpoints (PyObject *self, PyObject *args)
       return NULL;
     }
 
-  return PyList_AsTuple (list);
+  tuple = PyList_AsTuple (list);
+  Py_DECREF (list);
+
+  return tuple;
 }
 
 /* Call the "stop" method (if implemented) in the breakpoint
index afc3eb0..d6086dc 100644 (file)
@@ -377,9 +377,7 @@ build_inferior_list (struct inferior *inf, void *arg)
 PyObject *
 gdbpy_inferiors (PyObject *unused, PyObject *unused2)
 {
-  int i = 0;
-  PyObject *list, *inferior;
-  struct inferior *inf;
+  PyObject *list, *tuple;
 
   list = PyList_New (0);
   if (!list)
@@ -391,7 +389,10 @@ gdbpy_inferiors (PyObject *unused, PyObject *unused2)
       return NULL;
     }
 
-  return PyList_AsTuple (list);
+  tuple = PyList_AsTuple (list);
+  Py_DECREF (list);
+
+  return tuple;
 }
 
 /* Membuf and memory manipulation.  */