Discussion :

[salseropom]

Bonjour, voici le code suivant :

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

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

1
2
 
free(p); p=NULL;

Savez-vous pourquoi ?
Merci.

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

[salseropom]

OK je comprends.
Et comment libérer la mémoire dans mon cas ?

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

[salseropom]

Et plus simple : j'ai remplacé

Code :

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

1
2
 
char * p=strdup(line);

par

Code :

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

1
2
 
 char * p=line;

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

[Emmanuel Delahaye]

Et plus simple : j'ai remplacé

Code :

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

char * p=strdup(line);

par

Code :

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

 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.