Discussion :

[Dud225]

Code :

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

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?

[zooro]

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

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

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

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

[koala01]

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 :

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

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 :

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

[Thierry Chappuis]

Code :

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

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

[koala01]

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

[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

[Dud225]

Merci de vos réponses.

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