[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 0/3] python: Fix memory leaks



On 12/31/2011 06:27 PM, ajia redhat com wrote:
Please ignore this.
From: Alex Jia<ajia redhat com>

This series patches fix memory leaks issues. Detected by valgrind.

Alex Jia (3):
* python/libvirt-override.c: fix memory leaks on libvirt_virDomainBlockStatsFlags.
* python/libvirt-override.c: fix memory leaks on libvirt_virDomainGetSchedulerParameters
   and libvirt_virDomainGetSchedulerParametersFlags.
* python/libvirt-override.c: fix memory leaks on libvirt_virDomainGetMemoryParameters.

Signed-off-by: Alex Jia<ajia redhat com>
---
  python/libvirt-override.c |    8 ++++++++
  1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/python/libvirt-override.c b/python/libvirt-override.c
index d2aad0f..a138090 100644
--- a/python/libvirt-override.c
+++ b/python/libvirt-override.c
@@ -176,6 +176,8 @@ libvirt_virDomainBlockStatsFlags(PyObject *self ATTRIBUTE_UNUSED,

          key = libvirt_constcharPtrWrap(params[i].field);
          PyDict_SetItem(info, key, val);
+        Py_DECREF(key);
+        Py_DECREF(val);
      }

      free(params);
@@ -369,6 +371,8 @@ libvirt_virDomainGetSchedulerParameters(PyObject *self ATTRIBUTE_UNUSED,

          key = libvirt_constcharPtrWrap(params[i].field);
          PyDict_SetItem(info, key, val);
+        Py_DECREF(key);
+        Py_DECREF(val);
      }
      free(params);
      return(info);
@@ -451,6 +455,8 @@ libvirt_virDomainGetSchedulerParametersFlags(PyObject *self ATTRIBUTE_UNUSED,

          key = libvirt_constcharPtrWrap(params[i].field);
          PyDict_SetItem(info, key, val);
+        Py_DECREF(key);
+        Py_DECREF(val);
      }
      free(params);
      return(info);
@@ -994,6 +1000,8 @@ libvirt_virDomainGetMemoryParameters(PyObject *self ATTRIBUTE_UNUSED,

          key = libvirt_constcharPtrWrap(params[i].field);
          PyDict_SetItem(info, key, val);
+        Py_DECREF(key);
+        Py_DECREF(val);
      }
      free(params);
      return(info);


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]