Recent changes in servers/slapd/daemon.c break AIX (ITS#839)

[gombasg@inf.elte.hu]

Full_Name: Gabor Gombas
Version: 2.x-devel
OS: AIX 4.3.3.0
URL: http://valerie.inf.elte.hu/~gombasg/patches/openldap.2000-10-16
Submission from: (NULL) (157.181.161.4)


Hello,

Usage of getaddrinfo() in daemon.c breaks slapd on AIX. The problem is that
the AIX implementation of getaddrinfo() does not fill in the sa_len and
sa_family
fields of the returned sockaddr structure.

The provided patch solves the problem (and another syntax error too). But I do
not
think that getaddrinfo() is a good idea at the moment, because different OSes
implement
it in incompatible ways. So either
- OpenLDAP should provide it's own implementation of getaddrinfo() overriding
the
  native one, or
- not using getaddrinfo() at all until some standard defines it better than
RFC2133.

I'm having problems due to getaddrinfo() with Heimdal too, just because RFC2133
does not define it precisely and the AIX implementation makes different
assumptions
than Heimdal. So currently I consider getaddrinfo() being non-portable (even if
it's purpose would be quite the opposite).

Gabor