]> git.proxmox.com Git - mirror_qemu.git/commit - util/oslib-posix.c
util: add socket_set_fast_reuse function which will replace setting SO_REUSEADDR
authorSebastian Ottlik <ottlik@fzi.de>
Wed, 2 Oct 2013 10:23:12 +0000 (12:23 +0200)
committerStefan Weil <sw@weilnetz.de>
Wed, 2 Oct 2013 17:20:31 +0000 (19:20 +0200)
commit606600a176c981addcfedb0698f13fd0f2f4446e
treeb9ddd847daa6eeb96e3de8b754a8ea14ab434214
parenta684f3cf9b9b9c3cb82be87aafc463de8974610c
util: add socket_set_fast_reuse function which will replace setting SO_REUSEADDR

If a socket is closed it remains in TIME_WAIT state for some time. On operating
systems using BSD sockets the endpoint of the socket may not be reused while in
this state unless SO_REUSEADDR was set on the socket. On windows on the other
hand the default behaviour is to allow reuse (i.e. identical to SO_REUSEADDR on
other operating systems) and setting SO_REUSEADDR on a socket allows it to be
bound to a endpoint even if the endpoint is already used by another socket
independently of the other sockets state. This can even result in undefined
behaviour.

Many sockets used by QEMU should not block the use of their endpoint after being
closed while they are still in TIME_WAIT state. Currently QEMU sets SO_REUSEADDR
for such sockets, which can lead to problems on Windows. This patch introduces
the function socket_set_fast_reuse that should be used instead of setting
SO_REUSEADDR when fast socket reuse is desired and behaves correctly on all
operating systems.

As a failure of this function can only be caused by bad QEMU internal errors, an
assertion handles these situations. The return value is still passed on, to
minimize changes in client code and prevent unused variable warnings if NDEBUG
is defined.

Signed-off-by: Sebastian Ottlik <ottlik@fzi.de>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Stefan Weil <sw@weilnetz.de>
include/qemu/sockets.h
util/oslib-posix.c
util/oslib-win32.c