Discussion :

[blackhawk94]

Bonjour tout le monde
je veux déclarer une structure censée représenter un télégramme radio. Ce télégramme est supposé contenir:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
struct Telegramme
{
  uint8_t   u8choice;             //identifiant
  uint8_t   u8Header;            
  uint8_t   u8Data[];             //données applicatives
  uint32_t  u32SenderId;        // ID
  uint8_t   u8Status;             //statut
  uint8_t   u8Chk;                 //Checksum
};

le datagramme doit avoir cet ordre bien précis. Le problème je ne connais pas la taille des données applicatives avant exécution car elle dépend justement du u8Header.Le header est censé définir la longueur du champ u8Data[].Disons que suivant le type du télégramme le tableau u8Data[] peut inclure 1, 2 ou 3 de ces variables :

Code :

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

1
2
3
4
 
uint32_t  u32RLC   // 4 byte
uint32_t  u32RND   // 4 byte
uint32_t  u32CMAC  // 4 byte

Ne faites pas attention aux noms des variables (c'est des notions souvent utilisés en cryptographie) peu importe. je veux que le tableau prenne la taille des variables qui seront incluses suivant les types de telegrammes (si deux variables le tableau aura une taille de 8 bytes si 3 alors 12 bytes) je ne veux pas de buffer non plus avec une taille maximale de 12 bytes pour le worst case.

J'ai donc pensé à déclarer une structure:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
struct Telegramme
{
  uint8_t   u8choice;             //identifiant
  uint8_t   u8Header;            
  uint32_t  u32SenderId;        // ID
  uint8_t   u8Status;             //statut
  uint8_t   u8Chk;                 //Checksum
  uint8_t   u8LengthData;       //Taille des données applicatives    
  uint8_t*   u8Data[];             //données applicatives
 };

avec un pointeur sur le tableau u8Data[] car sur mon development tool je n'ai pas le droit d'utiliser malloc.donc je me suis dit qu'avec un pointeur sur u8Data[] je pourrais suivant u8LengthData remplir les champs du tableau au cas par cas.
Une autre contrainte: u8Header est un champ de bits comme suit:

Code :

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

1
2
3
4
5
6
7
8
9
 
struct header {
uint8          u1Prpse:  1;
uint8          u1PrpseInfo: 1;
uint8              : 4;                      // unused bits 
uint8          u1RNDinc: 1;
uint8          u1CMACinc: 1;
} ;
struct header u8Header;

j'ignore totalement comment "agencer" cette structure Telegramme avec toutes ces contraintes et surtout comment remplir u8Data dans l'application.j'imagine qu'avec une boucle for ca devrait faire l'affaire mais je ne vois pas comment:

Code :

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

1
2
3
4
5
6
7
8
9
int main()
{
   int i;    //counter
   struct Telegramme myTel;     //je déclare une structure de type Telegramme
   for(i=0;i<2;i++)    //prenons le cas u8LengthData est égal à 2 c-à-d les données applicatives font 8 bytes
   {
      myTel.u8Data[i]= 0x11223344;   //comment faire?? est-ce juste?
   }

Merci merci !!

[Sve@r]

Le problème je ne connais pas la taille des données applicatives avant exécution car elle dépend justement du u8Header...
...je veux que le tableau prenne la taille des variables qui seront incluses suivant les types de telegrammes (si deux variables le tableau aura une taille de 8 bytes si 3 alors 12 bytes) je ne veux pas de buffer non plus avec une taille maximale de 12 bytes pour le worst case...
...car sur mon development tool je n'ai pas le droit d'utiliser malloc...

Bonjour

Tes contraintes sont exclusives. Tu ne peux pas créer de truc à taille variable sans faire (au choix)

• définir un truc avec une taille maximale
• allouer ton truc via malloc/calloc

Ce que tu peux faire, c'est donc créer une structure contenant la data (avec taille maximale) et sa taille réelle

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
typedef struct {
    uint8_t data[4];
    size_t len;
} t_u8Data;
 
typedef struct {
  uint8_t   u8choice;             //identifiant
  uint8_t   u8Header;            
  uint32_t  u32SenderId;        // ID
  uint8_t   u8Status;             //statut
  uint8_t   u8Chk;                 //Checksum
  uint8_t   u8LengthData;       //Taille des données applicatives    
  t_u8Data   u8Data;             //données applicatives
} t_Telegramme;
 
int main()
{
    t_Telegramme telg;
 
    // Ensuite, quand la taille est connue
    assert taille_connue <= 4;
    telg.u8Data.len=taille_connue;
    for (i=0; i < telg.u8Data.len; i++)
        telg.u8Data.data[i]=... // La valeur que tu veux lui mettre
}

C'est en fait ce à quoi tu avais pensé dans ta dernière structure (celle avec ton pointeur). Sauf que moi, déjà je supprime ce pointeur. Cela n'aurait d'ailleurs pas marché car au lieu de définir un tableau de uint8, tu définissais un tableau de pointeurs uint8 !!!
Et comme j'aime bien grouper les choses qui vont ensembles, j'ai associé spécifiquement lengthData avec data...

[blackhawk94]

Merci Sve@r
oui tu as raison j'avais créé un tableau de pointeurs bête de a part.Il est vrai que ta solution est beaucoup plus lisible et pratique.Je te remercie beaucoup
cordialement