pyldb: Don't use the internal macro PyObject_REPR

Signed-off-by: Petr Viktorin <[email protected]> Reviewed-by: Andrew Bartlett <[email protected]> Reviewed-by: Stefan Metzmacher <[email protected]>
darabosj · Nov 5, 2015 · 2e298c1 · 2e298c1
1 parent 7935796
commit 2e298c1
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/lib/ldb/pyldb.c b/lib/ldb/pyldb.c
@@ -3002,10 +3002,16 @@ static PyGetSetDef py_ldb_msg_getset[] = {
 
 static PyObject *py_ldb_msg_repr(PyLdbMessageObject *self)
 {
-	PyObject *dict = PyDict_New(), *ret;
+	PyObject *dict = PyDict_New(), *ret, *repr;
 	if (PyDict_Update(dict, (PyObject *)self) != 0)
 		return NULL;
-	ret = PyString_FromFormat("Message(%s)", PyObject_REPR(dict));
+	repr = PyObject_Repr(dict);
+	if (repr == NULL) {
+		Py_DECREF(dict);
+		return NULL;
+	}
+	ret = PyString_FromFormat("Message(%s)", PyString_AsString(repr));
+	Py_DECREF(repr);
 	Py_DECREF(dict);
 	return ret;
 }