Discussion :

[lappeldupeuple]

Bonjour à tous,

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

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
 
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

[Médinoc]

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"...

[lappeldupeuple]

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

Code :

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

1
2
3
4
5
6
 
if ( w->pFunc
             && (*w->pFunc)( w->Id, w, key) )
        {
            break;
        }

[diogene]

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

[salseropom]

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()

[trax44]

valgrind est ton ami

[Grulim]

Code :

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

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) ?

[Gui13]

Suspect oui mais pas interdit!

[Grulim]

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 ...
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

[diogene]

Code :

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

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 :

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

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