fuite de mémoire Gdiplus

Version imprimable

Bonjour, j'ai fait un classe Screen qui charge un bitmap de l'image à l'écran, mais lorsque je fait une boucle contenant le constructeur et destructeur de cette classe, j'ai la ram qui se remplie. J'ai essayé de delete(scnBmpData) et delete(scnOct) mais ça fait cracher mon programme. Je suis débutant en programmation et ça fait 1 semaine que mon programme est bloqué à cause de ça. Si quelqu'un pouvait m'aider j'en serais reconaissant. Merci d'avance.

Code:

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
screen::screen()
{  
  // Donne les dimensions de l'écran
  int x1, y1, x2, y2;
  x1  = GetSystemMetrics(SM_XVIRTUALSCREEN);
  y1  = GetSystemMetrics(SM_YVIRTUALSCREEN);
  x2  = GetSystemMetrics(SM_CXVIRTUALSCREEN);
  y2  = GetSystemMetrics(SM_CYVIRTUALSCREEN);
  this->w   = x2 - x1;
  this->h   = y2 - y1;
 
  // copie l'écran dans le bitmap
  this->hScreen = GetDC(NULL);
  this->hDC     = CreateCompatibleDC(hScreen);
  this->hBitmap = CreateCompatibleBitmap(hScreen, w, h);
  HGDIOBJ old_obj = SelectObject(hDC, hBitmap);
  BOOL    bRet    = BitBlt(hDC, 0, 0, w, h, hScreen, x1, y1, SRCCOPY);
 
  //Charge le bitmap dans le buffer
  this->scnBmp = new Gdiplus::Bitmap(hBitmap,NULL);
  Gdiplus::Rect rectScnBmp(0,0, w, h);
  this->scnBmpData = new Gdiplus::BitmapData;
  scnBmp->LockBits(&rectScnBmp, Gdiplus::ImageLockModeRead,
		   PixelFormat32bppARGB, scnBmpData);
  this ->scnOct = (byte*)(void*)scnBmpData->Scan0;
}
screen::~screen()
{
  scnBmp->UnlockBits(scnBmpData);
  DeleteDC(hDC);
  ReleaseDC(NULL, hScreen);
  DeleteObject(hBitmap);
  delete(scnBmpData);
}

//ma boucle qui fait monter la ram, mais qui marche quand même.

Code:

1
2
3
4
5
6
7
8
 while(!pos.x && !pos.y)
  {
    screen monEcran;
    //pos = monEcran.findBmp(monbmp);
    monEcran.~screen();
    //if(pos.x && pos.y)
    //  cout << "trouvé en: (" << pos.x << ","<< pos.y << ")"<< endl;
  }

edit: j'ai trouvé (en fait non)

07/01/2017, 17h14
Bousk

Je vois au moins un new sans delete, sur scnBmp :calim2:

j'ai déplacé la partie buffer dans une autre partie du programme et j'ai delete le plus possible de choses mais ça n'a pas l'air de marcher, en fait dès que je lance le programme le programme atteint les 3go de ram très vite et il y reste. Et même avec un sleep(1000) il finit par les atteindre.

Code:

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
screen::screen()
{  
  // get screen dimensions
  int x1, y1, x2, y2;
  x1  = GetSystemMetrics(SM_XVIRTUALSCREEN);
  y1  = GetSystemMetrics(SM_YVIRTUALSCREEN);
  x2  = GetSystemMetrics(SM_CXVIRTUALSCREEN);
  y2  = GetSystemMetrics(SM_CYVIRTUALSCREEN);
  this->w   = x2 - x1;
  this->h   = y2 - y1;
 
  // copy screen to bitmap
  this->hScreen = GetDC(NULL);
  this->hDC     = CreateCompatibleDC(hScreen);
  this->hBitmap = CreateCompatibleBitmap(hScreen, w, h);
  HGDIOBJ old_obj = SelectObject(hDC, hBitmap);
  BOOL    bRet    = BitBlt(hDC, 0, 0, w, h, hScreen, x1, y1, SRCCOPY);
  this->scnBmp = new Gdiplus::Bitmap(hBitmap,NULL);
 
  DeleteObject(old_obj);
 }
