Bug 222

Summary: configure finds getnameinfo() but not getaddrinfo()
Product: Portable OpenSSH Reporter: Robert Urban <urban>
Component: Build systemAssignee: OpenSSH Bugzilla mailing list <openssh-bugs>
Status: CLOSED FIXED    
Severity: major    
Priority: P2    
Version: -current   
Hardware: Alpha   
OS: OSF/1   
Attachments:
Description Flags
Fake HAVE_GETADDRINFO when [on]getaddrinfo exists none

Description Robert Urban 2002-04-18 12:26:43 AEST 
there are two versions of getaddrinfo() in libc, ogetaddrinfo() and
ngetaddrinfo().  This is allows a choice between two compatibility
modes.  Programmers must include <netdb.h> in order to suck
in the correct #definition for getaddrinfo().  In the configure test
program used to determine if getaddrinfo() is present, <netdb.h>
is not included, and configure thus concludes that getaddrinfo()
is missing, and #undefs HAVE_GETADDRINFO in config.h

However, configure does find getnameinfo(), so it #defines
HAVE_GETNAMEINFO.  The OpenSSH getaddrinfo() and
Tru64's getnameinfo() don't play well together.

It would help countless poor Tru64 souls avoid a traumatic
OpenSSH failure if you could address this problem!
Comment 1 Damien Miller 2002-04-18 23:07:24 AEST 
Could you please try CVS -current with the following patch. You will need to
regenerate configure using autoconf.
Comment 2 Damien Miller 2002-04-18 23:08:41 AEST 
Created attachment 83 [details]
Fake HAVE_GETADDRINFO when [on]getaddrinfo exists
Comment 3 Chris Adams 2002-04-19 00:37:52 AEST 
What compiler, CFLAGS, and configure options are you using to build
OpenSSH on Tru64?

The reason I ask is that I've used OpenSSH starting with (IIRC) 2.1 on
Tru64 4.0F, 4.0G, and now 5.1A, and I haven't had any trouble with what
you describe.  I've always compiled with "cc -std1", running configure
like:

CC="cc -std1" ./configure <options here>
Comment 4 Robert Urban 2002-04-19 10:28:20 AEST 
um...

I've tested the patch.  configure now finds [no]getaddrinfo(), but
HAVE_GETADDRINFO is still not getting set properly.  Here is an excerpt
from configure's output:

[...]
checking for getaddrinfo... no
checking for getcwd... yes
checking for getgrouplist... no
checking for getnameinfo... yes
[...]
checking for ngetaddrinfo... yes
checking for openpty... yes
checking for ogetaddrinfo... yes
[...]

from config.h:

/* Define to 1 if you have the `getaddrinfo' function. */
/* #undef HAVE_GETADDRINFO */

[...]

/* Define to 1 if you have the `getnameinfo' function. */
#define HAVE_GETNAMEINFO 1

[...]

Please let me know if I can do anything...
Comment 5 Damien Miller 2002-04-19 15:38:42 AEST 
That shouldn't be a problem - have a look at the defines.h hunk of the patch: we
define HAVE_GETADDRINFO if either of the replacement functions are found in libc.

Does the patch work?
Comment 6 Robert Urban 2002-04-19 23:58:37 AEST 
umm, sorry about that.  Was a bit thick of me.

The patch works!  Thanks for your time.
Comment 7 Damien Miller 2002-04-23 20:30:24 AEST 
Actually, you should wait until the fix is committed before marking a bug RESOLVED 

The fix has just been committed anyway :)
Comment 8 Damien Miller 2004-04-14 12:24:18 AEST 
Mass change of RESOLVED bugs to CLOSED