decref results of PyStr_FromString

Where we create temporary objects (which are added to containers)
these objects already get there ref count incremented. In this case
we need to decref those objects to ensure they are released.

Signed-off-by: Noel Power <noel.power@suse.com>
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>

1 files changed, 12 insertions, 3 deletions

diff --git a/source4/param/provision.c b/source4/param/provision.c
index 10a657b981b..ebcc0cd77da 100644
--- a/source4/param/provision.c
+++ b/source4/param/provision.c
@@ -34,25 +34,34 @@
 static PyObject *provision_module(void)
 {
 	PyObject *name = PyStr_FromString("samba.provision");
+	PyObject *mod = NULL;
 	if (name == NULL)
 		return NULL;
-	return PyImport_Import(name);
+	mod = PyImport_Import(name);
+	Py_CLEAR(name);
+	return mod;
 }
 
 static PyObject *schema_module(void)
 {
 	PyObject *name = PyStr_FromString("samba.schema");
+	PyObject *mod = NULL;
 	if (name == NULL)
 		return NULL;
-	return PyImport_Import(name);
+	mod = PyImport_Import(name);
+	Py_CLEAR(name);
+	return mod;
 }
 
 static PyObject *ldb_module(void)
 {
 	PyObject *name = PyStr_FromString("ldb");
+	PyObject *mod = NULL;
 	if (name == NULL)
 		return NULL;
-	return PyImport_Import(name);
+	mod = PyImport_Import(name);
+	Py_CLEAR(name);
+	return mod;
 }
 
 static PyObject *PyLdb_FromLdbContext(struct ldb_context *ldb_ctx)