Discussion :

[the_ionic]

Salut,
voilà, c'est tout simple, je souhaite récupérer une adresse MAC dans un tableau (d'unsigned char évidemment) saisie par l'utilisateur au clavier.
Je n'ai pas trouvé de fonction qui convertissait directement une chaine en notation avec des points style 87.23.CD.43.22.45 vers un tableau d'unsigned char (à la manière de inet pour les adresse IP) donc je me suis dis que j'allais passer par un bon vieux scanf.

Mon code est le suivant:

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
 
unsigned char         host_mac_addr[ETH_ALEN];    /* source MAC address*/
unsigned char         dst_mac_addr[ETH_ALEN];     /* dest MAC address */
int i;
 
printf("DST MAC ADDRESS?\n");
  scanf("%2x.%2x.%2x.%2x.%2x.%2x[^\n]",&dst_mac_addr[0],&dst_mac_addr[1],&dst_mac_addr[2],&dst_mac_addr[3],&dst_mac_addr[4],&dst_mac_addr[5]);
 
printf("HOST MAC ADDRESS?\n");
scanf("%2x.%2x.%2x.%2x.%2x.%2x[^\n]",&host_mac_addr[0],&host_mac_addr[1],&host_mac_addr[2],&host_mac_addr[3],&host_mac_addr[4],&host_mac_addr[5]);
 
/* AFFICHAGE */
for(i=0;i<ETH_ALEN;i++)
{
    printf("DST_ADDR[i]: %2x\n",dst_mac_addr[i]);
}
 
for(i=0;i<ETH_ALEN;i++)
{
    printf("HOST_ADDR[i]: %2x\n",host_mac_addr[i]);
}

Où est le problème? Et bien lorsque j'affiche les adresses, les 3 premiers octets de DST_ADDR sont toujours à 0... je n'arrive pas à comprendre pourquoi. Je sais bien que c'est dû au deuxième scanf mais pour moi on affecte les valeurs une fois pour toute avec le scanf, c'est pas un pointeur vers les valeurs de l'entrée standard...
Donc si quelqu'un peut m'expliquer mon problème et/ou au passage me donner peut être une solution plus maline pour récupérer une adresse MAC saisie par un utilisateur dans un tableau (d'unsigned char de taille 6). Merci d'avance!

[Gruik]

donc je me suis dis que j'allais passer par un bon vieux scanf.

Lis ça avant d'utiliser scanf
http://xrenault.developpez.com/tutoriels/c/scanf/

Pourquoi tu veux decouper l'adresse mac?

[Emmanuel Delahaye]

Code :

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

1
2
3
 
unsigned char         host_mac_addr[ETH_ALEN];    /* source MAC address*/
scanf("%2x.%2x.%2x.%2x.%2x.%2x[^\n]",&dst_mac_addr[0],&dst_mac_addr[1],&dst_mac_addr[2],&dst_mac_addr[3],&dst_mac_addr[4],&dst_mac_addr[5]);

"%x" attend l'adresse d'un unsigned int, pas celle d'un unsigned char...

fgets() + une boucle avec strtoul() est plus judicieuse...

[the_ionic]

Pourquoi tu veux decouper l'adresse mac?

Tout simplement parce que cette adresse MAC est destinée à remplir la structure sockaddr_ll utilisée pour les sockets bas niveau et que celle-ci attends une adresse sous la forme d'un tableau d'unsigned char de taille 6...
Et donc mon appli doit demander à l'utilisateur de rentrer l'adresse MAC de destination et source et je me dis que la manière la plus naturelle pour moi est de rentrer l'adresse séparée par des "."
Je dois donc passer d'une notation pointée à un tableau d'unsigned char, voilà pourquoi je découpe.

[Gruik]

Tout simplement parce que cette adresse MAC est destinée à remplir la structure sockaddr_ll utilisée pour les sockets bas niveau et que celle-ci attends une adresse sous la forme d'un tableau d'unsigned char de taille 6...
Et donc mon appli doit demander à l'utilisateur de rentrer l'adresse MAC de destination et source et je me dis que la manière la plus naturelle pour moi est de rentrer l'adresse séparée par des "."
Je dois donc passer d'une notation pointée à un tableau d'unsigned char, voilà pourquoi je découpe.

Ah okay, désolay.
Bein j'aurais découpé à la main.. Mais en reflechissant, je m'aperçois que c'est pas aussi simple, à cause de la conversion hexa => decimal. Donc, oui, ptet que scanf est le mieux (à condition que l'utilisateur soit digne de confiance).
Sinon, il faudrait faire fgets+sscanf avec un controle à la main de la chaine saisie.

