2011-04-10 155 views
1

这个drawItem函数通过一个循环被多次调用,每次调用它时都会遇到内存泄漏问题。我在想这个问题是由resizeImage()函数引起的,但我似乎无法找到问题所在,这是C++ \ CLI与OpenCV库的关系。OpenCV内存泄漏问题

drawItem() 
{ 
imgItem = resizeImage(imgItem, newItemWidth, newItemHeight, false); 
imgMask = resizeImage(imgMask, newItemWidth, newItemHeight, false); 
cvSetImageROI(image3, cvRect(x1,y1,newItemWidth, newItemHeight)); 
cvCopy(imgItem, image3, imgMask); 
cvResetImageROI(image3); 
cvReleaseImage(&imgItem); 
cvReleaseImage(&imgMask); 
} 

IplImage* resizeImage(const IplImage *origImg, int newWidth, int newHeight, bool keepAspectRatio) 
{ 
    IplImage *outImg = 0; 
    int origWidth; 
    int origHeight; 
    if (origImg) { 
     origWidth = origImg->width; 
     origHeight = origImg->height; 
    } 
    if (newWidth <= 0 || newHeight <= 0 || origImg == 0 
     || origWidth <= 0 || origHeight <= 0) { 
     //cerr << "ERROR: Bad desired image size of " << newWidth 
     // << "x" << newHeight << " in resizeImage().\n"; 
     exit(1); 
    } 

    if (keepAspectRatio) { 
     // Resize the image without changing its aspect ratio, 
     // by cropping off the edges and enlarging the middle section. 
     CvRect r; 
     // input aspect ratio 
     float origAspect = (origWidth/(float)origHeight); 
     // output aspect ratio 
     float newAspect = (newWidth/(float)newHeight); 
     // crop width to be origHeight * newAspect 
     if (origAspect > newAspect) { 
      int tw = (origHeight * newWidth)/newHeight; 
      r = cvRect((origWidth - tw)/2, 0, tw, origHeight); 
     } 
     else { // crop height to be origWidth/newAspect 
      int th = (origWidth * newHeight)/newWidth; 
      r = cvRect(0, (origHeight - th)/2, origWidth, th); 
     } 
     IplImage *croppedImg = cropImage(origImg, r); 

     // Call this function again, with the new aspect ratio image. 
     // Will do a scaled image resize with the correct aspect ratio. 
     outImg = resizeImage(croppedImg, newWidth, newHeight, false); 
     cvReleaseImage(&croppedImg); 

    } 
    else { 

     // Scale the image to the new dimensions, 
     // even if the aspect ratio will be changed. 
     outImg = cvCreateImage(cvSize(newWidth, newHeight), 
      origImg->depth, origImg->nChannels); 
     if (newWidth > origImg->width && newHeight > origImg->height) { 
      // Make the image larger 
      cvResetImageROI((IplImage*)origImg); 
      // CV_INTER_LINEAR: good at enlarging. 
      // CV_INTER_CUBIC: good at enlarging.   
      cvResize(origImg, outImg, CV_INTER_LINEAR); 
     } 
     else { 
      // Make the image smaller 
      cvResetImageROI((IplImage*)origImg); 
      // CV_INTER_AREA: good at shrinking (decimation) only. 
      cvResize(origImg, outImg, CV_INTER_AREA); 
     } 

    } 
    return outImg; 
} 

IplImage* cropImage(const IplImage *img, const CvRect region) 
{ 
    IplImage *imageCropped; 
    CvSize size; 

    if (img->width <= 0 || img->height <= 0 
     || region.width <= 0 || region.height <= 0) { 
     //cerr << "ERROR in cropImage(): invalid dimensions." << endl; 
     exit(1); 
    } 

    if (img->depth != IPL_DEPTH_8U) { 
     //cerr << "ERROR in cropImage(): image depth is not 8." << endl; 
     exit(1); 
    } 

    // Set the desired region of interest. 
    cvSetImageROI((IplImage*)img, region); 
    // Copy region of interest into a new iplImage and return it. 
    size.width = region.width; 
    size.height = region.height; 
    imageCropped = cvCreateImage(size, IPL_DEPTH_8U, img->nChannels); 
    cvCopy(img, imageCropped); // Copy just the region. 

    return imageCropped; 
} 

回答

3

我发现了这个问题,这是由于多种内存分配outImage。 imgItem是指向之前的东西,但是在我做了之后。

imgItem现在指向另一件事情,创建imgItem时创建的内存永远丢失,没有变量指向它,因此内存泄漏。

所以我用tempImgItem来解决问题,

tempImgItem = resizeImage(imgItem, newItemWidth, newItemHeight, false); 
tempImgMask = resizeImage(imgMask, newItemWidth, newItemHeight, false); 
cvReleaseImage(&imgItem); 
cvReleaseImage(&imgMask); 
imgItem = tempImgItem; 
imgMask = tempImgMask; 
tempImgItem = NULL; 
tempImgMask = NULL; 
1

我想这是因为你本身就是所说的“resizeImage”,你忘了释放内部

+0

我试图把一个cvReleaseImage(&imgItem);在IplImage * outImg = 0之后;但它仍然泄漏内存:/ – Qmage 2011-04-12 12:04:27

+0

我的意思是(&outImg)。我也不能把cvReleaseImage(&outImg)在返回之前,程序会崩溃 – Qmage 2011-04-12 12:10:36

+0

我相信你是正确的,我必须释放outImg,因为我用cvCreateImage创建它,它必须被释放。但现在的问题是我如何释放它,因为我需要“返回”它:/ – Qmage 2011-04-12 12:28:35