Форум программистов, компьютерный форум CyberForum.ru

Утечка памяти - C++

Восстановить пароль Регистрация
 
AlexTeos
1 / 1 / 0
Регистрация: 03.06.2012
Сообщений: 13
19.09.2013, 21:36     Утечка памяти #1
В небольшой программе с использованием OpenCV происходит утечка памяти, с чем связана не понимаю, прошу помощи, спасибо!

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
#include "stdafx.h"
#include <cv.h>
#include <highgui.h>
#include <iostream>
#include <vector>
#include <stdio.h>
#include <stdlib.h>
using namespace std;
 
class myFrame{
public:
    myFrame(){}
    myFrame(IplImage* _image) {
        image=_image;
        originalimage = cvCloneImage(image);
    }
    IplImage* image;
    IplImage* originalimage;
};
 
class myVideoStream{
public:
    myVideoStream(CvCapture* _capture){capture=_capture;}
    //Берем новый кадр
    IplImage* myGetNewImageFromQuery(){
        return cvQueryFrame(capture);
    }
 
    //Применяем пороговое преобразование
    void myThresholdConvert(IplImage* image) //Черно-Белый+Сглаживание+Пороговое+Границы
    {
        int trs1=125,trs2=50;
        int trs=80;
        IplImage* dst = 0;
        dst = cvCreateImage( cvGetSize(image), IPL_DEPTH_8U, 1 );
        cvConvertImage(image,dst,0); //В черно белый
        cvSmooth(dst, dst, CV_GAUSSIAN, 3, 3); // сглаживаем изображение
        cvThreshold(dst,dst, trs, 255, CV_THRESH_BINARY_INV); //Пороговое преобразование
        cvCanny(dst, dst, trs1, trs2, 3); //Выделяем границы
        cvConvertImage(dst,image,0);
 
        cvReleaseImage(&dst);
    }
 
    CvCapture* capture;
    myFrame frame;
};
 
int main(int argc, char* argv[]) {
 
    cvNamedWindow( "Capture", 1 );
    cvMoveWindow("Capture",1376/2+100,200);
 
    CvCapture* capture = cvCreateFileCapture("test5_4_2.mp4" );
    myVideoStream stream(capture);
 
    int i=0;
    while(i<353){
        i++;
        stream.frame=cvQueryFrame(capture);
        stream.myThresholdConvert(stream.frame.image);
 
        cvShowImage( "Capture",stream.frame.originalimage);
 
        cvWaitKey(1);
    }
 
    cvReleaseCapture( &capture );
    cvDestroyWindow("capture");
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
19.09.2013, 21:36     Утечка памяти
Посмотрите здесь:

Утечка памяти C++
Утечка памяти?! C++
C++ Утечка памяти
Утечка памяти C++
Утечка памяти C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
19.09.2013, 23:59     Утечка памяти #2
Утечка у вас из-за этого:
C++
1
originalimage = cvCloneImage(image);
Зачем это?
AlexTeos
1 / 1 / 0
Регистрация: 03.06.2012
Сообщений: 13
20.09.2013, 00:17  [ТС]     Утечка памяти #3
Цитата Сообщение от alsav22 Посмотреть сообщение
Утечка у вас из-за этого:
C++
1
originalimage = cvCloneImage(image);
Зачем это?
Для дальнейшей работы мне необходим оригинал изображения. Утечка исчезла, большое спасибо. Но почему эта операция ее вызывает?
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
20.09.2013, 00:38     Утечка памяти #4
Можно вот так ещё сделать:
C++
1
2
3
4
5
6
7
8
9
10
11
while(i<353){
        i++;
        stream.frame=cvQueryFrame(capture);
        stream.myThresholdConvert(stream.frame.image);
 
        cvShowImage( "Capture",stream.frame.originalimage);
        
        cvReleaseImage(&stream.frame.originalimage);
 
        cvWaitKey(1);
  }
DU
1477 / 1053 / 45
Регистрация: 05.12.2011
Сообщений: 2,279
20.09.2013, 00:47     Утечка памяти #5
потому что клонирование - это создание копии объекта. динамическое создание. а удаления нет.
почитайте что такое RAII. в новом стандарте стандартные птры можно адаптировать под всякие подобные библиотеки:

пример из вашего кода:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
 IplImage* dst = cvCreateImage( cvGetSize(image), IPL_DEPTH_8U, 1 );
 // тут куча всего прочего.
 // после этого нужно не забыть позвать релиз. иначе утечка.
 // а если будет исключение, то релиз не выполнится и снова будет утечка.
 cvReleaseImage(&dst);
 
 
//////////////////////////////////////////////
в новом стандарте нужно написать класс - освобождалку:
struct IplImageReleaser
{
  void operator () (IplImage* img) const
  {
    if (img != nullptr)
    {
      cvReleaseImage(&img);
    }
  }
};
 
потом делаем тайпдеф:
typedef std::unique_ptr<IplImage, IplImageReleaser> ImplImageUniquePtr;
// тоже можно сделать и для shared_ptr, если нужно будет копировать такой объект.
 
// после этого создаем:
ImplImageUniquePtr dst(cvCreateImage( cvGetSize(image), IPL_DEPTH_8U, 1 ));
//тут делаем что хотим. при выходе за пределы видимости, позовется
//деструктор ImplImageUniquePtr, который вызовет наш релизер.
//единсвенный минус - немного более многословно другие функции вызывать
 
  cvSmooth(dst.get(), dst.get(), CV_GAUSSIAN, 3, 3); // сглаживаем изображение
  cvThreshold(dst.get(),dst.get(), trs, 255, CV_THRESH_BINARY_INV); //Пороговое преобразование
  cvCanny(dst.get(), dst.get(), trs1, trs2, 3); //Выделяем границы
 
  // для других объектов тоже подобные релизеры можно написать.
Yandex
Объявления
20.09.2013, 00:47     Утечка памяти
Ответ Создать тему
Опции темы

Текущее время: 10:51. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru