Ne sin_addr.s_addr = INADDR_ANY; besoin htonl à tous?

Depuis d'autres constantes comme INADDR_LOOPBACK sont dans l'ordre des octets de l'hôte, je pense que toutes les constantes dans cette famille devrait avoir htonl appliquées, y compris INADDR_ANY.

(Note: j'ai écrit cette réponse alors que @Mat a été l'édition; sa réponse maintenant, il est également préférable d'être cohérent et de toujours utiliser htonl.)

Justification

Il est un danger pour les futurs responsables de votre code si vous l'écrire comme ceci:

if (some_condition)
    sa.s_addr = htonl(INADDR_LOOPBACK);
else
    sa.s_addr = INADDR_ANY;

Si je passais en revue ce code, je le ferais immédiatement la question de savoir pourquoi l'une des constantes a htonl appliquée et l'autre pas. Et je voudrais signaler un bug, si oui ou non il m'est arrivé d'avoir des "connaissances" que INADDR_ANY est toujours 0 si la conversion c'est un no-op.

Le code que vous écrivez n'est pas seulement sur le fait d'avoir le bon comportement d'exécution, il devrait être aussi évident lorsque c'est possible et facile de croire qu'il est correct. Pour cette raison, vous ne devez pas foirer le htonl autour de INADDR_ANY. Les trois raisons pour ne pas utiliser htonl que je peux voir sont:

1. Il peut offenser expérimentés prise de programmeurs à utiliser htonl parce qu'ils savent qu'il ne fait rien (étant donné qu'ils connaissent la valeur de la constante par cœur).
2. Il nécessite moins de taper à l'omettre.
3. Un faux "performance" d'optimisation (de toute évidence, elle n'a pas d'importance).

N'est droit, dans le sens que les deux INADDR_ANY et htonl sont obsolètes, et conduire à des complexes, laid code qui ne fonctionne qu'avec IPv4. Commutateur à l'aide de getaddrinfo pour l'ensemble de votre adresse de socket besoins de création:

struct addrinfo *ai, hints = { .ai_flags = AI_PASSIVE|AI_ADDRCONFIG };
getaddrinfo(0, "1234", &hints, &ai);

Remplacer "1234" avec votre numéro de port ou le nom du service.

Stevens utilise htonl(INADDR_ANY) constamment dans le livre UNIX Réseau de Programmation (mon exemplaire est à partir de 1990).

La version actuelle est la version de FreeBSD définit 12 INADDR_ constantes dans netinet/in.h; 9 des 12 exiger htonl() pour une bonne fonctionnalité. (Les 9 sont INADDR_LOOPBACK et 8 autres de groupe de multidiffusion tels que les adresses INADDR_ALLHOSTS_GROUP et INADDR_ALLMDNS_GROUP.)

Dans la pratique, il ne fait aucune différence si vous utilisez INADDR_ANY ou htonl(INADDR_ANY), autres que le possible des performances de htonl(). Et de même que possible des performances peut ne pas exister, avec mon 64 bits gcc 4.2.1, tournant sur n'importe quel niveau de l'optimisation à tout semble activer au moment de la compilation htonl() conversion de constantes.

En théorie, il serait possible de certains opérateur de redéfinir INADDR_ANY à une valeur où htonl() réellement fait quelque chose, mais un tel changement permettrait de briser des dizaines de milliers de pièces existantes de code là-bas et ne survivrait pas dans le "monde réel"... Trop de code qui dépend explicitement ou implicitement sur INADDR_ANY être défini comme une sorte de valeur zéro entier. Stevens n'ont probablement pas l'intention d'inciter quiconque à assumer que INADDR_ANY est toujours à zéro quand il a écrit:

cli_addr.sin_addr.s_addr = htonl(INADDR_ANY);
cli_addr.sin_port        = htons(0);

Dans l'attribution d'une adresse locale pour l'
client à l'aide de bind, nous avons mis l'
Adresse Internet INADDR_ANY et
les 16 bits d'Internet de port à zéro.

Je n'ai pas l'habitude de répondre quand il y a déjà un "décent" réponse. Dans ce cas, je vais faire une exception parce que l'information que j'ai ajoutée à la réponse à ces questions est d'être mal interprété.

INADDR_ANY est défini comme un zéro les bits d'adresse IPv4, 0.0.0.0 ou 0x00000000. L'appel de htonl() sur cette valeur entraîne la même valeur zéro. Par conséquent, l'appel htonl() sur cette valeur de la constante n'est pas techniquement nécessaire.

INADDR_ALL est définie comme une de tous les bits de l'adresse IPv4, 255.255.255.255 ou 0xFFFFFFFF. L'appel de htonl() avec INADDR_ALL sera de retour INADDR_ALL. Encore une fois, l'appel de htonl() n'est pas techniquement nécessaire.

Une autre constante définie dans les fichiers d'en-tête est INADDR_LOOPBACK, définie comme 127.0.0.1, ou 0x7F000001. Cette adresse est donnée dans le réseau d'ordre d'octet, et ne peuvent être transmis à l'interface sockets sans htonl(). Vous devez utiliser htonl() avec cette constante.

Certains suggèrent que la cohérence et la lisibilité du code à la demande que les programmeurs utilisent htonl() pour toute constante nommée INADDR_* -- car il est nécessaire pour certains d'entre eux. Ces affiches sont mauvais.

Un exemple donné dans ce fil de discussion est:

if (some_condition)
    sa.s_addr = htonl(INADDR_LOOPBACK);
else
    sa.s_addr = INADDR_ANY;

Citant "Jean Zwinck":

"Si j'étais la révision de ce code, je le ferais immédiatement la question de savoir pourquoi l'une des constantes a htonl appliquée et l'autre pas. Et je le signaler comme un bug, si oui ou non il m'est arrivé d'avoir des "connaissances" que INADDR_ANY est toujours 0 si la conversion c'est un no-op. Et je pense (et espère) de nombreux autres responsables feraient la même chose."

Si je recevaient ce un rapport de bug, je voudrais immédiatement jeter. Ce processus me sauver beaucoup de temps, de la mise en service de rapports de bogues à partir de gens qui n'ont pas la "base de connaissances minimales" que INADDR_ANY est toujours 0. (Ce qui suggère que le fait de connaître les valeurs de INADDR_ANY et coll. en quelque sorte, viole l'encapsulation ou tout ce qui est un autre non-starter-même les nombres sont utilisés dans le netcat de sortie et à l'intérieur du noyau. Les programmeurs ont besoin de connaître la forme de valeurs numériques. Les gens qui ne savent pas ne sont pas dépourvues de à l'intérieur de connaissances, ils manquent de base connaissance de la région.)

Vraiment, si vous avez un programmeur maintien de sockets code, et que le programmeur n'a pas à connaître les motifs de bits de INADDR_ANY et INADDR_ALL, vous êtes déjà en difficulté. Habillage de 0 dans une macro qui renvoie 0, c'est le genre de mentalité qui est l'esclave des sens de la cohérence et de ne pas respecter les connaissances du domaine.

Maintien de sockets code, c'est plus que la compréhension de C. Si vous ne comprenez pas la différence entre INADDR_LOOPBACK et INADDR_ANY à un niveau compatible avec netstat de sortie, alors vous êtes dangereux dans ce code et ne devrait pas être le changer.

De la paille, l'homme des arguments proposés par Zwinck concernant l'inutile l'utilisation de htonl():

1. Il peut offenser expérimentés prise de programmeurs à utiliser htonl parce qu'ils savent qu'il ne fait rien (étant donné qu'ils connaissent la valeur de la constante par cœur).

C'est une paille argument parce que nous avons une représentation qui expérimentés socket programmeurs connaissent la valeur de la INADDR_ANY par cœur. C'est comme d'écrire que seul un expérimentés programmeur C connaît la valeur de NULL par cœur. L'écriture de "par cœur" donne l'impression que le nombre est légèrement difficile à mémoriser, peut-être un peu les chiffres, tels que 127.0.0.1. Mais non, nous sommes hyperbolically discuter de la difficile de mémoriser les habitudes nommée "zéro tous les bits" et "toutes les un bits."

Considérant que ces valeurs numériques apparaissent dans la sortie de, par exemple, netstat et d'autres utilitaires de système, et considérant également que certaines de ces valeurs apparaissent dans les en-têtes IP, il n'y a pas une telle chose comme un compétent sockets programmeur qui ne connaissent pas ces valeurs, que ce soit par le coeur ou par le cerveau. En fait, la tentative de sockets de programmation sans la connaissance de ces bases peut être dangereux pour la disponibilité du réseau.

1. Il nécessite moins de taper à l'omettre.

Cet argument est destiné à être absurde et dédaigneux, il n'est pas besoin de beaucoup de le réfuter.

1. Un faux "performance" d'optimisation (de toute évidence, elle n'a pas d'importance).

Il est difficile de savoir où cet argument vient. Il pourrait être une tentative de fourniture stupide-semblant arguments de l'opposition. En tout cas, pas à l'aide de la htonl() macro ne fait pas de différence de performances lorsque vous fournissez une constante et l'utilisation typique d'un compilateur C -- les expressions constantes sont réduits à une constante dans les deux cas.

---

Une raison de ne pas utiliser htonl() avec INADDR_ANY, c'est que les plus expérimentés sockets programmeur sait qu'il n'est pas nécessaire. Quoi de plus: les programmeurs qui ne savent pas besoin d'apprendre. Il n'y a aucun "coût" avec l'utilisation de htonl(), le problème est le coût de l'établissement d'une norme de codage qui favorise l'ignorance d'une telle critique des valeurs importantes.

Par définition, l'encapsulation favorise l'ignorance. Que très ignorance, c'est l'habitude de bénéficier de l'aide d'un encapsulé interface -- la connaissance est coûteux et limité et, par conséquent, l'encapsulation est généralement bonne. La question devient: quels efforts de programmation sont les mieux améliorée via l'encapsulation? Sont là des tâches de programmation qui sont disserved par encapsulation?

Il n'est pas techniquement incorrect d'utiliser htonl(), parce qu'il n'a pas d'effet sur cette valeur. Toutefois, les arguments que vous devrait utilisation, il peut être trompeuse.

Il y a ceux qui diront qu'une meilleure situation serait celui dans lequel le développeur n'a pas besoin de savoir que INADDR_ANY est tous les zéros et ainsi de suite. Cette terre de l'ignorance est le pire, pas mieux. Considérer que ces "la magie des valeurs" sont utilisés tout au long des différentes interfaces avec le protocole TCP/IP. Par exemple, lors de la configuration d'Apache, si vous souhaitez écouter que pour IPv4 (et pas IPv6), vous devez spécifier:

Listen 0.0.0.0:80

J'ai couru dans programmeurs qui, à tort, fourni l'adresse IP locale au lieu de INADDR_ANY (0.0.0.0) ci-dessus. Ces programmeurs ne sais pas ce que INADDR_ANY est, et ils ont probablement l'envelopper dans htonl() alors qu'ils sont à elle. C'est la terre de l'abstraction-la pensée et l'encapsulation.

Les idées de "l'encapsulation" et "abstraction" ont été largement acceptés et trop largement appliquées, mais elles ne s'appliquent pas toujours. Dans le domaine de l'adressage IPv4, il n'est pas approprié pour traiter ces valeurs de constantes comme "abstrait" -- ils sont convertis directement en morceaux sur le fil.

---

Mon point est ceci: il n'y a pas de "bonne" utilisation de INADDR_ANY avec htonl() -- les deux sont équivalents. Je ne recommande pas l'adoption d'une exigence que la valeur soit utilisée d'une manière particulière, parce que le INADDR_X de la famille de constantes seulement quatre membres, et un seul d'entre eux, INADDR_LOOPBACK a une valeur qui est différente en fonction de l'octet de commande. Il est préférable de savoir ce fait que d'établir un standard pour utiliser les valeurs qui transforme les "yeux" pour les modèles de bits des valeurs.

Dans beaucoup d'autres Api, il est utile pour les programmeurs de procéder sans connaître la valeur numérique ou modèles de bits de constantes utilisées par l'Api. Dans le cas de l'API sockets, ces modèles de bits et les valeurs sont utilisées comme entrées et affichées de façon omniprésente. Il est préférable de connaître ces valeurs numériquement que de passer du temps à réfléchir sur l'utilisation de htonl() sur eux.

Lors de la programmation en C, en particulier, la plupart des "utilisation" de l'API sockets consiste à passer de l'autre personne code source, et de l'adapter. C'est une autre raison, il est donc important de savoir ce que INADDR_ANY est avant de toucher à une ligne qui l'utilise.