utilisation du free sur un char*

Version imprimable

30/01/2009, 15h29
kase74

utilisation du free sur un char*

Bonjour,

Dans une fonction, j'ai declare une variable char* :

Code:

char* Separateurs = " ,;\t"

A la fin de ma fonction, j'ai fait :

Code:

free(Separateurs )

Sans qu'il y est de plantage, j'avais des soucis lors du debuggage : le compilo faisait comme si il y avait plusieurs breakpoints invisibles dans l'instruction free.

J'ai enleve le free et plus aucun probleme.

Pouvez vous me confirmer la non utilite du free et m'en expliquer la raison ?
Merci d'avance pour votre aide.
30/01/2009, 16h27
ram-0000

Dans l'exemple que tu fournies, Separateur est un pointeur sur une chaine de caractères qui est en mémoire.

Tu n'as absolument pas besoin de la libérer, c'est même une erreur que de faire un free() dessus, le fait qu'il n'y ait pas eu plantage est un hasard.

Donc non, pas de free() dessus
30/01/2009, 16h29
Invité(e)

Bonjour,

Ton pointeur Separateurs pointe vers un tableau de caractère statique créé par le compilateur. Il ne faut pas le libérer.

D'un point de vu mémoire,

Code:

char toto[] = "titi";

est équivalent à

Code:

char * toto = "titi";

.

Le free ne s'appelle que pour les allocations faites à l'aide de malloc, calloc, realloc et strdup.

OK merci beaucoup pour vos lumières.
Encore une petite precision, j'ai dans le meme temps declare des variables qui elles sont initialisees avec une copie par exemple avec strdup ou strtok

Voici le code complet de ma fonction, est ce que vous y voyez une erreur ?

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
/** @brief The function search and scan all DATAS block into the NODES block.
*
* @param NodesElem is the element in the XML witch is the begin of the NODES block.
*
* First it retrieves a header which give destination variables and then it retrieves the datas stocked in table format.
**/
void CStudySytem::LoadXML_TabNodesValues( TiXmlElement *NodesElem)
{
    TiXmlElement *DatasElem,*HeaderElem,*ValuesElem;
    char *HeaderString=NULL, *HeaderWord=NULL, *ValuesString=NULL, *ValueWord=NULL, *Separators = " ,;\t";
 
    DatasElem = NodesElem->FirstChildElement(XML_DatasTable);
    while ( DatasElem)
    { // Here I'm in a DATAS block
        // I search the first Header element :
        HeaderElem=DatasElem->FirstChildElement(XML_DatasHeader);
        // and the first block Values element :
        ValuesElem=DatasElem->FirstChildElement(XML_DatasValues);
        if (!HeaderElem or !ValuesElem)
        {   // No header or no Values! I leave this DATAS block and continue
            DatasElem=DatasElem->NextSiblingElement(XML_DatasTable); // I loop on all possible datas block
            continue; // go to the next DATAS block
        }
        // I stock each header element seperatly in a list
        HeaderString = strdup(HeaderElem->GetText());
        list<string> Header;
        HeaderWord = strtok (HeaderString,Separators); // scan with space, coma, semicolon and tab caracter
        while (HeaderWord)
        {
            Header.push_back( HeaderWord);
            HeaderWord = strtok (NULL,Separators);
        }
 
        if( Header.empty())
        {   // No header : I leave this DATAS block and continue
            DatasElem=DatasElem->NextSiblingElement(XML_DatasTable); // I loop on all possible datas block
            continue; // go to the next DATAS block
        }
 
        ValuesString = strdup(ValuesElem->GetText());
        ValueWord = strtok (ValuesString,Separators); // scan with space, coma, semicolon and tab caracter
        list<string>::iterator iHeader = Header.begin(); // I place a pointer on the first header
 
        TIndex Index = 0;
        TNumeric X=0.0f,Y=0.0f,Z=0.0f;
        while ( ValueWord)
        {
 
//            cout<<*iHeader<<" = "<<ValueWord<<endl;
 
            if      ( strcmp((*iHeader).c_str(),XML_NodeIndex.c_str()) == 0 )
                Index = strtol(ValueWord,NULL,10);
            else if ( strcmp((*iHeader).c_str(),XML_NodeXCoord.c_str()) == 0 )
                X = strtof(ValueWord,NULL);
            else if ( strcmp((*iHeader).c_str(),XML_NodeYCoord.c_str()) == 0 )
                Y = strtof(ValueWord,NULL);
            else if ( strcmp((*iHeader).c_str(),XML_NodeZCoord.c_str()) == 0 )
                Z = strtof(ValueWord,NULL);
//            else // Error, header unknown !
 
            // treat the next value :
            ValueWord = strtok (NULL,Separators);
            ++iHeader;
            if ( iHeader == Header.end())
            {
                // I create the new node with his properties :
                TIteratorOnANode iNewNode = AddANode(Index);
                if (iNewNode!=NULL)
                    iNewNode->SetPositionInSpace(X,Y,Z);
                // Next value will be for the first header, with blank properties :
                Index = 0;
                X=0.0f;
                Y=0.0f;
                Z=0.0f;
                iHeader = Header.begin();
            }
        }
 
        DatasElem=DatasElem->NextSiblingElement(XML_DatasTable); // I loop on all possible datas block
    }
 
    free(HeaderString);
    free(HeaderWord);
    free(ValuesString);
    free(ValueWord);
//    free(Separators);
}

