Discussion :

[Marseyeah]

Voila j'ai un problème en utilisant la fonction fgets :

J'ai une fonction définie dans un .h :

Code :

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

char * lireFichierSource(FILE * pt_fichier);

qui réalise ce traitement :

Code :

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

1
2
3
4
 
     char * buffer; // enregistrement lu
     fgets(buffer,300,pt_fichier);
     return (buffer) ;

Lorsque j'apelle cette fonction dans le main de cette manière le programme plante au niveau de l'apelle à la fonction fgets() :

Code :

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

1
2
3
4
 
     char * buffer ;
     FILE * fichier = ouvrirFichierSource("c:\\test.txt"); //fonction définie par ailleurs et qui fonctionne
     buffer = lireFichierSource(fichier) ;

Par contre si j'écrit le bout de code de la fonction lireFichierSource directement dans le main cela fonctionne sans problème. Quelqu'un aurait une idée ?

[gangsoleil]

Bonjour,

Nulle part je ne vois traces d'allocations mémoires. Est-ce parce que tu n'en fais pas (auquel cas il est normal que ton programme plante), ou bien parce que tu ne les as pas recopiées ici ?

[jowo]

qui réalise ce traitement :

Code :

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

1
2
3
4
 
     char * buffer; // enregistrement lu
     fgets(buffer,300,pt_fichier);
     return (buffer) ;

Peut-on avoir la source du code?

Comment alloues-tu de la mémoire pour ton buffer?

[Emmanuel Delahaye]

J'ai une fonction définie dans un .h :

Code :

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

char * lireFichierSource(FILE * pt_fichier);

C'est le prototype de la fonction qui est défini. Ca a pour conséquence la declaration de la fonction. La définition de la fonction est faite ailleurs (enfin, j'espère).

qui réalise ce traitement :

Code :

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

1
2
3
4
 
     char * buffer; // enregistrement lu
     fgets(buffer,300,pt_fichier);
     return (buffer) ;

Lorsque j'apelle cette fonction dans le main de cette manière le programme plante au niveau de l'apelle à la fonction fgets() :

Ben oui, comportement indéfini...

Code :

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

1
2
3
4
 
     char * buffer ;
     FILE * fichier = ouvrirFichierSource("c:\\test.txt"); //fonction définie par ailleurs et qui fonctionne
     buffer = lireFichierSource(fichier) ;

Dans la fonction lireFichierSource(), le pointeur buffer n'est pas initialisé. La valeur passée à fgets() est donc indéterminée et le comportement est indéfini. Il faut un espace mémoire d'au moins 300 bytes, et qui soit valide après l'appel de la fonction.

Avant tout, il faut vérifier que la fonction ouvrirFichierSource() retourne bien NULL en cas d'erreur. Rien ne dit que "c:\\test.txt" existe et soit accessible sur la machine de test...

[Marseyeah]

ok c bon merci pour vos réponses, j'avais complétement oublié d'allouer de la mémoire pour le buffer dans la fonction lireFichierSource . Ca marche maintenant. C'est que je suis pas vraiment un spécialiste du C .