Fuite mémoire et désapointement

**gloubi** · 02/08/2006, 15h26

Bonjour.

J'ai un problème de fuite mémoire dans un code sous VC++6.
J'utilise un controle CWebBrowser, et a chaque fois que la fenetre est rafraichie (apres le rafraichissement pour etre precis) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
 
             lpDispatch = m_browser.GetDocument(); // m_browser est mon CWebBrowser2
	IHTMLDocument2 *doc2 = NULL;
	lpDispatch->QueryInterface(IID_IHTMLDocument2,(void**)&doc2);
	if ( doc2 == 0 )
		return;
	IHTMLElement *lpBodyElement; 
	IHTMLBodyElement *lpBody; 
	IHTMLTxtRange *lpText; 		
	doc2->get_body(&lpBodyElement); 
	if ( lpBodyElement )
	{
		lpBodyElement->QueryInterface(IID_IHTMLBodyElement,(void**)&lpBody); 
		if ( lpBody )
		{
			lpBody->createTextRange(&lpText); 
			BSTR bstrText; 
			lpText->get_htmlText(&bstrText); 
 
                                      // mon traitement

Et je fini mon traitement avec :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
 
			lpBody->Release();			
			//delete(lpBody); 	
		}
		//delete(lpBodyElement);
		lpBodyElement->Release();
	}
	doc2->Release();

Ce qui devrait désaloué tout ce petit monde.
Pourtant, a chaque refresh de ma page web dans mon CWebBrowser, mon processus se prend entre 300 et 500k dans la figure... au bout de quelque heures d'utiilisation, mon processus pèse près de 70Mo alors qu'il en pesait 16 au lancement.
Compte tenu de la taille qu'il prend a chaque refresh, je suis persuadé que c'est le contenu HTML de la page web qui est chargée en mémoire qui n'est pas désaloué.

J'ai donc essayé des free() et des delete(), mais le résultat reste le même.
Etant expert en PHP, mais pas du tout en C, j'aurais vraiment besoin qu'on m'explique ce qui ne vas pas...

J'ai également essayé de faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
 // CString i2a(int); // converti un int en Cstring
	MessageBox(i2a((int)doc2));
	doc2->Release();
	free(doc2);
	MessageBox(i2a((int)doc2));

Pourtant, les 2 MessageBox m'affichent le même résultat... docs ne devrait-il pas etre égale à NULL apres le free ? même chose avec delete :/

Quelqu'un comprend t-il pourquoi ça fait ça.

Merci d'avance.

edit : il n'y a bien entendu pas de return dans le code

edit2 : j'ai coupé mon code trop tot pour voir le delete sur lpText, mais je l'avais testé aussi

**r0d** · 02/08/2006, 17h25

Bonjour,

pourquoi n'utilises-tu pas les delete (pourquoi les as-tu mis en commentaire)? Je ne trouve pas de doc sur cette méthode Release

**gloubi** · 02/08/2006, 17h33

A vrai dire, j'ai pompé cette partie du code sur la msdn et des forums.
Pour le delete, je les ai mis en commantaire car c'est moi qui les avait ajouté, et que cela ne changeait strictement rien.

Ce sont des delete, mais j'ai egalement essayé de mettre des free() sans plus de résultat...

La combinaison Release + delete ne donne rien non plus...

Cependant, je me pose toujours la question de savoir si c'est normal que mon pointeur indique toujours la meme adresse apres un delete. Il ne devrait pas passer a NULL ?

**r0d** · 02/08/2006, 17h41

Envoyé par gloubi

A vrai dire, j'ai pompé cette partie du code sur la msdn et des forums.
Pour le delete, je les ai mis en commantaire car c'est moi qui les avait ajouté, et que cela ne changeait strictement rien.

Ce sont des delete, mais j'ai egalement essayé de mettre des free() sans plus de résultat...

La combinaison Release + delete ne donne rien non plus...

Cependant, je me pose toujours la question de savoir si c'est normal que mon pointeur indique toujours la meme adresse apres un delete. Il ne devrait pas passer a NULL ?

Non.
Losque tu fais un delete (ou un Release, puisqu'apparemment c'est la même chose) le système désalloue la mémoire mais il ne réinitialise pas le pointeur. Si tu tiens absolument à le passer à NULL (ce qui peut servir pour des tests ultérieurs par exemple) il faut le faire "à la main":

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

lpBody = NULL;

**gloubi** · 02/08/2006, 17h50

Et est-ce que le delete est supposé libérer de façon récursive tous les pointeurs contenus dans la class/structure passée en paramètre ?

Cela est-il prévu dans le destroy des class que j'utilise ?

J'ai vraiment l'impression d'être devant un problème insoluble... j'ai beau mettre des delete partout, mon prog continu a gonfler quand je le fait pas planter...

J'ai tester en dehors de ce module, je ne fait aucun new() ni malloc() (a part une fonction de file, qui elle se désaloue bien)...

des idées ?

**r0d** · 02/08/2006, 18h08

Envoyé par gloubi

Et est-ce que le delete est supposé libérer de façon récursive tous les pointeurs contenus dans la class/structure passée en paramètre ?

Si la classe concernée est bien programmée, oui. Si c'est une classe que tu implémentes toi-même, c'est à toi de gérer la libération de mémoire des objets qu'elle contient.

Envoyé par gloubi

Cela est-il prévu dans le destroy des class que j'utilise ?

C'est une classe de la MFC, donc sans hésitation: oui.

Envoyé par gloubi

J'ai vraiment l'impression d'être devant un problème insoluble... j'ai beau mettre des delete partout, mon prog continu a gonfler quand je le fait pas planter...

J'ai tester en dehors de ce module, je ne fait aucun new() ni malloc() (a part une fonction de file, qui elle se désaloue bien)...

des idées ?

Comme ça, c'est difficile. Les fuites mémoires sont souvent difficiles à localiser. Beaucoup de sujets en parlent sur le site, avec une petite recherche, tu devrais trouver beaucoup d'indications. Par exemple: http://www.developpez.net/forums/sho...uite+m%E9moire

**gloubi** · 02/08/2006, 18h24

Merci pour la piste, je vais potasser ça et je vous tien au courrant

**Médinoc** · 02/08/2006, 20h30

Attention, pour une interface COM/OLE, il ne faut faire ni delete ni free().
Juste l'appel à Release(), puis on met le pointeur à NULL.

Normalement, il faut à un Release "par interface".
C'est-à-dire que chaque appel à CoCreateInstance(), QueryInterface() ou AddRef() doit avoir un Release() qui lui correspond (et à mon avis, le GetDocument() contient un AddRef(), donc il faudra faire Release() à la fin).

Note: Évitez de tester les pointeurs en retour de QueryInterface() : C'est la valeur de retour qu'il faut tester, avec la macro SUCCEEDED() ou FAILED().

**Brouzouf** · 03/08/2006, 14h47

Salut,
Effectivement, apparemment il manque un release sur lpDispatch et j'ajouterai aussi que les BSTR doivent être désaloués une fois que l'on ne les utilise plus ou avant de les réutiliser avec un jolie SysfreeString sinon grosses pertes de blocs !

**gloubi** · 05/08/2006, 19h41

meric beaucoup ^^
je vais jetter un oeil a tout ça :p

Fuite mémoire et désapointement

Visual C++

Discussions similaires

Partager

Partager