pb avec mon free

Version imprimable

pb avec mon free

Bonjour, voici le code suivant :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
FILE * file=NULL;
    double d;
    char line[10000];
    for(i=0;i<Z;++i)
    {
      if(i==0) file=fopen("../Outputs/Particle1.txt","r");
      else if(i==1) file=fopen("../Outputs/Particle2.txt","r");
      else file=fopen("../Outputs/Particle3.txt","r");
      for(j=0;j<3;++j) fgets(line,sizeof(line),file);
      char *p=strdup(line);
      d=strtod(p,&p); d=strtod(p,&p);
      NV_Ith_S(y,(i+1)*mec->ns+i)=strtod(p,&p);
      for(j=0;j<mec->ns;++j) NV_Ith_S(y,i*(mec->ns+1)+j)=strtod(p,&p);
      fclose(file);
      free(p); p=NULL;
    }

ce code compile mais lors de l'exécution il plante à la ligne

Code:

1
2
 
free(p); p=NULL;

Savez-vous pourquoi ?
Merci.

27/04/2006, 09h53
DaZumba

free() doit recevoir exactement la valeur retournee par malloc() [ou strdup(), dans ton cas]. Et tu modifies cette valeur via l'appel a strtod()...
27/04/2006, 09h56
salseropom

OK je comprends.
Et comment libérer la mémoire dans mon cas ?
27/04/2006, 09h58
DaZumba

Tu dois modifier ton code pour la valeur de p ne soit pas modifiee entre strdup() et free(). Donc, utilise une autre variable en deuxieme argument strtod().
27/04/2006, 10h13
salseropom
Et plus simple : j'ai remplacé
Code:

1 2 char * p=strdup(line);
par
Code:

1 2 char * p=line;
ce qui fait que je n'ai plus besoin de free(p);
Est-ce correct ?
27/04/2006, 10h31
Emmanuel Delahaye

Citation:

Envoyé par salseropom

Et plus simple : j'ai remplacé

Code:

char * p=strdup(line);

par

Code:

char * p=line;

ce qui fait que je n'ai plus besoin de free(p);
Est-ce correct ?

Si tu n'as pas besoin de copie, oui.
27/04/2006, 10h51
salseropom

Non je n'ai pas besoin de copie. Merci.
27/04/2006, 10h52
salseropom

Où est la balise "RESOLU" ?
27/04/2006, 10h53
Skyounet

Citation:

Envoyé par salseropom

Où est la balise "RESOLU" ?

En haut dans Outils de la discussion.