Discussion :

[Invité]

Bonjour à tous,
Dans mon programme flex-bison, je fais un malloc sur une variable globale à chaque fois qu'une expression est reconnu, c'est à dire en quelques sortes j'ai ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
variable globale :
char* s;
 
fonction fct :
if(Condition){
  s = malloc(20*sizeof(char));
  /* traitement */
}
 
main :
fct();
free(s);

Evidement je ne peux passé s en paramètre a la fonction fct, car celle ci est fictive pour illustrer l'exemple, je rappelle que c'est un programme flex-bison.
J'aurais voulu savoir si le seul free de s malgré plusieurs malloc sur s était suffisant ou bien s'il y avait des fuites de mémoire.

Merci de votre aide.

[Neckara]

Bonjour,

Il y aura en effet une fuite de mémoire.
Il faut faire un free par malloc.

[transgohan]

Le mieux serait d'initialiser le pointeur.
Puis avant le malloc vérifier que le pointeur ne cible pas une zone mémoire.
Si tel est le cas on utilise free avant de faire un nouveau malloc.

[Invité]

Tout d'abord merci de vos réponses.
Le problème c'est que si je fais un free avant de refaire un autre malloc, mon programme ne marche plus.
J'aimerais faire un free global en fin de programme mais ça ne marche pas.
Par exemple la taille de s = sizeof(s) / sizeof(char);

Code :

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

1
2
3
for(i=0;i<sizeof(s) / sizeof(char);i++){
  free(s+i);
}

Là je désalloue de la mémoire déja désalloué.

Le problème vient du fait que s n'est pas une matrice. Donc comment faire plusieurs free ?

[transgohan]

Malheureux on ne désaloue pas un char. Tu tentes de faire un truc du genre :

Code c :

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

1
2
char s = 'a';
free(s);

Ce qui est différent de :

Code c :

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

1
2
3
4
5
char *s;
s = (char*) malloc (2 * sizeof(char) );
s[0] = 'a';
s[1] = '\0';
free(s);

Le problème c'est que si je fais un free avant de refaire un autre malloc, mon programme ne marche plus.

As-tu vérifié que tu ne tentais pas de désalouer du vide ?
Ceci fonctionne à merveille :

Code c :

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

1
2
3
4
5
char *s;
s = (char*) malloc (2 * sizeof(char) );
free(s);
s = (char*) malloc (5 * sizeof(char) );
free(s);

[plxpy]

Le problème c'est que si je fais un free avant de refaire un autre malloc, mon programme ne marche plus.

Là, ça cache un autre problème qu'on ne peut pas détecter avec le code succint fourni mais il y a, assurément, une grosse cagade quelque part ailleurs !

Le mieux serait d'initialiser le pointeur.
Puis avant le malloc vérifier que le pointeur ne cible pas une zone mémoire.
Si tel est le cas on utilise free avant de faire un nouveau malloc.

Oui mais ... non. Si, techniquement, ça tient parfaitement la route et c'est tout à fait juste, en procédant de la sorte, le code n'a plus/pas beaucoup de sens (quoique ... ça n'arrête pas toujours tout le monde !) : les allocations sont toujours de la même taille, on manipule toujours la même variable pointeur (s) et donc ça reviendrait à :

• initialiser s à NULL au tout début
• désallouer s (s'il n'est pas NULL)
• "perdre" son contenu
• ré-allouer s, à la même taille
• changer son contenu

Pourquoi faire simple quand ...

Je pense qu'il nous manque des éléments pour qu'on puisse t'aider efficacement.

[Invité]

Ceci fonctionne à merveille :

Code c :

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

1
2
3
4
5
char *s;
s = (char*) malloc (2 * sizeof(char) );
free(s);
s = (char*) malloc (5 * sizeof(char) );
free(s);

Oui mais moi j'essais de faire ceci :

Code :

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

1
2
3
4
char *s;
s = (char*)malloc(20*sizeof(char));
s = (char*)malloc(20*sizeof(char));
free(s);

Car si je fais free entre les deux mallocs, la valeur d'un attribut de ma structure change pour la valeur du deuxième s et etc... Bref je sais pas si je suis clair, mais en tout les cas c'est vrai que ça sens la grosse cagade !