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" ?