Merge pull request #19403 from theoniko/theoniko-memory-leak

mattip · web-flow · commit b32b72e3d98d · 2021-07-09T09:09:22.000+03:00
BUG: Fix memory leak in function npyiter_multi_index_set
diff --git a/numpy/core/src/multiarray/number.c b/numpy/core/src/multiarray/number.c
@@ -429,6 +429,7 @@ is_scalar_with_conversion(PyObject *o2, double* out_exponent)
             return NPY_NOSCALAR;
         }
         val = PyLong_AsSsize_t(value);
+        Py_DECREF(value);
         if (error_converting(val)) {
             PyErr_Clear();
             return NPY_NOSCALAR;

Original file line number	Diff line number	Diff line change
`@@ -429,6 +429,7 @@ is_scalar_with_conversion(PyObject o2, double out_exponent)`
`429`	`429`	`return NPY_NOSCALAR;`
`430`	`430`	`}`
`431`	`431`	`val = PyLong_AsSsize_t(value);`
	`432`	`+ Py_DECREF(value);`
`432`	`433`	`if (error_converting(val)) {`
`433`	`434`	`PyErr_Clear();`
`434`	`435`	`return NPY_NOSCALAR;`