Problème au niveau du sprintf

Bonjour à tous,

j'ai écrit une fonction en C qui transforme un tableau de bytes en un string.
Ma fonction marche super bien sous Linux. Maintenant, je suis passé à Windows pour la tester et là ça fait des trucs bizarres, d'après mes tests c'est le sprintf qui foire.

Voyez vous c'est quoi le problème?? car moi ça m'a rendu fou 8O

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

#include <stdio.h> #include <string.h>   void bytesToString(unsigned char *bytes,int t_bytes,char *string){   int i; char tmp[]="00";   for(i=0;i<t_bytes;i++){ sprintf(tmp,"%02hx",bytes[i]); strcat(string,tmp); } }   int main(){   unsigned char bytes[4]={0xf1,0xf2,0xf3,0xf4}; char str[2*4+1]; bytesToString(bytes, 4,str); printf("\n%s\n",str); return 0; }

---

Merci,
ilikecz

Bonjour,

As-tu vérifié que ce n'est un problème de little/big endian ?

Sinon tu as une solution relativement simple :

Code:

---

1 2 3

char conversion[] = '1','2',....'F'; tab[i] = conversion[nombre%16]; tab[i+1] = conversion[nombre/16];

---

Merci pour ton code, mais franchement je préfère le mien pour la simple raison que cette fonction je l'utilise un petit peu partout dans mon projet, donc je ne veux pas changer.

ce qui m'étonne c'est le comportement du sprintf qui, en faisant un debug, j'ai pu remarqué qu'il s'exécute une fois et après on sort directement de la boucle, avec une valeur de i=49 8O

Déjà ton code minimal ne compile pas.
Tu as oublié un return 0 et aussi de déclarer i.
De plus studio.h n'existe pas :aie:

Je pense que ton erreur vient de cette ligne :

Code:

---

char tmp[]="";

---

Ici tu déclare un tableau de char d'un char.

Il faut faire :

Code:

---

char tmp[3] = "";

---

oui j'ai dû me tromper quand j'ai rédigé sur le forum, mais sinon ce n'est pas ça le problème, mon code compile bien et même marche parfaitement sous linux \o/

pour tmp j'avais mis char tmp[]="00"; il se débrouille après. mais le problème ne vient pas de ça ni des remarques que tu m'as faites.

Merci qd même =) essayes un debug pour voir -si tu peux bien sûr- c'est le sprintf qui foire :S

Je suis aussi sous Linux.
Peut-être que c'est le format hx qui n'est pas reconnu sous windows?
Pourrais-tu nous montrer ce que donne le sprintf ?

Le comportement du programme est indéterminé et si il marche super bien sous Linux, c'est un malheureux hasard :
Le strcat() dans bytesToString() suppose que le paramètre string est une chaine de caractères (vide ou non). Un simple tableau ne suffit pas.
Or, lors de l'appel, l'argument correspondant, le tableau str, est local donc non initialisé et contient n'importe quoi.
Il faut l'initialiser par char str[2*4+1]=""; .

Citation:

---

Envoyé par diogene

Le comportement du programme est indéterminé et si il marche super bien sous Linux, c'est un malheureux hasard :
Le strcat() dans bytesToString() suppose que le paramètre string est une chaine de caractères (vide ou non). Un simple tableau ne suffit pas.
Or, lors de l'appel, l'argument correspondant, le tableau str, est local donc non initialisé et contient n'importe quoi.
Il faut l'initialiser par char str[2*4+1]=""; .

---

Merci, ça marche maintenant ;)