[the_ionic]

Bon, j'ai un truc qui marche, ça me paraît un poil compliqué car y'a même pas de vérification d'erreurs encore mais bon, ça marche... je le mets au cas où quelqu'un à le même problème.
Si quelqu'un a un commentaire, qu'il hésite pas, je suis preneur de mieux!

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
 
  #define MAC_POINTED_MAX_LENGTH 19
 
  unsigned char s_board_addr[MAC_POINTED_MAX_LENGTH];
  unsigned char         dst_mac_addr[ETH_ALEN];  
  int nbSeparator = 0;
  int currentPos = 0;
  int separatorPos = 0;
 
  printf("BOARD MAC ADDRESS?\n");
 
  fgets(s_board_addr,MAC_POINTED_MAX_LENGTH,stdin);
  s_board_addr[MAC_POINTED_MAX_LENGTH-1] = '.';
 
  while (nbSeparator < 6 && currentPos<MAC_POINTED_MAX_LENGTH){
    if(s_board_addr[currentPos]=='.'){
      dst_mac_addr[nbseparator] = strtoul(s_board_addr+separatorPos,NULL,16);
      nbSeparator++;
      separatorPos = currentPos+1;
    }
    currentPos++;
  }

Voili voilou, merci de m'avoir guidé!

[Emmanuel Delahaye]

Code :

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

1
2
 
      dst_mac_addr[nbseparator] = strtoul(s_board_addr+separatorPos,NULL,16);

Apprends à utiliser le 2ème paramètre de stroul()... C'est le moment !

[the_ionic]

Apprends à utiliser le 2ème paramètre de stroul()... C'est le moment !

Bon, encore une fois grace à tes conseils avisés (et à ceux des autres aussi), me voici arrivé à une solution plus courte et concise.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
 
  char **separatorptr;
  char s_board_addr[MAC_POINTED_MAX_LENGTH];
 
  fgets(s_board_addr,MAC_POINTED_MAX_LENGTH,stdin);
 
  addr[0] = strtoul(s_board_addr,separatorptr,16);
  addr[1] = strtoul(*separatorptr+1,separatorptr,16);
  addr[2] = strtoul(*separatorptr+1,separatorptr,16);
  addr[3] = strtoul(*separatorptr+1,separatorptr,16);
  addr[4] = strtoul(*separatorptr+1,separatorptr,16);
  addr[5] = strtoul(*separatorptr+1,separatorptr,16);

Oui une boucle for serait encore plus joli mais bon, je donne la source tant que j'y pense, ça pourra aider quelqu'un et la boucle n'est pas le plus chiant dans cette affaire!

[Gruik]

Ah bein ca tue, on peut donner la base à strtoul, çay cool

[Mayti4]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
char **separatorptr;
  char s_board_addr[MAC_POINTED_MAX_LENGTH];
 
  fgets(s_board_addr,MAC_POINTED_MAX_LENGTH,stdin);
 
  addr[0] = strtoul(s_board_addr,separatorptr,16);
  addr[1] = strtoul(*separatorptr+1,separatorptr,16);
  addr[2] = strtoul(*separatorptr+1,separatorptr,16);
  addr[3] = strtoul(*separatorptr+1,separatorptr,16);
  addr[4] = strtoul(*separatorptr+1,separatorptr,16);
  addr[5] = strtoul(*separatorptr+1,separatorptr,16);

Ce code fonctionne ?

[fearyourself]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
char **separatorptr;
  char s_board_addr[MAC_POINTED_MAX_LENGTH];
 
  fgets(s_board_addr,MAC_POINTED_MAX_LENGTH,stdin);
 
  addr[0] = strtoul(s_board_addr,separatorptr,16);
  addr[1] = strtoul(*separatorptr+1,separatorptr,16);
  addr[2] = strtoul(*separatorptr+1,separatorptr,16);
  addr[3] = strtoul(*separatorptr+1,separatorptr,16);
  addr[4] = strtoul(*separatorptr+1,separatorptr,16);
  addr[5] = strtoul(*separatorptr+1,separatorptr,16);

Ce code fonctionne ?

Bien sûr que non...

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
39
40
 
