Fuite mémoire en sortie de fonction

Bonjour à tous,

Je travaille sur une application embarqué et il semble que j'ai une fuite mémoire en sortie d'une fonction:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

typedef struct winstruct *win;     void function( int key ) { win w=global.w,parent; while ( w ) { parent = w->info.wParent;   /*code   *   *   */fincode   w = parent; } }

---

J'ai du mal à comprendre pourquoi j'ai une fuite mémoire en sorti de cette fonction.
Si vous avez des idées je suis preneur.

Merci

Si cette fonction ne crée rien, elle ne peut pas causer de fuite mémoire.

Il faudrait voir ce qui est fait dans la partie "code...fincode"...

Il y a juste l'appel à un pointeur de fonction:

Code:

---

1 2 3 4 5 6

if ( w->pFunc && (*w->pFunc)( w->Id, w, key) ) { break; }

---

Alors, c'est peut-être dans la fonction appelée via ce pointeur

Salut, il y a fuite mémoire si après un *alloc il n'y a pas de free. Dans tes fonctions que tu nous présentes, il n'y a pas de *alloc, donc il n'y a pas de fuite mémoire...

n'oublie pas qu'après strdup() il faut faire un free() car il y a un malloc() de caché dans le strdup()

valgrind est ton ami

Citation:

---

Envoyé par lappeldupeuple

Code:

---

1 2	win w=global.w,parent;

---

---

C'est pas une construction un peu ... suspecte (pas pour la fuite de mémoire, mais en général) ?

Suspect oui mais pas interdit!

Si je lis ça dans un code que je dois maintenir, plein de questions vont m'envahir (pourquoi faire ? effets secondaires ? ...) pour peut-être au final pas grand chose :calim2:...
Donc c'est du code que je ré-écrirais certainement pour le rendre plus lisible pour moi et les autres mainteneurs.
Mais ce n'est que mon humble avis :mrgreen:

Code:

---

win w=global.w,parent;

---

Il n'y a pas de problèmes sur cette ligne (il ne s'agit pas de l'opérateur ,) : définition de deux variables de type win (w et parent) dont la première (w) est initialisée avec la valeur de global.w.
Enfin, à part que global est une variable globale, ce qui est discutable, et qu'il aurait été plus lisible, donc préférable, de mettre ce code sur deux lignes

Code:

---

1 2	win w=global.w; win parent;

---