Discussion :

[gindrou]

Bonjour,

Je me permet de vous soumettre une petite question d'un débutant que je suis.
DEV C++ avec la SDL génère le code qui suit.
Ce qui me gène, c'est le free (msg).
Comment peut on libérer une mémoire qu'on a pas allouée ?

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
int main (int argc, char *argv[])
{
    char *msg;
 
    /* Initialize SDL */
    if (SDL_Init (SDL_INIT_VIDEO) < 0)
    {
        sprintf (msg, "Couldn't initialize SDL: %s\n", SDL_GetError ());
        MessageBox (0, msg, "Error", MB_ICONHAND); 
        free (msg);
        exit (1);
    }
 
    return 0;
}

[Spout]

Et c'est Dev-C++ qui te génère ça ?

Si ce code compile (ce qui serait déjà une erreur), il est clair qu'il te claquera dans les doigts à l'exécution!
Effectivement, tu ne peux pas utiliser msg car ce pointeur est indéfini, il ne pointe vers rien. sprintf() va donc provoquer une erreur lorsqu'il voudra copier la chaîne en paramètre dans l'espace pointé par msg... qui n'existe pas!

La question du free() ne se pose alors même pas vu que tu n'iras pas jusque là . Mais de toute façon, effectuer un free() sur un pointeur indéfini aura +/- le même effet .

[CSIE_Angel#5]

Il faut soit transformer le char *msg en char msg[500] (Allocation statique) et dans ce cas supprimer l'appel a la fonction free().

Ou sinon faire un jolie petit

Code :

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

msg =(char*)malloc(Taille*sizeof(char))

De plus pour un soucis de debugage j'ai tendance a écrire :

Code :

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

char *msg = NULL

ainsi lors du debugage on voit facilement tous les pointeurs qui n'ont pas été alloué plutot que de recherché des 0xcdcdcdcd

[gindrou]

He bien merci pour vos réponses.
Me voilà rassuré.
Pour être précis, il s'agit de wxDev-C++.
Je prend note de NULL.

[Spout]

?