#include <stdio.h>
#include <stdlib.h>
 
int main()
{
char *separatorptr;
  char s_board_addr[256] = "87.23.CD.43.22.45";
  long addr[6];
 
#if 0  
  if(fgets(s_board_addr,sizeof s_board_addr,stdin)!=NULL) {
#else
  {
#endif
 
    addr[0] = strtoul(s_board_addr,&separatorptr,16);
 
    if(separatorptr!=NULL) {
       addr[1] = strtoul(separatorptr+1,&separatorptr,16);
       if(separatorptr!=NULL) {
          addr[2] = strtoul(separatorptr+1,&separatorptr,16);
          if(separatorptr!=NULL) {         
            addr[3] = strtoul(separatorptr+1,&separatorptr,16);
            if(separatorptr!=NULL) {         
              addr[4] = strtoul(separatorptr+1,&separatorptr,16);
               if(separatorptr!=NULL) {         
                 addr[5] = strtoul(separatorptr+1,&separatorptr,16);
	         printf("On a donc: %x %x %x %x %x %x\n",addr[0],  
                     addr[1],addr[2],addr[3],addr[4],addr[5]);
		 return EXIT_SUCCESS;
               }
             }
          }
       }
    }
  }
printf("ERREUR\n");
return EXIT_FAILURE;
}

Me semble déjà mieux...

Jc

[the_ionic]

Bien sûr que non...

Euh, ben si ça fonctionne... je n'aurais quand même pas posté un code qui marche pas.

De plus, je ne vois pas la différence avec ta version. Certes tu fais une vérification d'erreurs en plus mais j'avais bien marqué plus haut que pour l'instant, je n'en faisais pas, et enfin tu remplaces un char ** par un char * en jouant avec les & ce qui j'en conviens marche tout aussi bien mais je ne vois pas la différence... peux tu m'expliquer pourquoi ton code va mieux marcher?

Attention, je suis d'accord, ton code est plus joli et carré, mais c'est par rapport au fait de dire que bien sûr que non, mon code ne marche pas.

Merci d'avance.

[Emmanuel Delahaye]

Euh, ben si ça fonctionne... je n'aurais quand même pas posté un code qui marche pas.

Erreur classique. Ce n'est pas parce que 'ça fonctionne' (en apparence) que le code est correct. Le C, c'est pas un langage de débutant. Des erreurs, on peut en faire plein, et ça ne se traduira pas forcément par un comportement visible.

En l'occurence, ton code invoque un comportement indéterminé.

Code :

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

char **separatorptr;

separatorptr est un pointeur non initialisé. Sa valeur est indéterminée, il pointe n'importe où.

Code :

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

1
2
3
4
5
char s_board_addr[MAC_POINTED_MAX_LENGTH];
 
  fgets(s_board_addr,MAC_POINTED_MAX_LENGTH,stdin);
 
  addr[0] = strtoul(s_board_addr,separatorptr,16);

Tu passes une valeur indéterminée à une fonction. Le comportement est indéfini. C'est un bug. Il peut arriver n'importe quoi.

Comme indiqué. Il fallait définir un pointeur de char :

Code :

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

char *separatorptr;

et passer son adresse à la fonction :

Code :

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

  addr[0] = strtoul(s_board_addr,&separatorptr,16);

etc.
Voilà. Maintenant, le code est correct.

(Et évite de faire le malin, tu es repéré...)

[the_ionic]

Merci pour la réponse.

(Et évite de faire le malin, tu es repéré...)

Je voulais pas faire le malin, c'est vrai que quand je relis mon post, ça fait un peu branleur, mais en fait, je voulais juste qu'on m'explique d'où venait l'erreur.

Désolé si j'ai heurté la sensibilité de certains par le ton du message.

Encore merci pour la réponse qui est complète et qui me fait comprendre pourquoi ça aurait pu foirer...

[fearyourself]

Merci pour la réponse.

Je voulais pas faire le malin, c'est vrai que quand je relis mon post, ça fait un peu branleur, mais en fait, je voulais juste qu'on m'explique d'où venait l'erreur.

Désolé si j'ai heurté la sensibilité de certains par le ton du message.

Encore merci pour la réponse qui est complète et qui me fait comprendre pourquoi ça aurait pu foirer...

Maintenant tu sais et aucune sensibilité a été heurté, sinon cela ferait bien longtemps qu'on aurait des bleus partout...

Jc