Discussion :

[kass28]

Bonjour,

J'ai lu ce bout de code , mais il y a certaines lignes que je n'ais pas compris.
D'abord voila le code :

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
 
      bool toto(unsigned char data[BUFFER_SIZE], size_t bytes_transferred)
{  
            static const int TRANSPORT_START_VALUE = 0x02;
static const int TRANSPORT_SIZE_LOW_BYTE = 1;
             unsigned int i=0;
	unsigned char* adrBuffer=(unsigned char*)data;
	uint16 * pt;
	int taille_trame;
	while(i<bytes_transferred)
	{
	   pt=(uint16*)              (adrBuffer+i+TRANSPORT_SIZE_LOW_BYTE);
		taille_trame=(int)(*pt);
		if(i+taille_trame>bytes_transferred)
		{
			taille_envoi_session=i;
		}
		i=i+taille_trame;
		//subtilite reseau des fois le paquet recu contient 1 bit de trop par rapport a la taille du paquet TCS
		if(i<bytes_transferred)
		{
			if(data[i]!=TRANSPORT_START_VALUE || taille_trame==0)
			{
				i++;
			}
		}
	}
.....................

les lignes que je n'ai pas compris sont les suivants :

1. unsigned char* adrBuffer=(unsigned char*)data;

1. pt=(uint16*)(adrBuffer+i+byte_packet::TRANSPORT_SIZE_LOW_BYTE);

1. if(data[i]!=TRANSPORT_START_VALUE || taille_trame==0)
   {
   i++;
   }
   ici je n'ai pas compris la condition ???

Merci pour votre reponse

[Médinoc]

1. Je dirais que le cast est inutile.
2. Ce pointeur (qui devrait être const) est utilisé pour lire un entier de 16 bits se trouvant dans le buffer, à l'offset i+TRANSPORT_SIZE_LOW_BYTE.
   
   • Je vois une erreur là-desdans : l n'y a pas de gestion de l'endianness.
3. Ça fait (à deux lettres près) ce que le commentaire dit: Si le paquet reçu contient un byte de trop, la data ne sera pas la valeur "magique" attendue, et on saute l'octet en trop.
   
   • Pour le reste de la condition, j'ai un doutetoutefois, et je flaire un bug (on saute l'octet tout seul, mais pas la les 16 bits de taille)

[kass28]

1. Je dirais que le cast est inutile.
2. Ce pointeur (qui devrait être const) est utilisé pour lire un entier de 16 bits se trouvant dans le buffer, à l'offset i+TRANSPORT_SIZE_LOW_BYTE.
   
   • Je vois une erreur là-desdans : l n'y a pas de gestion de l'endianness.
3. Ça fait (à deux lettres près) ce que le commentaire dit: Si le paquet reçu contient un byte de trop, la data ne sera pas la valeur "magique" attendue, et on saute l'octet en trop.
   
   • Pour le reste de la condition, j'ai un doutetoutefois, et je flaire un bug (on saute l'octet tout seul, mais pas la les 16 bits de taille)

C'est quoi la gestion de l'endianness ?

[Médinoc]

http://fr.wikipedia.org/wiki/Endianness

Les PCs sont des machines Little-Endian.
Les Macs non-intel sont des machines Big-Endian
D'autres machines (comme certains serveurs unix) sont également des machines Big-Endian.

C'est pourquoi, pour les transmissions réseau, il a bien fallu choisir un standard. Et ce standard préconise la transmission de données en Big-Endian.
Si le poste qui reçoit les données est un PC, il faut soit lire les mots octet par octet en tenant compte de l'endianness (ce qui n'est pas fait ici), soit traduire avec les fonctions faites pour ça (ici, ce sera ntohs(), comme network-to-host-short). Le code ne fait ni l'un ni l'autre.