screen::~screen()
{
  DeleteDC(hDC);
  ReleaseDC(NULL, hScreen);
  DeleteObject(hScreen);
  DeleteObject(hBitmap);  
  DeleteObject(scnBmp);
}

07/01/2017, 19h00
jo_link_noir

Pour quelle raison fais-tu un appel explicite au destructeur ?

L'objet est détruit et le destructeur appeler tout seul à la fin de sa vie/portée.
07/01/2017, 19h03
aperdemak

je savais pas ça merci, mais le problème vient pas de là.

edit: c'est à partir du bitblt que ça merde
07/01/2017, 20h06
jo_link_noir

Il manque le delete sur scnBmp, comme l'a indiqué Bousk.
07/01/2017, 20h22
aperdemak

j'ai déjà essayé ça fait cracher le programme, je pense le bitmap n'est pas libéré correctement car ça fuit de 9mo par passage dans la boucle.
07/01/2017, 20h54
foetus

Pas sûr à 100% :oops: mais je pense que c'est la variable old_obj qui n'est pas libérée.

Dans les codes Internet, cette variable est sauvegardée et après l'opération, elle est "remise en place" SelectObject(XXXDC, old_obj); afin d'avoir "le même état" avant et après.

Dans ton cas, il faut peut être juste faire un DeleteObject(old_obj);.

Édit: dans ta deuxième version tu le fais déjà :whistle:

Oui je viens de voir ça, par contre ça change rien. La seule chose efficace pour le moment c'est de commenter l'initialisation du bitmap.

Code:

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
screen::screen()
{  
  // get screen dimensions
  int x1, y1, x2, y2;
  x1  = GetSystemMetrics(SM_XVIRTUALSCREEN);
  y1  = GetSystemMetrics(SM_YVIRTUALSCREEN);
  x2  = GetSystemMetrics(SM_CXVIRTUALSCREEN);
  y2  = GetSystemMetrics(SM_CYVIRTUALSCREEN);
  this->w   = x2 - x1;
  this->h   = y2 - y1;
 
  // copy screen to bitmap
  this->hScreen = GetDC(NULL);
  this->hDC     = CreateCompatibleDC(hScreen);
  this->hBitmap = CreateCompatibleBitmap(hScreen, w, h);
  this->old_obj = SelectObject(hDC, hBitmap);
  BOOL    bRet    = BitBlt(hDC, 0, 0, w, h, hScreen, x1, y1, SRCCOPY);
  this->scnBmp = new Gdiplus::Bitmap(hBitmap,NULL); // si je commente= => plus de fuite
 }
screen::~screen()
{
  ReleaseDC(NULL, hScreen);
  DeleteDC(hScreen);
  SelectObject(hDC, old_obj);
  DeleteObject(hBitmap);
  DeleteObject(scnBmp);
  DeleteObject(old_obj);
  DeleteDC(hDC);
}

edit: le Bitmap se libère en faisant un delete apparemment, donc je comprends pas pourquoi ça crache, peut être qu'il se libère à la fin du constructeur mais je ne vois pas pourquoi

07/01/2017, 21h38
aperdemak

J'ai trouvé vos réponses font sens maintenant :D
En fait il fallait juste faire un delete scnBmp et supprimer l'appel du destructeur.
Merci bien !

Mon programme consomme do,c que 9mb et 1% cpu pour trouver un bmp à l'écran en temps réel, le lockbits est définivement meilleur que getpixel
08/01/2017, 11h07
Bousk

http://cpp.developpez.com/faq/cpp/?p...un-destructeur
http://cpp.developpez.com/faq/cpp/?p...r-d-une-classe
http://cpp.developpez.com/faq/cpp/?p...un-destructeur
http://cpp.developpez.com/faq/cpp/?p...objets-membres