[Résolu] Fuite mémoire, que libérer ?

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [Résolu] Fuite mémoire, que libérer ?

kyom

Bonjour,

Je dois avoir du mal pour libérer mes structs, pointeurs et autre parce que mon programme a de grosse fuite mémoire (quelques giga en quelques secondes). Je crois avoir isoler le probleme dans ces deux fonctions, pouvez vous m'aider a me montrer ce que j'ai oublié de libérer et comment les libérer.

Code :

BITMAP take_screenshot()
{
HDC hCaptureDC = CreateCompatibleDC(hDesktopDC);
HBITMAP hCaptureBitmap =CreateCompatibleBitmap(hDesktopDC,
nScreenWidth, nScreenHeight);
SelectObject(hCaptureDC,hCaptureBitmap);
BitBlt(hCaptureDC,0,0,nScreenWidth,nScreenHeight,
hDesktopDC,0,0,SRCCOPY|CAPTUREBLT);
//Get Cursor Pos
POINT xPoint;
GetCursorPos( &xPoint );
//Draw the Cursor
bool g_recordcursor = 1;
if (g_recordcursor == 1) {
BOOL ret;
CURSORINFO cinfo;
cinfo.cbSize = sizeof(CURSORINFO);
ret = GetCursorInfo(&cinfo);
DrawIcon( hCaptureDC, cinfo.ptScreenPos.x, cinfo.ptScreenPos.y, cinfo.hCursor);
}
HBITMAP hNewBitmap = (HBITMAP)SelectObject(hCaptureDC, hCaptureBitmap);
BITMAP bm;
ZeroMemory(&bm,sizeof(BITMAP));
GetObject(hNewBitmap, sizeof(bm), &bm);
int bm_size = bm.bmWidthBytes * bm.bmHeight;
bm.bmBits = new char[bm_size];
long nfetched = GetBitmapBits(hCaptureBitmap, bm_size, bm.bmBits);
DeleteDC(hCaptureDC);
DeleteObject(hCaptureBitmap);
DeleteObject(hNewBitmap);
return bm;
}
static void fill_yuv_image(AVFrame *pict, int frame_index, int width, int height)
{
BITMAP bm = take_screenshot();
SwsContext *fooContext = sws_getContext(nScreenWidth,nScreenHeight,PIX_FMT_RGB32,
width,height,STREAM_PIX_FMT,
sws_flags,NULL,NULL,NULL);
uint8_t *movie_dib_bits = reinterpret_cast<uint8_t *>(bm.bmBits);
int dibrowbytes = bm.bmWidthBytes;
uint8_t* data_out[4];
int stride_out[4];
data_out[0] = movie_dib_bits;
stride_out[0] = dibrowbytes;
sws_scale(fooContext,data_out,stride_out,0,nScreenHeight,pict->data,pict->linesize);
// A partir d'ici j'ai besoin de plus rien sauf pict
free(movie_dib_bits);
DeleteObject(&bm);
}

Merci !

Message édité par kyom le 02-09-2009 à 16:51:53

Publicité

olivthill

Peut-être que ce serait dû à l'ordre des libérations des ressources. Essayer de les faire dans l'ordre inverse de leur création, en remplaçant les lignes 37 à 39 par :

DeleteObject(hNewBitmap);
DeleteObject(hCaptureBitmap);
DeleteDC(hCaptureDC);

kyom

Aucun changement...

olivthill

Autre chose.

BITMAP take_screenshot()
{
...
BITMAP bm;
...
return bm;
}

static void fill_yuv_image(AVFrame *pict, int frame_index, int width, int height)
{
BITMAP bm = take_screenshot();
...

La ligne BITMAP bm;réserve de l'espace sur la pile pour la structure bm, par exemple à l'adresse 1234.
La ligne return bm renvoie 1234, mais ne renvoie pas le contenu de la structure.
La ligne BITMAP bm = take_screenshot();associe 1234 au nouveau bm
Ensuite la pile est altérée par la suite du programme, ce qui est normale, et donc les octect à l'adresse 1234 ne veulent plus rien dire.

Pour le passage de paramètres, il faut procéder autrement. Il faut les mettre dans les paramètres de la fonction, et non pas dans le pointeur de retour d'une fonction, et il ne faut pas les allouer à l'intérieur d'une sous fonction, mais les allouer à l'extérieur, et donc écrire :

int take_screenshot(... BITMAP bm ...)
{
...
...bm...
...
return 0;
}

static void fill_yuv_image(AVFrame *pict, int frame_index, int width, int height)
{
int code_retour;
BITMAP bm;
code_retour = take_screenshot(...bm...);
...

Message édité par olivthill le 02-09-2009 à 16:18:15

tpierron

Hmm, il y a un truc qui me parait être la cause, c'est ton premier appel à SelectObject, ça devrait plutôt être :

Code :

HBITMAP old = SelectObject(hCaptureDC,hCaptureBitmap);

Et avant de libérer ton DC, il faut toujours remettre les valeurs d'origine avec GDI :

Code :

SelectObject(hCaptureDC, old);
DeleteDC(hCaptureDC);

Et ton hNewBitmap ne sert à rien, ton second appel à SelectObject va te renvoyer un pointeur sur hCaptureBitmap.

Edit: oh purée, je n'avais pas vu ton "DeleteObject(&bm);" à la fin. Incroyable que ça ne fasse pas planter ton programme. &bm n'est pas du tout un handle de bitmap, fait simplement un "delete[] bm.bmBits".

Edit2: [:prozac] purée et tu fais un reinterpret_cast sur bm.bmBits dans un uint8_t *, que tu passes à la fonction free(), apràs l'avoir alloué avec new char[x].

Message édité par tpierron le 02-09-2009 à 16:29:23

kyom

Merci bien, avec toute ces modifs ça fonctionne bien mieux.

FORUM HardWare.fr

Programmation

API Win32

[Résolu] Fuite mémoire, que libérer ?

Sujets relatifs
[RESOLU]Arrondi à partir d'une chaine de caractère	[résolu][Velocity & Hudson] Templates non copiés dans le classpath
SDL_GetRGB (résolu)	[Résolu] Génération flux RSS - Format de date
[Résolu] [BATCH] Tracer la commande delete	[Algo][Résolu]Générer tous les accords.
sendAndLoad pas de retour sous flash {résolu}	Problème d'authentification avec sha1[RESOLU]
[Résolu] Expressions régulières... je bloque. :(	[Résolu] CSS lien en block absolu sur IE
Plus de sujets relatifs à : [Résolu] Fuite mémoire, que libérer ?

Page générée en 0.059 secondes