libcody: Do not use a dummy port number in getaddrinfo().
authorIain Sandoe <iain@sandoe.co.uk>
Sun, 13 Mar 2022 16:29:45 +0000 (16:29 +0000)
committerIain Sandoe <iain@sandoe.co.uk>
Fri, 18 Mar 2022 15:23:23 +0000 (15:23 +0000)
commit99fcd602a82e28e1d6a843e5cbe011b570dc81fa
tree910b447b760b47e8dbf2b1b44d37e16a9128bcf8
parent356e2720e9030927579024c2f060d665a0b9080f
libcody: Do not use a dummy port number in getaddrinfo().

the getaddrinfo() requires either a non-null name for the server or
a port service / number.  In the code that opens a connection we have
been calling this with a dummy port number of "0".  Unfortunately this
triggers a bug in some BSD versions and OSes importing that code.

In this part of the code we do not really need a port number, since it
is not reasonable to open a connection to an unspecified host.

Setting hints info field to 0, and the servname parm to nullptr works
around the BSD bug in this case.

Signed-off-by: Iain Sandoe <iain@sandoe.co.uk>
libcody/ChangeLog:

* netclient.cc (OpenInet6): Do not provide a dummy port number
in the getaddrinfo() call.
libcody/netclient.cc