On Thu, May 8, 2014 at 6:08 AM, Jason McIntyre <[email protected]> wrote:

> CVSROOT:        /cvs
> Module name:    src
> Changes by:     [email protected]     2014/05/08 07:08:43
>
> Modified files:
>         usr.bin/telnet : telnet.1
>         share/man/man5 : hosts.5 networks.5
>         share/man/man4 : netintro.4
>         sbin/mountd    : exports.5
>         usr.sbin/arp   : arp.8
>
> Log message:
> inet(3) -> inet_net(3);


Hmm, I don't think this is correct.  inet_net(3) is for network specifiers
(e.g, 127/8) and not for host addresses; the default reference for stuff
that takes a host address should be inet_ntop(3).  However, some programs
continue to use the older inet_aton() routine for backwards compat and
should reference inet_addr(3).

So: telnet uses getaddrinfo(), which accepts the stricter format and should
reference inet_ntop(3).  mountd and arp on the other hand use inet_aton(),
as does the libc routine that parses /etc/networks, so networks(5),
exports(5), and arp(8) should reference inet_addr(3).


As for netintro(4), I would be inclined to not reference any of the section
3 inet* pages there, but rather include references to inet_ntop(3) and
inet_net(3) on the inet(4) page, but that's a matter of style instead of
correctness.

IMHO...


Philip Guenther

Reply via email to