qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] slirp: Port redirection option behave different


From: Stefan Weil
Subject: Re: [Qemu-devel] [PATCH] slirp: Port redirection option behave differently on Linux and Windows
Date: Tue, 13 Aug 2013 22:21:55 +0200
User-agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130803 Thunderbird/17.0.8

Am 13.08.2013 22:00, schrieb Taimoor:
> From: Taimoor Mirza <address@hidden>
>
> port redirection code uses SO_REUSEADDR socket option before binding to
> host port. Behavior of SO_REUSEADDR is different on Windows and Linux.
> Relaunching QEMU with same host and guest port redirection values on Linux
> throws error but on Windows it does not throw any error.
> Problem is discussed in 
> http://lists.gnu.org/archive/html/qemu-devel/2013-04/msg03089.html
>
> Signed-off-by: Taimoor Mirza <address@hidden>
> ---
>  slirp/socket.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/slirp/socket.c b/slirp/socket.c
> index 8e8819c..23780b3 100644
> --- a/slirp/socket.c
> +++ b/slirp/socket.c
> @@ -627,7 +627,9 @@ tcp_listen(Slirp *slirp, uint32_t haddr, u_int hport, 
> uint32_t laddr,
>       addr.sin_port = hport;
>  
>       if (((s = qemu_socket(AF_INET,SOCK_STREAM,0)) < 0) ||
> +#ifdef _WIN32

Shouldn't this be replaced by #ifndef _WIN32?

>           (qemu_setsockopt(s, SOL_SOCKET, SO_REUSEADDR, &opt, sizeof(int)) < 
> 0) ||
> +#endif
>           (bind(s,(struct sockaddr *)&addr, sizeof(addr)) < 0) ||
>           (listen(s,1) < 0)) {
>               int tmperrno = errno; /* Don't clobber the real reason we 
> failed */




reply via email to

[Prev in Thread] Current Thread [Next in Thread]