cvReshapeMatND after cvCreateMatND,memory can not be released.

asked 2014-02-17 03:19:20 -0600

updated 2014-02-17 06:05:23 -0600

I use OpenCV 2.4.8 with VS2010. When I try to release some memory using the following code, the memory is not released. I think something went wrong with the cvReshapeMatND() function. If you comment this function out, the memory can be released.(Please open the windows task manager and watch the memory of the excution.) Please shed me some light about this problem, thank you very much!

#include "stdafx.h"
#include <opencv.hpp>
#include <Windows.h>

using namespace std;
using namespace cv;

int _tmain(int argc, _TCHAR* argv[])
{
    Mat Ca = Mat::zeros(7000, 10000, CV_32FC1);
    for(int i = 0; i < 1000; i++)
    {
        int new_size[2] = {7000, 10000};    

        CvMatND TpCa = Ca;
        CvMatND *Caimg1 = cvCreateMatND(2, new_size, CV_32F);

        cvReshapeMatND(&TpCa, sizeof(CvMatND), Caimg1, 0, 2, new_size);

        cout<<"Before"<<endl;
        Sleep(2000);    
        cvReleaseMatND(&Caimg1);
        cout<<"After"<<endl;
        Sleep(2000);
    }
    return 0;
}

edit retag flag offensive close merge delete

add a comment

1

answered 2014-02-17 03:34:52 -0600

tuannhtn

1500 ●10 ●19 http://4fire.wordpress...

updated 2014-02-17 03:35:38 -0600

If you comment out the release function call, the memory is released because your variable Caimg1 is a local one, it will be free automatically. Your code contains error that can lead to a memory leak: CvMatND *Caimg1 = cvCreateMatND(2, new_size, CV_32F); this statement allocates a memory and Caimg1 points to that, but the next statement: cvReshapeMatND(&TpCa, sizeof(CvMatND), Caimg1, 0, 2, new_size); will make Caimg1 points to memory data of TpCa, this causes the memory unmanaged. Indeed, the function cvReshapeMatND create a new header matrix and changes the data pointer (see more at https://github.com/Itseez/opencv/blob/master/modules/core/src/array.cpp).

edit flag offensive delete link

Comments

Thank you! You are right! This is indeed why the memory is lost somehow... I figured out the solution:use cvCreateMatNDHeader() instead of the cvCreateMatND(), that solved the problem.

amazingguo ( 2014-02-17 05:48:57 -0600 )edit

add a comment

0

answered 2014-02-17 03:51:11 -0600

berak
32993 ●7 ●81 ●312

updated 2014-02-17 03:55:22 -0600

consistent usage of the c++ api will solve all your problems:

 Mat Ca = Mat::zeros(7000, 10000, CV_32FC1);
 Mat Cb = Ca.reshape(1,7000);

that's it already. no manual allocation, no manual release required.

(also you don't need MatND anymore, its an alias for Mat)

do yourself (and us) a favour, and get rid of the outdated c-api calls there

edit flag offensive delete link

Comments

Thank you for you answer! I know Mat class can automatically manage the memory which is more convinient and safer. But I really needs a method to convert a 3-dimensional Mat to a 2-dimensional Mat, so I have to use that CvMatND struct as a bridge to do the convertion. As far as I know the cvReshapeMatND() is the only method that can do so(the normal reshape method seems to only deal with the 2-dimensional Mat). The program above is only a small demo, not my original code. It will be very appreciated if you can give a better sollution, thank you very much!

amazingguo ( 2014-02-17 05:58:56 -0600 )edit

you probably need to show your 3-dimensional Mat then.

berak ( 2014-02-17 06:18:48 -0600 )edit

This is how I do the 3-dimensional to 2-dimensional convert for now(Ca -> Ca2) I wonder if there is any better solution.

int mat_size[3] = {100, 700, 1000};
Mat Ca = Mat(3, mat_size, CV_32FC1);

int new_size[2] = {7000, 10000};    

CvMatND TpCa = Ca;
CvMatND *Caimg1 = cvCreateMatNDHeader(2, new_size, CV_32F);

Caimg1 = (CvMatND*)cvReshapeMatND(&amp;TpCa, sizeof(CvMatND), Caimg1, 0, 2, new_size);

Mat Ca2(Caimg1);

amazingguo ( 2014-02-17 19:59:05 -0600 )edit

add a comment

cvReshapeMatND after cvCreateMatND,memory can not be released.

2 answers

Comments

Comments

Links

Question Tools

Stats

Related questions

cvReshapeMatND after cvCreateMatND,memory can not be released. edit

2 answers

Comments

Comments

Links

Question Tools

Stats

Related questions

cvReshapeMatND after cvCreateMatND,memory can not be released.