Sujet Précédent Sujet Suivant

Pb fichier, langage C

Résolu/Fermé
bizu53 Messages postés 1274 Date d'inscription samedi 30 août 2008 Statut Membre Dernière intervention 21 juin 2015 - 12 avril 2009 à 19:20
lucieb31 Messages postés 345 Date d'inscription mercredi 14 janvier 2009 Statut Membre Dernière intervention 28 juillet 2012 - 12 avril 2009 à 22:46
Bonjour,

Mon programme plante et je n'arrive pas à en trouver la raison.

Code de la fonction : 
void hexaToAscii(char *filenameIn, char *filenameOut)
{
	FILE *in = fopen(filenameIn, "rb");
	FILE *out = fopen(filenameOut, "w");
	char c, read[2], str[3];
	int nb;

	if(!in || !out)
	{
		printf("\nerreur");
		return;
	}
	
	nb = fread(read, 2, 1, in);
	while(nb != 0)
	{
		str[0] = read[0];
		str[1] = read[1];
		str[2] = '\0';
printf("\n%s", str);
		sscanf(str, "%x", &c);
printf("\nsscanf() -> %c", c);
		fprintf(out, "%c", c);
printf("\nfprintf()");
		nb = fread(read, 2, 1, in);
printf("\nfread()");
	}

	fclose(out);
	fclose(in);
}


Un exemple de sortie : 
41
sscanf() -> A

(sans plus, il plante au fprintf())

J'ai essayé de changer de mode w en wb (avec la fonction fwrite), même plantage au même endroit.

Quelqu'un voit le problème dans mon code ?

Merci d'avance
A voir également:

4 réponses

Réponse 1 / 4
lucieb31 Messages postés 345 Date d'inscription mercredi 14 janvier 2009 Statut Membre Dernière intervention 28 juillet 2012 62
12 avril 2009 à 20:35
tu as essayé de changer le mode w en rw ?
sinon essaie avec fputc(c, out);
0
fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 844
12 avril 2009 à 21:24
Tu as essayé de changer le mode w en rw ?
Le mode rw n'existe pas ;-))). Soit le mode r+, soit w+.
0
Réponse 2 / 4
fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 844
12 avril 2009 à 21:32
Salut,
Pas étonnant, tu mets : sscanf(str, "%x", &c); alors que c est un char.
Les octets supplémentaires vont déborder et récrire dans la pile. Ainsi l'adresse du pointeur in ne sera plus la bonne. Et lors de la prochaine lecture par fread, cela fera un joli segfault.
A la place de char c, utilise unsigned int c (le format %x attend un type unsigned int).
Cela devrait mieux fonctionner.
De plus, while(nb != 0) est un peu dangereux. Il vaudrait mieux utiliser while(nb==1) puisque fread renvoie le nombre d'éléments lus. S'il y a une erreur ou la fin du fichier est atteinte, la valeur de retour sera plus petit que 0,sans forcément être nulle.
Cdlt
0
bizu53 Messages postés 1274 Date d'inscription samedi 30 août 2008 Statut Membre Dernière intervention 21 juin 2015 860
12 avril 2009 à 22:28
"tu mets : sscanf(str, "%x", &c); alors que c est un char"
En effet mais comme je ne lui donne que 2 caractères en hexa je pensais que ça irait. En fait je n'avais pas pensé au débordement.
Je me suis en fait rendu compte qu'un sscanf(str, "%02x", &c); passe bien. J'aurais dû être plus rigoureux dans mon code pour le compilo :).

Merci
0
Réponse 3 / 4
lucieb31 Messages postés 345 Date d'inscription mercredi 14 janvier 2009 Statut Membre Dernière intervention 28 juillet 2012 62
12 avril 2009 à 21:33
dsl je me suis trompé, c'est le mode wt que je voulais dire
0
fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 844
12 avril 2009 à 21:36
c'est le mode wt que je voulais dire
Le mode wt n'existe pas non plus ^^.
0
lucieb31 Messages postés 345 Date d'inscription mercredi 14 janvier 2009 Statut Membre Dernière intervention 28 juillet 2012 62 > fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022
12 avril 2009 à 21:37
si
0
fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 844 > lucieb31 Messages postés 345 Date d'inscription mercredi 14 janvier 2009 Statut Membre Dernière intervention 28 juillet 2012
12 avril 2009 à 21:44
T'as le choix entre : a, a+,r,r+,w,w+. Après tu peux utiliser ou pas b dans les modes précédents.
Mais t n'est pas standard et ne doit donc pas être utilisé.
0
lucieb31 Messages postés 345 Date d'inscription mercredi 14 janvier 2009 Statut Membre Dernière intervention 28 juillet 2012 62 > fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022
12 avril 2009 à 21:48
alors faudra que tu dise ça a mes profs de programmation, parce que c'est ecris dans mon cours, et on l'utilise lorsqu'on programme, ça permet de definir un fichier texte
0
fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 844 > lucieb31 Messages postés 345 Date d'inscription mercredi 14 janvier 2009 Statut Membre Dernière intervention 28 juillet 2012
12 avril 2009 à 22:10
Eh bien tes profs ne connaissent pas la norme.
Voici un extrait de la norme C99 pris sur le site officiel (http://www.open-std.org/JTC1/SC22/WG14/www/docs/n1124.pdf pour t'en convaincre (il en va de même pour les autres). 
The argument mode points to a string. If the string is one of the following, the file is
open in the indicated mode. Otherwise, the behavior is undefined.231)
                   open text file for reading
r
                   truncate to zero length or create text file for writing
w
                   append; open or create text file for writing at end-of-file
a
                   open binary file for reading
rb
                   truncate to zero length or create binary file for writing
wb
                   append; open or create binary file for writing at end-of-file
ab
                   open text file for update (reading and writing)
r+
                   truncate to zero length or create text file for update
w+
                   append; open or create text file for update, writing at end-of-file
a+
231) If the string begins with one of the above sequences, the implementation might choose to ignore the
     remaining characters, or it might use them to select different kinds of a file (some of which might not
     conform to the properties in 7.19.2).
r+b or rb+ open binary file for update (reading and writing)
w+b or wb+ truncate to zero length or create binary file for update
a+b or ab+ append; open or create binary file for update, writing at end-of-file

Comme tu peux le voir, aucune mention du "t". Preuve que cela n'est pas standard. Pour le mode texte, a, w et r sont largement suffisant.
0
Réponse 4 / 4
lucieb31 Messages postés 345 Date d'inscription mercredi 14 janvier 2009 Statut Membre Dernière intervention 28 juillet 2012 62
12 avril 2009 à 22:24
merci pour cet eclaiciement, mais alors comment cela ce fait il qu'il n'y ai pas d'erreur lors de la compilation ?
0
fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 844
12 avril 2009 à 22:39
mais alors comment cela ce fait il qu'il n'y ai pas d'erreur lors de la compilation ?
Tout d'abord tout dépend du compilateur que tu utilises. Certain respecte la norme, d'autre non. Ensuite, si certaines choses peuvent être détectées par le compilateur comme (void main au lieu d'int main, etc.), pour d'autre cas c'est beaucoup plus dur à détecter, notamment pour le code dynamique (arguments et compagnie).
0
lucieb31 Messages postés 345 Date d'inscription mercredi 14 janvier 2009 Statut Membre Dernière intervention 28 juillet 2012 62 > fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022
12 avril 2009 à 22:46
ok merci de tes reponses
0

Discussions similaires

comment donner les droits sur fichier partage
ch'ti du 57 -
fil1958 -

2 réponses