From: James Bowman <no@email>
Date: Tue, 25 Jan 2011 01:05:12 +0000 (+0000)
Subject: #489, leaking cvarrmat test and fix
X-Git-Tag: accepted/tizen/6.0/unified/20201030.111113~7838
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3a53d8667a523a9e451398d68126d6567b5bf1ca;p=platform%2Fupstream%2Fopencv.git

#489, leaking cvarrmat test and fix
---

diff --git a/modules/python/cv.cpp b/modules/python/cv.cpp
index 80df480..ffd35c3 100644
--- a/modules/python/cv.cpp
+++ b/modules/python/cv.cpp
@@ -1734,11 +1734,21 @@ static int convert_to_pts_npts_contours(PyObject *o, pts_npts_contours *dst, con
   return 1;
 }
 
-struct cvarrseq {
+class cvarrseq {
+public:
   union {
     CvSeq *seq;
     CvArr *mat;
   };
+  int freemat;
+  cvarrseq() {
+    freemat = false;
+  }
+  ~cvarrseq() {
+    if (freemat) {
+      cvReleaseMat((CvMat**)&mat);
+    }
+  }
 };
 
 static int is_convertible_to_mat(PyObject *o)
@@ -1782,6 +1792,7 @@ static int convert_to_cvarrseq(PyObject *o, cvarrseq *dst, const char *name = "n
     }
     assert(size != -1);
     CvMat *mt = cvCreateMat((int)PySequence_Fast_GET_SIZE(fi), 1, CV_32SC(size));
+    dst->freemat = true; // dealloc this mat when done
     for (Py_ssize_t i = 0; i < PySequence_Fast_GET_SIZE(fi); i++) {
       PyObject *e = PySequence_Fast_GET_ITEM(fi, i);
       PyObject *fe = PySequence_Fast(e, name);
diff --git a/tests/python/leak4.py b/tests/python/leak4.py
new file mode 100644
index 0000000..3268200
--- /dev/null
+++ b/tests/python/leak4.py
@@ -0,0 +1,9 @@
+import cv
+import math
+import time
+
+N=50000
+print "leak4"
+while True:
+    seq=list((i*1., i*1.) for i in range(N))
+    cv.Moments(seq)