Discussion :

[gangsoleil]

Bonjour,

J'utilise les fonctiosn gethostname et gethostbyname afin de récupérer l'adresse IP d'une machine.

La fonction gethostbyname rempli une variable de type hostent :

Code :

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

1
2
3
4
5
6
7
8
 
 struct hostent {
         char    *h_name;         /* canonical name of host */
         char    **h_aliases;     /* alias list */
         int     h_addrtype;      /* host address type */
         int     h_length;        /* length of address */
         char    **h_addr_list;   /* list of addresses */
     };

Mon soucis est le suivant : est-il nécessaire d'initialiser les champs de cette structure ou non (normalement oui), sachant que l'exemple du man (solaris 5.8) ne les rempli pas ?
Voici un exemple de test, basé sur celui du man.

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
41
42
 
 
/* Necessite d'utiliser l'option -lnsl pour compiler */
 
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <netinet/in.h>
#include <arpa/inet.h>
#include <netdb.h>
#include <unistd.h>
 
 
int main (void)
{
	struct hostent *hp; /* la variable de type hostent qui pose probleme */
	char **p; /* utilisé pour parcourir le tableau de resultat */
	int ret; /* code de retour */
	char hostname[256]; /*nom d'hote*/
 
	ret = sethostent (1); /* initalisation des hotes */
	printf ("ret sethostent = %d\n", ret); :*debug*/
 
	ret = gethostname (hostname, 256); /* récupération du nom d'hote */
	printf ("ret gethostname = %d ; hostname : %s\n", ret, hostname); /*debug*/
 
	hp = gethostbyname (hostname); /* appel pour recuperer les adresses IP */
 
	for (p = hp->h_addr_list; *p != 0; p++) { /*parcour du tableau de resultat, issu du man */
		struct in_addr in;
		char **q;
		(void) memcpy(&in.s_addr, *p, sizeof (in.s_addr));
		(void) printf("%s\t%s", inet_ntoa(in), hp->h_name);
		for (q = hp->h_aliases; *q != 0; q++)
		(void) printf(" %s", *q);
		(void) putchar('\n');
	}
	endhostent();
	exit (0);
}

Merci.

[gege2061]

Salut,
Il y a un problème si tu n'initialise pas la structure :
Quelle problème :
Normalement, il n'y as pas d'initialisation à faire (peut être testé le retour de la fonction si ele peut échouer).

P.S. : quelques conseilles pour le code

• Tu peut enlevé les (void) avant les fonction memcpy, printf...
• La fonction exit() ne termine pas correctement un programme, il faut utiliser return
• Le nom de l'hote ne peut pas dépasser les 255 caractères :

[DaZumba]

Juste un detail,

Tu peut enlevé les (void) avant les fonction memcpy, printf...

Peut-etre que gangsoleil utilise un verificateur de code tel que lint, qui a la desagreable habitude de se plaindre des qu'un retour de fonction est ignore (comme souvent avec printf). Mettre un void devant permet de lui couper le sifflet en disant "je sais, cette fonction retourne quelque-chose, mais je m'en moque".

[gege2061]

Peut-etre que gangsoleil utilise un verificateur de code tel que lint, qui a la desagreable habitude de se plaindre des qu'un retour de fonction est ignore (comme souvent avec printf). Mettre un void devant permet de lui couper le sifflet en disant "je sais, cette fonction retourne quelque-chose, mais je m'en moque".

Dans ce quoi, soit faut en mettre à toutes les fonctions et ça alourdie inutilement le code, soit changer de logiciel de vérification.

[Ti-R]

gege2061:
Le nom de l'hote ne peut pas dépasser les 255 caractères

En principe le nom de l'hôte ne doit pas dépasser 255 caractères sous Windows, même si je ne suis pas 100% sur... et même gangsoleil spécifie la taille maximum :

Code :

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

gethostname (hostname, 256);

Donc il n'y aura jamais de débordement !

Je ne suis pas sur à 100% car je n'ai pas de code avec moi, mais je crois que je ne l'initialise pas non plus, et je n'ai jamais eu de problème avec.

[gangsoleil]

je vous remercie pour toutes ces précisions.

Pour info,

• les (void) devant le code sont ceux de l'exemple du man. Je les ai supprimé dans mon code
• Pareil pour l'appel à exit
• Effectivement, j'ai un doute sur le nombre de caractères maximal du nom d'hôte. je vais me renseigner et régler ce problème

merci à vous

PS

En principe le nom de l'hôte ne doit pas dépasser 255 caractères sous Windows,

Comme précisé, je développe sous Solaris 5.8

[Ti-R]

sachant que l'exemple du man (solaris 5.8 ne les rempli pas ? )

Le man de solaris pouvait être trouvé sur le net comme exemple... c'est ce que j'en ai déduit

Sinon je confirme que sous Windows je n'initialise pas la variable hostent et cela parait logique, car je n'avais pas trop regardé ton code, je pensais me souvenir à tord que c'était un passage en paramètre du pointeur, ce qui pouvait générer ton problème, mais c'est un retour de fonction..... donc forcément il n'y a pas besoin d'être initialisé... car la fonction s'appelle gethostbyname (hostname) et a la fin de l'exécution renvoie un pointeur sur une structure, la question serait +, faut t'il la libérer...

Extrait de la MFC:
The application must never attempt to modify this structure or to free any of its components. Furthermore, only one copy of this structure is allocated per thread, so the application should copy any information it needs before issuing any other Windows Sockets function calls.

Donc en principe les sockets doivent gérer ce bout de code...à vérifier sur solaris