Discussion :

[gloubi]

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]

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]

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]

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]

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]

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.

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.

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