apr-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Trawick <traw...@attglobal.net>
Subject Re: cvs commit: apr/network_io/unix sa_common.c
Date Thu, 31 Jan 2002 16:00:43 GMT
martin@apache.org writes:

> martin      02/01/31 05:21:32
>   Modified:    network_io/unix sa_common.c
>   Log:
>   Be a bit more conservative regarding all those broken implementations out
>   there. Not everyone has FreeBSD or Linux.

broken?  whatever...  AFAICT, FreeBSD is "good" and Linux (or at least
some very popular levels of glibc) is "bad"

The unfortunate fact is that getnameinfo() and getaddrinfo() are in
general problematic right now when you look at the install base of
glibc, AIX, Solaris, et al.  Things like APR don't like the
getaddrinfo() on older AIX.  Things like lynx don't like the
getaddrinfo() on Solaris.  The informational RFC unfortunately
neglected to specify some of the behaviors that apps look for (and
sometimes don't find).  Spelling out how getaddrinfo() reports such
problems is just one example of that.

>   However, and because h_errno has been zeroed out before the call,
>   I added another test for nonzero h_errno. Everone happy?

It looks okay to me.

When there is some breathing room, I'll pursue some autodetection of
how getnameinfo() reports feedback so that clumsy run-time checks like
I had originally and like you restored are not necessary.

Jeff Trawick | trawick@attglobal.net | PGP public key at web site:
             Born in Roswell... married an alien...

View raw message