struct sockaddr_in et gethostbyname

Version imprimable

Bonjour,

Je m'embrouille un peu avec toutes les structures et fonctions relatives aux adresses, j'aimerais savoir si le code suivant est bien valide (argv[2] est un nom d'hôte) :
Code:

1 2 3 4 5 6 7 8 9 10 unsigned long adr; struct hostent *host; struct sockaddr_in serveur_adr; if((host = gethostbyname(argv[2])) == NULL){ /* raler */ } adr = inet_addr( inet_ntoa( *((struct in_addr *)host->h->addr) ) ); serveur_adr.sin_addr.s_addr = htonl(adr);
Merci de votre aide !

Normalement, ça fonctionne comme ça :

Code:

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
  char *nom_serveur;
  int port_serveur;
  struct sockaddr_in adresse;
  struct hostent *infos_serveur;
 
....
 
infos_serveur = gethostbyname(nom_serveur);
 
  if (infos_serveur == NULL)
  {
    fprintf(stderr, "Bad resolution :  %s\n",
            nom_serveur);
    exit(EXIT_FAILURE);
  }
 
  adresse.sin_family = infos_serveur->h_addrtype;
 
  memcpy(&adresse.sin_addr.s_addr, infos_serveur->h_addr,
         infos_serveur->h_length);
 /*voir */
 /*
    memcpy(&adresse.sin_addr.s_addr, *(infos_serveur->h_addr_list,
         sizeof(adresse.sin_addr.s_addr);
 */

Au passage, inet_addr retourne un in_addr_t et pas un long.

En fait le problème est que argv[2] peut être soit :

_ une adresse IP;
_ un nom d'hôte;
_ rien du tout (on utilisera alors INADDR_ANY);

Donc le code ressemble à ceci :
Code:

1 2 3 4 5 6 7 8 9 10 11 adr = INADDR_ANY; if(argc == 3){ if(is_ip_addr(argv[2]){ adr = inet_addr(argv[2]); } else if(is_host_name(argv[2])){ adr = inet_addr( inet_ntoa( *((struct in_addr *)host->h->addr) ) ); } } serveur_adr.sin_addr.s_addr = htonl(adr);

Citation:
Envoyé par j0o0
Code:

1 2 3 4 5 6 7 8 9 10 11 adr = INADDR_ANY; if(argc == 3){ if(is_ip_addr(argv[2]){ adr = inet_addr(argv[2]); } else if(is_host_name(argv[2])){ adr = inet_addr( inet_ntoa( *((struct in_addr *)host->h->addr) ) ); } } serveur_adr.sin_addr.s_addr = htonl(adr);
Cette ligne est inutilement compliquée non ? Tout ça pour passer du type in_addr au type in_addr_t (uint32_t). Cela pourrait être amélioré en sachant que s_addr (du type in_addr_t donc) est un membre (le seul d'ailleurs) de la structure in_addr, non ?

Edit : Mhh j'ai rien dit, ça a l'air qd même plus complexe que ça ^^

19/05/2007, 13h44
j0o0

Code:

adr = (in_addr_t)((struct in_addr *)host->h_addr);

?
19/05/2007, 13h54
millie

Citation:

Envoyé par j0o0

Code:

adr = (in_addr_t)((struct in_addr *)host->h_addr);

?

Hein ?

Normalement, adr est censé être un in_addr_t, je ne sais pas pourquoi tu l'as déclaré en unsigned long :?

Ben il devait y avoir une raison (surement pas bonne) que j'ai oublié :?

Et si je fais simplement :
Code:

1 2 3 4 5 6 7 8 9 10 in_addr_t adr; struct hostent *host; struct sockaddr_in serveur_adr; if((host = gethostbyname(argv[2])) == NULL){ /* raler */ } adr = (in_addr_t)host->h_addr; serveur_adr.sin_addr.s_addr = htonl(adr);
Y'a une erreur quelquepart ou pas ?

Voilà, je suis arrivé à ça, ça fonctionne, mais je sais pas si le code est bon ;)

Code:

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
/* Entetes etc. */
#define MON_PORT 80
 
int main(int argc, char **argv)
{
    struct sockaddr_in	server;
    struct hostent	*h_server = NULL;
    char		*name = argv[1];
 
    memset(&server, 0, sizeof server);
    server.sin_port = htons(MON_PORT);
 
    /* Si on a lancé le programme avec un paramètre */
    /* NB : Je commence à argv[1] */
    if(argc == 2)
    {
        if((h_server = gethostbyname(name)) != NULL)
        {
            memcpy(&server.sin_addr.s_addr, *h_server->h_addr_list, sizeof server.sin_addr.s_addr);
            server.sin_family = h_server->h_addrtype;
        }
        else
        {
            server.sin_addr.s_addr = inet_addr(name);
            server.sin_family = AF_INET;
        }
    }
    else
    {
        server.sin_addr.s_addr = htonl(INADDR_ANY);
        server.sin_family = AF_INET;
    }
 
    printf("Adresse IP : %s\n", inet_ntoa(server.sin_addr));
 
    return 0;
}

Edit : Si tu travailles sous UNIX/Linux, je te conseille d'utiliser inet_ntop et inet_pton ;)

19/05/2007, 14h26
j0o0

Ok je vais tester ça, merci de votre aide !