_dbus_connect_tcp_socket_with_nonce: don't create an extra fd (which is then leaked)
authorSimon McVittie <simon.mcvittie@collabora.co.uk>
Wed, 22 Jun 2011 10:59:32 +0000 (11:59 +0100)
committerSimon McVittie <simon.mcvittie@collabora.co.uk>
Mon, 18 Jul 2011 14:11:47 +0000 (15:11 +0100)
commitaf13ae7e43c444e1ff08d477b454bdee8cd9f892
treebafc87cc6f407cc3c197d1291c2bbb0db6b47322
parentc6c18741796ba99978878106ba915b3e0be225e4
_dbus_connect_tcp_socket_with_nonce: don't create an extra fd (which is then leaked)

This block should have been deleted in 2007, when IPv6 support was added:
previously, the fd allocated at the beginning of the function was used
for connect(), but for IPv6 support, the socket() call has to be inside
the loop over getaddrinfo() results so its address family can be changed.

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37258
Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
dbus/dbus-sysdeps-unix.c