tursom/memgraph
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix memory leaks in Py*Iter* methods

Browse Source 
Reviewers: teon.banek

Reviewed By: teon.banek

Subscribers: pullbot

Differential Revision: https://phabricator.memgraph.io/D2699
This commit is contained in:
Ivan Paljak 2020-03-02 13:31:47 +01:00
parent 14c4c6d060
commit ec0caad44c
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/query/procedure/py_module.cpp
View File

@ -176,6 +176,7 @@ PyObject *PyGraphIterVertices(PyGraph *self, PyObject *Py_UNUSED(ignored)) {
auto *py_vertices_it =
PyObject_New(PyVerticesIterator, &PyVerticesIteratorType);
if (!py_vertices_it) {
mgp_vertices_iterator_destroy(vertices_it);
PyErr_SetString(PyExc_MemoryError,
"Unable to allocate _mgp.VerticesIterator.");
return nullptr;
@ -702,6 +703,7 @@ PyObject *PyVertexIterInEdges(PyVertex *self, PyObject *Py_UNUSED(ignored)) {
}
auto *py_edges_it = PyObject_New(PyEdgesIterator, &PyEdgesIteratorType);
if (!py_edges_it) {
mgp_edges_iterator_destroy(edges_it);
PyErr_SetString(PyExc_MemoryError,
"Unable to allocate _mgp.EdgesIterator for in edges.");
return nullptr;
@ -727,6 +729,7 @@ PyObject *PyVertexIterOutEdges(PyVertex *self, PyObject *Py_UNUSED(ignored)) {
}
auto *py_edges_it = PyObject_New(PyEdgesIterator, &PyEdgesIteratorType);
if (!py_edges_it) {
mgp_edges_iterator_destroy(edges_it);
PyErr_SetString(PyExc_MemoryError,
"Unable to allocate _mgp.EdgesIterator for out edges.");
return nullptr;