2010-11-14 25 views
7

我在这里问,因为到目前为止我还没有得到OpenCV开发者的任何帮助。我将这个问题简化为一个非常简单的测试用例,所以任何具有CPython背景的人都可以在这里找到帮助。OpenCV:内存泄漏与Python界面,但不是在C版本

这种C代码不泄漏:

int main() { 
    while(true) { 
     int hist_size[] = {40}; 
     float range[] = {0.0f,255.0f}; 
     float* ranges[] = {range}; 
     CvHistogram* hist = cvCreateHist(1, hist_size, CV_HIST_ARRAY, ranges, 1); 
     cvReleaseHist(&hist); 
    } 
} 

这Python代码不会泄漏:

while True: cv.CreateHist([40], cv.CV_HIST_ARRAY, [[0,255]], 1) 

我通过CPython的代码搜索(OpenCVs当前SVN主干代码的),并发现这样的:

struct cvhistogram_t { 
    PyObject_HEAD 
    CvHistogram h; 
    PyObject *bins; 
}; 

...

/* cvhistogram */ 

static void cvhistogram_dealloc(PyObject *self) 
{ 
    cvhistogram_t *cvh = (cvhistogram_t*)self; 
    Py_DECREF(cvh->bins); 
    PyObject_Del(self); 
} 

static PyTypeObject cvhistogram_Type = { 
    PyObject_HEAD_INIT(&PyType_Type) 
    0,          /*size*/ 
    MODULESTR".cvhistogram",    /*name*/ 
    sizeof(cvhistogram_t),     /*basicsize*/ 
}; 

static PyObject *cvhistogram_getbins(cvhistogram_t *cvh) 
{ 
    Py_INCREF(cvh->bins); 
    return cvh->bins; 
} 

static PyGetSetDef cvhistogram_getseters[] = { 
    {(char*)"bins", (getter)cvhistogram_getbins, (setter)NULL, (char*)"bins", NULL}, 
    {NULL} /* Sentinel */ 
}; 

static void cvhistogram_specials(void) 
{ 
    cvhistogram_Type.tp_dealloc = cvhistogram_dealloc; 
    cvhistogram_Type.tp_getset = cvhistogram_getseters; 
} 

...

static PyObject *pycvCreateHist(PyObject *self, PyObject *args, PyObject *kw) 
{ 
    const char *keywords[] = { "dims", "type", "ranges", "uniform", NULL }; 
    PyObject *dims; 
    int type; 
    float **ranges = NULL; 
    int uniform = 1; 

    if (!PyArg_ParseTupleAndKeywords(args, kw, "Oi|O&i", (char**)keywords, &dims, &type, convert_to_floatPTRPTR, (void*)&ranges, &uniform)) { 
    return NULL; 
    } 
    cvhistogram_t *h = PyObject_NEW(cvhistogram_t, &cvhistogram_Type); 
    args = Py_BuildValue("Oi", dims, CV_32FC1); 
    h->bins = pycvCreateMatND(self, args); 
    Py_DECREF(args); 
    if (h->bins == NULL) { 
    return NULL; 
    } 
    h->h.type = CV_HIST_MAGIC_VAL; 
    if (!convert_to_CvArr(h->bins, &(h->h.bins), "bins")) 
    return NULL; 

    ERRWRAP(cvSetHistBinRanges(&(h->h), ranges, uniform)); 

    return (PyObject*)h; 
} 

而来自OpenCV的C头:

typedef struct CvHistogram 
{ 
    int  type; 
    CvArr* bins; 
    float thresh[CV_MAX_DIM][2]; /* For uniform histograms.      */ 
    float** thresh2;    /* For non-uniform histograms.     */ 
    CvMatND mat;     /* Embedded matrix header for array histograms. */ 
} 
CvHistogram; 

我完全不明白了一切,因为我从来没有与C-接口之前曾到Python。但是,我正在寻找的错误可能在这段代码中。

我对不对?或者我应该在哪里寻找错误?我将如何解决它?

(注意:有人看过这个问题的早期版本:我查看了错误的代码,他们的SWIG接口已被弃用,不再使用(但代码仍然存在于SVN中,这就是为什么我把它弄糊涂了所以不考虑interfaces/swig,这个代码是旧的和不使用。当前的代码住在modules/python


Upstream bug report: memleak in OpenCV Python CreateHist

回答

2

它已被修复。

更换3周前由jamesb

  • 状态从接受变更为设定为固定

固定的封闭

  • 分辨率r4526

    的范围参数没有被释放,并且范围上的迭代器未被DECREF处理。回归现在通过,原始循环不会泄漏。

  • 0

    我觉得你有垃圾收集问题,你永远不会离开回路。

    这是否更符合预期?

    while True: 
        cv.CreateHist([40], cv.CV_HIST_ARRAY, [[0,255]], 1) 
        cv = None 
    
    +0

    'CreateHist'返回一个对象。因为它没有分配给任何变量,它将在未来的某个时间被释放(它应该直接进入GC)。正如Python本身所做的那样,OpenCV必须另外引用一些东西。 – Albert 2011-01-23 16:32:02

    +0

    另外,'cv = None'与问题本身并不真正相关。 'cv'是一个模块,它在Python的退出时被正确释放。 – Albert 2011-01-23 16:33:46