Encore merci.

30/01/2009, 17h17
Médinoc

C'est du C++, ça...
30/01/2009, 17h17
Jean-Marc.Bourguet

Citation:

Envoyé par mabu

D'un point de vu mémoire,

Code:

char toto[] = "titi";

est équivalent à

Code:

char * toto = "titi";

.

Non. La premiere forme alloue un tableau de 5 caracteres pouvant etre modifie. La deuxieme alloue un pointeur, pouvant etre modifie, qui est initialise a l'adresse d'un tableau de 5 caracteres ne pouvant pas etre modifie (meme si il ne faut pas que toto soit const pour que ce code passe en C).
30/01/2009, 17h18
Médinoc

Seules HeaderString et ValuesString doivent être libérées avec free(), car seules ces deux chaînes sont allouées avec malloc() ou strdup().
strtok() retourne un pointeur vers la chaîne elle-même, donc pas besoin de free() dessus.

En déclarant ainsi tes variables, tu peux mieux voir ce qui doit être détruit ou non:
Code:

1 2 3 4 char *HeaderString=NULL, *ValuesString=NULL; char const *HeaderWord=NULL; char const *ValueWord=NULL; char const * const Separators = " ,;\t";

30/01/2009, 17h24
diogene

Citation:

j'ai dans le meme temps declare des variables qui elles sont initialisees avec une copie par exemple avec strdup ou strtok

strdup crée une nouvelle chaîne par allocation dynamique et copie la chaîne en argument. Par conséquent, il y a lieu de faire un free sur le pointeur renvoyé par strdup pour récupérer la mémoire.

Par contre, strtok ne crée pas une chaîne ni ne l'initialise. Il modifie la chaîne passée en argument lors du premier appel. Par conséquent, il ne faut pas faire de free sur les pointeurs renvoyés par cette fonction (ValueWord et HeaderWord).
Dans ton cas, il me semble que lorsqu'on arrive à free(ValueWord ) et free(HeaderWord), ValueWord et HeaderWord sont à NULL, ce qui te sauve et fait que ça ne plante pas. Néammoins, ces deux free sont inutiles et illogiques.

Citation:

Envoyé par kase74

OK merci beaucoup pour vos lumières.
Encore une petite precision, j'ai dans le meme temps declare des variables qui elles sont initialisees avec une copie par exemple avec strdup ou strtok

strdup() (fonction POSIX.1, mais non standard C, je le rappelle) fait une copie dynamique avec malloc(), mais strtok() ne fait aucune copie. Elle se contente de découper la chaire originale et de retourner l'adresse du début de chaque sous chaine...

Il ne faut pas coder au hasard. Il faut lire la doc des fonctions pour savoir ce qu'on fait et ce qu'il faut faire...

Une bonne référence :

http://www.opengroup.org/onlinepubs/...ns/strdup.html
http://www.opengroup.org/onlinepubs/...ns/strtok.html
Citation:
Voici le code complet de ma fonction, est ce que vous y voyez une erreur ?
Code:

1 2 3 4 5 6 7 /** @brief The function search and scan all DATAS block into the NODES block. * * @param NodesElem is the element in the XML witch is the begin of the NODES block. * * First it retrieves a header which give destination variables and then it retrieves the datas stocked in table format. **/ void CStudySytem::LoadXML_TabNodesValues( TiXmlElement *NodesElem)
Pas du C.

30/01/2009, 19h02
kase74

Merci pour vos reponses.
Je prends bien note du site reference que je vais mettre en haut de ma liste.