[MFC] Thread & memory leaks

**Racailloux** · 15/03/2005, 09h39

Salut,

A la fermeture de mon appli, j'attends la fin d'un processus, mais il y a tt de meme un memory leak :

Code :

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

1
2
3
4
5
6
7
8
The thread 0x30C has exited with code 0 (0x0).
Detected memory leaks!
Dumping objects ->
strcore.cpp(118) : {2397} normal block at 0x002F66B0, 80 bytes long.
 Data: <        C   100,> 01 00 00 00 07 00 00 00 43 00 00 00 31 30 30 2C 
thrdcore.cpp(166) : {2306} client block at 0x00991E78, subtype 0, 112 bytes long.
a CWinThread object at $00991E78, 112 bytes long
Object dump complete.

Si je mets un AfxMessageBox apres avoir demandé la fin du thread, ces memory leaks n'apparaissent plus... magie??
meme si je n'attends plus la fin du thread apres...

Pourriez vous m'aider? merci
Racailloux

mat.M · 15/03/2005, 10h09

Les "fuites de mémoires" apparaissent lorsqu'on a allloué des instances de classes et qu'on les a mal effacées notamment des collections comme CList , CArray , CObArray , CObList etc....
Ou bien avec new sans delete !
Donc revérifier le code pour voir si tous les objets sont bien désinstanciés.
Un thread c'est un peu un autre programme qui s'exécute au sein d'un programme père .
Lorsqu'il se termine il faut effacer les objets alloués

**Racailloux** · 15/03/2005, 10h14

A vrai dire, je me connecte a du materiel.

Lorsque je me deconnecte et qu'apres je quitte l'appli, tt va bien.

Mais des que je quitte l'appli, sans me deconnecter, y a memory leak, sauf si il y a un AfxMessageBox lors de la deconnexion.

Donc je pense que mon thread n'arrive pas a se terminer si l'utilisateur a la main sur l'appli... mais je vois pas comment faire pour eviter de devoir mettre un messagebox

mat.M · 15/03/2005, 10h20

Mais des que je quitte l'appli, sans me deconnecter, y a memory leak, sauf si il y a un AfxMessageBox lors de la deconnexion.

Tout à fait Thierry le thread n'est pas totalement terminé lorsque le processus parent lui se termine.
En fait je me suis un peu trompé dans mon affirmation précédente les fuites de mémoires ne sont pas seulement entrainées par des non désallocations d'instances de classes mais aussi par des threads démarrés qui ne sont pas correctement terminés.
Parce que lorsqu'on démarre un thread il ya allocation mémoire spécifique , méchanisme de pile etc...

**Racailloux** · 15/03/2005, 10h24

super thierry... il me semblait que c'etait un truc dans le genre :p

Code :

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

m_pThRefresh = AfxBeginThread(ThreadRefresh,this,0,0,0,NULL);

Voila comment j'initialise mon thread, et pour se terminer, jusque la j'utilisais un bool qui le faisait se terminer, ce qui est le cas.
Donc comment liberer la memoire du thread lui meme? (j'ai essayer de remplacer l'AfxMessageBox par un vulgaire Sleep, mais la messagebox doit etre magique lol)

**Aurelien.Regat-Barrel** · 15/03/2005, 10h48

La méthode propre d'arrêter un thread c'est de le lui demander gentilment. Encore faut-il lui laisser le temps de s'arrêter, car si tu lui demandes à la fin de ton WinMain, dès le WinMain terminé le process est tué, et donc le 2° thread avec.
Ajoute une synchro avant de quiter, après avoir demander au thread de s'arrêter, dans le genre:

Code :

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

1
2
3
4
5
6
7
8
9
 
// demander au thread de s'arrêter
myThread->Stop();
// lui laisser 5 sec pour le faire
if ( WaitForSingleObject( myThread, 5000 ) == WAIT_TIMEOUT )
{
    // il ne s'est pas terminé, être un peu plus autoritaire
    TerminateThread( myThread ); // ou AfxEndThread peut etre
}