Libération d'une allocation dynamique faite dans une fonction

Version imprimable

14/12/2006, 15h47
Dud225

Libération d'une allocation dynamique faite dans une fonction
Code:

1 2 3 4 5 6 7 8 9 10 int *fonction(....) { int *pt; pt=(int*)malloc(5*sizeof(int)); ... return pt; }
Comment faire pour libérer pt? Si je fais le free avant le return, pt n'existe plus.
Si je le fais après le return, cela marche-t-il? Si ce n'est pas le cas, commet peut-on procéder?
14/12/2006, 15h58
zooro

Citation:

Envoyé par Dud225

Comment faire pour libérer pt? Si je fais le free avant le return, pt n'existe plus.
Si je le fais après le return, cela marche-t-il? Si ce n'est pas le cas, commet peut-on procéder?

Bonjour,
C'est simple : une fois que tu n'auras plus besoin de pt, tu peux le libérer par un free(), puis le mettre à NULL. Aucun problème pour le faire en dehors de la fonction. C'est un peu fait pour ça ;)
14/12/2006, 16h20
Thierry Chappuis
Salut,

Hors de ta fonction, tu récupères l'adresse du bloc alloué en stockant la valeur retournée par la fonction dans une variable de type pointeur-sur-int. Pour libérer l'espace alloué, il te suffit de passer ce pointeur en argument de free:
Code:

1 2 3 4 5 6 7 8 9 10 11 int *fonction(void); int main(void) { int *p_valeur; /*...*/ p_valeur = fonction(); /*...*/ free(p_valeur), p_valeur = NULL; return 0; }
Thierry

Salut,

Il faut juste penser à vérifier partout que l'allocation de la mémoire pour le pointeur se soit correctement déroulée...

Non seulement dans la fonction qui s'occupe de l'allocation

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
int* Mafonction(...)
{
    int *newptr;
    newptr=malloc(sizeof(int) *1520);
    if(newptr==NULL)
    {
         /*ca coince ici */
    }
    ...
    return newptr;
}

Mais aussi, chaque fois que tu compte sur cette fonction pour initialiser un pointeur

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
 
int main()
{
    int* Myptr=NULL; /* on le déclare, mais on l'utilisera plus tard...
                        ==>on l'initialise à NULL pour éviter les problèmes ;) */
...
    Myptr=Mafonction(...);
    /* Au fait, l'allocation dynamique a pu foirer... testons vite ce qu'il
       en est ... */
    if(Myptr==NULL)
    {
        /*Ca coince ici */
    }
    else
    {
        /* ici, ca passe */
    }
    /* et ici, ca n'a pas d'importance (parce qu'on ne s'en sert pas */
...
    /* on n'oubile pas de le libérer quand on en n'a plus besoin */
    /* mais uniquement s'il n'est pas nul car un free sur NULL posera
        problèmes */
    if(Myptr!=NULL)
        free(Myptr);
    /* ni de le remettre à NULL pour éviter les problèmes */
    Myptr=NULL;
...
}

14/12/2006, 16h54
Thierry Chappuis
Citation:
Envoyé par koala01

Code:

1 2 3 4 if(Myptr!=NULL) free(Myptr); }
Pourquoi? free() ne fait rien si on lui passe un pointeur NULL. Cette garde n'est donc pas nécessaire: http://man.developpez.com/man3/free.3.php

Thierry
14/12/2006, 17h28
koala01

:oops:

Il me semblait que, comme delete en C++, free avait un comportement indéfini si on essayer de lui faire libérer quelque chose de nul...

Autant pour moi...
14/12/2006, 17h32
koala01

D'apres la page que tu donnes, free peut, effectivement avoir un comportement indéterminé, mais pas sur un pointeur NULL... (si l'on essaye de libérer la mémoire allouée à un pointeur dont la mémoire a déjà été libérée, entre autres)

Donc, ce qui est, surtout important, c'est de bien faire pointer Newptr sur NULL apres la libération :P
14/12/2006, 20h24
Dud225

Merci de vos réponses.
14/12/2006, 23h37
Médinoc

D'ailleurs, le comportement de delete NULL est fixé aussi.

Par contre, fclose(NULL) a un comportement indéterminé. Avec la CRT 8 de Microsoft, ça plante exprès (invalid parameter handler).