All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-trivial] [PATCH] slirp: Port redirection option behave differently on Linux and Windows
@ 2013-08-13 20:00 ` Taimoor
  0 siblings, 0 replies; 14+ messages in thread
From: Taimoor @ 2013-08-13 20:00 UTC (permalink / raw)
  To: qemu-devel; +Cc: qemu-trivial, Taimoor Mirza

From: Taimoor Mirza <tmirza@codesourcery.com>

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 <tmirza@codesourcery.com>
---
 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
 	    (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 */
-- 
1.7.5.4



^ permalink raw reply related	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2013-08-14 16:08 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-08-13 20:00 [Qemu-trivial] [PATCH] slirp: Port redirection option behave differently on Linux and Windows Taimoor
2013-08-13 20:00 ` [Qemu-devel] " Taimoor
2013-08-13 20:21 ` [Qemu-trivial] " Stefan Weil
2013-08-13 20:21   ` Stefan Weil
2013-08-13 21:02   ` [Qemu-trivial] " Taimoor Mirza
2013-08-13 21:02     ` Taimoor Mirza
2013-08-13 21:05 ` [Qemu-trivial] " Taimoor
2013-08-13 21:05   ` [Qemu-devel] " Taimoor
2013-08-13 21:20   ` [Qemu-trivial] " Eric Blake
2013-08-13 21:20     ` Eric Blake
2013-08-14 15:49     ` [Qemu-trivial] " Taimoor Mirza
2013-08-14 15:49       ` Taimoor Mirza
2013-08-14 16:08       ` [Qemu-trivial] " Eric Blake
2013-08-14 16:08         ` Eric Blake

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.