7

これまでOpenCV開発者から何の助けも得られなかったので、ここで質問しています。私は問題を非常に単純なテストケースに減らしたので、おそらくCPythonのバックグラウンドを持っている人なら誰でもここで役立つでしょう。

この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)

(OpenCVの現在のSVNトランクコードの)CPythonコードを検索したところ、次のことがわかりました。

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;

これまでPythonへのCインターフェースを使用したことがなかったため、すべてを正確に理解しているわけではありません。しかし、おそらく私が探しているバグはこのコードのどこかにあります。

私は正しいですか?または、どこでバグを検索する必要がありますか?どうすれば修正できますか?

(この質問の以前のバージョンを見た人への注意:間違ったコードを見ました。彼らのSWIGインターフェースは非推奨になり、もう使用されていません(しかし、コードはまだSVNに残っていたので、混乱させました。だからしないでください。調べてみるとinterfaces/swig、このコードは古く、使用されていません。現在のコードはにありmodules/pythonます。)


アップストリームバグレポート:OpenCVPythonCreateHistのメモリリーク

4

2 に答える 2

2

修正されました。

3週間前にjamesbによって変更されました

  • ステータスが承認済みからクローズに変更されました
  • 解像度を固定に設定

r4526で修正

範囲パラメーターが解放されておらず、範囲の反復子がDECREFされていませんでした。回帰が通過し、元のループがリークしなくなりました。

于 2011-02-09T11:42:48.727 に答える
0

ループを離れることがないという点で、ガベージコレクションの問題があると思います。

これは期待どおりに機能しますか?

while True: 
    cv.CreateHist([40], cv.CV_HIST_ARRAY, [[0,255]], 1)
    cv = None
于 2011-01-21T22:42:54.260 に答える