Discussion :

[pinto_armindo]

Bonjour a tous, j'ai un petit soucis concernant la fonction malloc et snprintf:
Voila le sequencement de mon programme:

1. lecture des donnees dans un pipe (ex de donnees lues: 0xFDOO1F4142..... c'est du texte code en ASCII ne depassant pas 100 caracteres)
2. allocation dynamique de memoire de la taille des donnees lues dans le pipe
3. ecriture du message recu dans le pipe dans un ficher texte

Voici le code isole effectuant ces operations:

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
char *pMessageToSend;
int i;
char messageToReceive[1024];
 
fileRead  = ReadFile(
                  gphAutomaticTestTool_HandlePipe,
                  messageToReceive,
                  1024,
                  &bytesRead,
                  NULL);
 
pMessageToSend = malloc(bytesRead);
 
for(i = 0; i < bytesRead; i++)
                        {
                        snprintf(&pMessageToSend[i],
			sizeof(pMessageToSend),
			"%c",
			messageToReceive[i]);
                        }
 
written = fwrite(pMessageToSend,
                        1,
                        strlen(pMessageToSend),
                        gpphAutomaticTestTool_ErrorFile);
 
free(pMessageToSend);

Le probleme que je rencontre est: The memory at "0x79c...." could not be read ou un probleme de socket error 10055.

J'ai probablement mal utilise snprintf ou le malloc. Merci de votre lumiere.

[ram-0000]

Je pense que c'est la séquence "strlen(pMessageToSend" lors du fwrite() qui va pas.

En effet, la chaine que tu écris n'est pas terminée par la caractère 0, donc, le strlen va foirer (strlen() se base sur ce caractère pour calculer la longuer de la chaine)

Question 1:
pourquoi tu ne remplace pas

Code :

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

1
2
3
4
5
6
7
8
 
for(i = 0; i < bytesRead; i++)
{
   snprintf(&pMessageToSend[i],
			sizeof(pMessageToSend),
			"%c",
			messageToReceive[i]);
}

par

Code :

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

1
2
 
memcpy(pMessageToSend, messageToReceive, bytesRead);

Question 2:
Pourquoi fais tu une copie intermédiaire avant l'appel à fwrite(), tu pourrais faire directement :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
fileRead  = ReadFile(
                  gphAutomaticTestTool_HandlePipe,
                  messageToReceive,
                  1024,
                  &bytesRead,
                  NULL);
written = fwrite(messageToReceive,
                        1,
                        bytesRead,
                        gpphAutomaticTestTool_ErrorFile);

[pinto_armindo]

Merci de ta reponse et voila la reponse a tes questions:
Oui j'ai oublie des etapes dans ma sequence d'operations

Question1: je veux recuperer les valeur code en ASCII et les convertir pour lesenregistrer dans le fichier en caractere

Question2: elle decoule de la question 1. je formatte ma chaine a enregistrer. En gros je lis les caracteres ASCII dans le pipe et les transforme en texte pour les enregistrer dans un fichier


Voila merci. Par contre concernant le "strlen" lors du fwrite() il faudrait le remplacer par une autre fonction du genre sizeof(pMessageToSend) ???

[ram-0000]

D'abord, tu lis des caractères dans "messageToReceive" à l'aide de la focntion ReadFile()

Ensuite tu dupliques la chaine recue

Ensuite tu transmets cette chaine dupliquée à l'aide de fwrite()

Pour info, ta duplication est fausse car sizeof(pMessageToSend) va toujours te donner 4 (taille d'un pointeur) et non 1 (taille d'un caractère comme je pense que tu souhaites). La variable pMessageToSend est un pointeur, par contre sizeof(*pMessageToSend) va te donner 1 (taille de l'objet pointé par pMessageToSend)

Autre chose, je simplifie et schématise ton code :

Code :

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

1
2
3
4
 
char carac = 'a';
char autre_carac;
snprintf(&autre_carac, sizeof(new_carac), "%c", carac)

c'est ce que tu fais dans ta boucle for(i = 0; i < bytesRead; i++) c'est exactement la même chose que

Code :

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

1
2
3
4
 
char carac = 'a';
char autre_carac;
autre_carac = carac;

[pinto_armindo]

Ok c'est juste ce que tu me dis.
le hic c'est que la boucle for sert a convertir les donnees en ASCII du pipe en texte dans la variable tampon "pMessaToSend".
C'est du texte que je veux ecrire dans le ficher et pas le code ASCII.

Sinon le sizeof est correct car j'affiche bien les caracteres souhaite. Si je passe le sizeof(*pMessaToSend) j'ai des caracteres indesire a la fin.

[ram-0000]

OK, je commence à comprendre,

tu reçois une chaine de caractères "414243" et tu veux écrire "ABC"

Quel est le format de la chaine de caractères que tu reçois ?

C'est de l'héxadecimal tout le temps (donc un caractère en sortie utilise 2 caractère en entrée) ?

Est ce que le début de la chaine en entrée commence par "0x" ?

[pinto_armindo]

Desole si mon probleme etait confu !

Oui je recois que de l'hexadecimal (code ASCII) et ma chaine commence bien par un 0x...

Typiquement le client du serveur pipe ecrit : 0x00FC001F4143.....

Moi je traite ces donnes pour les ecrire dans un fichier texte

[ram-0000]

Alors cela resemblerait a quelquechose comme cela

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
 
 
// lit un message
char messageToReceive[1024];
fileRead  = ReadFile(
                  gphAutomaticTestTool_HandlePipe,
                  messageToReceive,
                  sizeof(messageToReceive), // au lieu de 1024, c'est plus sur
                  &bytesRead,
                  NULL);
char *ptr = messageToReceive;
 
// saute les 2 caracteres "0x"
ptr += 2;
 
for(i = 0; i < bytesRead; i++)
{
                        snprintf(&pMessageToSend[i],
			sizeof(pMessageToSend),
			"%c",
			messageToReceive[i]);
                        }

[ram-0000]

Alors cela resemblerait a quelquechose comme cela

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
29
30
31
32
33
34
35
36
37
38
 
// lit un message
char messageToReceive[1024];
fileRead  = ReadFile(
                  gphAutomaticTestTool_HandlePipe,
                  messageToReceive,
                  sizeof(messageToReceive), // au lieu de 1024, c'est plus sur
                  &bytesRead,
                  NULL);
 
// la taille de messageToReceive est toujours plus grande que messageToSend
// taille necessaire de messageToSend = (bytesRead / 2) - 2
char messageToSend[1024];
 
// saute les 2 caracteres "0x", c'est pour cela que la boucle commence a 2
// avance de 2 caractere à la fois puisque 2 codes ascii donne un caractere
int nb_byte_to_send = 0;
for(i = 2; i < bytesRead; i += 2)
{
   // transforme 2 caracteres ASCII en un nombre
   char tmp[3];
   tmp[0] = messageToReceive[i];
   tmp[1] = messageToReceive[i + 1];
   tmp[2] = 0;  // terminateur de chaine de caracteres
   int carac = strtol(tmp, NULL, 16);  // conversion de chaine base 16 en nombre
 
   // ecrit dans le buffer
   snprintf(&pMessageToSend[nb_byte_to_send],
         sizeof(pMessageToSend[1]),
         "%c", carac);
   nb_byte_to_send++;
}
 
// ecriture dans le fichier
written = fwrite(pMessageToSend,
                        1,
                        nb_byte_to_send,
                        gpphAutomaticTestTool_ErrorFile);

Attention, pour que ce code marche, il faut:
1/ que le flux d'entrée commence toujours par 2 caractères inutiles ("0x")
2/ qu'il n'y ait que de l'hexadecimal dans le flux d'entrée
3/ que le nombre de caractères soit toujours un multiple de 2