Discussion :

[Amroth]

Bonjour,
Dans un programme, la fonctionne realloc plante si je demande de réallouer a partir d'une certaine valeur, il doit y avoir probablement un free a faire mais je ne vois pas ou.

voici ma fonction pour réallouer une chaine de taille cpt_chaine à une nouvelle taille de la valeur indice :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
int reallocation_chaine(char **chaine, int *cpt_chaine, int indice){
  char *temp;
  *cpt_chaine=indice;
  temp=realloc(*chaine,*cpt_chaine);
  if(temp!=NULL)
    *chaine=temp;
  else
    return-1;
  return 0;
}

cette fonction est utilisé dans un fonction qui parcourt un fichier afin de créer une chaine :

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 creer_chaine(char **chaine,int *cpt_chaine,FILE *fichier, char *caractere, char fin){
 
  char *temp=*chaine;
  FILE *file=fichier;
  int cpt_c=*cpt_chaine;
 
  int cpt=0;
  while(*caractere!=fin && *caractere!=EOF){
    printf(" %d tour\n",cpt);
    if(cpt+2>cpt_c){
      if(reallocation_chaine(&temp,&cpt_c,cpt_c*2)==-1){
	return -1;	
	}
    }
    temp[cpt]=*caractere;
    *caractere=fgetc(file);
    cpt++;
  }
  *cpt_chaine=cpt+1;
  if(reallocation_chaine(&temp,&cpt,cpt+1)==-1)
    return -1;
  temp[cpt-1]='\0'; 
  *chaine=temp;
  *caractere=fgetc(file);
  return 0;
}

Merci pour votre aide!

[diogene]

Code :

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

1
2
3
4
5
int reallocation_chaine(char **chaine, int *cpt_chaine, int indice){
  char *temp=*chaine;
....
  if(realloc(temp,(*cpt_chaine)*sizeof(char))!=NULL)
    *chaine=temp;

1- A l'entrée de la fonction, *chaine doit être NULL ou une valeur obtenue par --alloc(). La partie de code montrée ne permet pas de répondre.

2- Par contre, la méthode de réallocation est fausse : La réallocation par realloc peut modifier la position mémoire du bloc alloué. Dans ce cas, realloc renvoie la nouvelle adresse (!=NULL) et la zone mémoire antérieurement utilisée est automatiquement libérée. Dans ce cas, l'allocation de temp à *chaine stockera une valeur devenue illégale. La fonction plantera à l'appel suivant.

Il faut suivre le schéma suivant :

Code :

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

1
2
3
4
temp =  realloc(*chaine,*cpt_chaine);
if(temp!=NULL) *chaine = temp;
else return -1; // *chaine n'est pas modifié
return 0;

[Amroth]

Merci pour la réponse,
J'ai modifié ma fonction reallocation_chaine mais le probleme est toujours le meme. Et à l'entrée de ma fonction, c'est bien une chaine alloué grace a malloc.

[edit]En mettant des commentaires pour tester ou a lieu l'erreur, un avant le realloc et un apres, j'en deduis que c'est lors du realloc que j'ai un souci, a savoir que la fonction est appelée plein de fois en boucle[/edit]

[diogene]

En mettant des commentaires pour tester ou a lieu l'erreur, un avant le realloc et un apres, j'en deduis que c'est lors du realloc que j'ai un souci, a savoir que la fonction est appelée plein de fois en boucle

Comme il n'y a pas de boucles dans reallocation_chaine, j'en déduis qu'il faut voir le problème dans les fonctions appelant reallocation_chaine. creer_chaine ? Mais on ne connait pas les paramètres d'appel à cette fonction !

[Amroth]

Dans ma fonction main, j'ai déclaré ceci :

Code :

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

1
2
3
4
char *parametre; 
int parametre_cpt; // entier qui retient la taille allouée
FILE *fichier; //un fichier que j'ouvre et dont je vais parcourir chaque caractere
char caractere; // le caractere courant

que j'ai alloué comme ceci :

Code :

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

1
2
3
4
5
 
parametre=malloc(10*sizeof(char));
if(parametre==NULL)
    return -1;
parametre_cpt=10;

et j'appelle ma fonction créer_chaine comme ceci :

Code :

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

1
2
3
 
if(creer_chaine(&parametre,&parametre_cpt,fichier,&caractere,']')==-1)
return -1;

[diogene]

Code :

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

1
2
3
4
//parametre est donc alloué pour 10 char dans main. zone initialisée ?-|
//Dans creer_chaine                                                    |
cpt_strlen=strlen(temp);  //                                           | 
//qui a mis une chaîne de caractères dans parametre ?           <-------