Discussion :

[zentaf]

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

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

[Neckara]

Bonjour,

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

Sinon tu as une solution relativement simple :

Code :

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

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

[zentaf]

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

[Neckara]

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

Je pense que ton erreur vient de cette ligne :

Code :

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

char tmp[]="";

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

Il faut faire :

Code :

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

char tmp[3] = "";

[zentaf]

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

[Neckara]

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 ?

[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]=""; .

[zentaf]

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