[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 6/6] vnc: distiguish between ipv4/ipv6 omitted vs set
From: |
Gerd Hoffmann |
Subject: |
[Qemu-devel] [PULL 6/6] vnc: distiguish between ipv4/ipv6 omitted vs set to off |
Date: |
Wed, 20 Jan 2016 07:53:47 +0100 |
From: "Daniel P. Berrange" <address@hidden>
The VNC code for interpreting QemuOpts does not currently
distinguish between ipv4/ipv6 being omitted, and being
set to 'off', because historically the 'ipv4' and 'ipv6'
options were just flags which did not accept a value.
The upshot is that if someone runs
$QEMU -vnc localhost:1,ipv6=off
QEMU still uses PF_UNSPEC and thus may still bind to IPv6,
when it should use PF_INET.
This is another instance of the problem previously fixed
for chardevs in
commit b77e7c8e99f9ac726c4eaa2fc3461fd886017dc0
Author: Paolo Bonzini <address@hidden>
Date: Mon Oct 12 15:35:16 2015 +0200
qemu-sockets: fix conversion of ipv4/ipv6 JSON to QemuOpts
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Daniel P. Berrange <address@hidden>
Message-id: address@hidden
Signed-off-by: Gerd Hoffmann <address@hidden>
---
ui/vnc.c | 18 ++++++++++++------
1 file changed, 12 insertions(+), 6 deletions(-)
diff --git a/ui/vnc.c b/ui/vnc.c
index 54673eb..4c303ce 100644
--- a/ui/vnc.c
+++ b/ui/vnc.c
@@ -3501,8 +3501,10 @@ void vnc_display_open(const char *id, Error **errp)
const char *websocket = qemu_opt_get(opts, "websocket");
int to = qemu_opt_get_number(opts, "to", 0);
- bool has_ipv4 = qemu_opt_get_bool(opts, "ipv4", false);
- bool has_ipv6 = qemu_opt_get_bool(opts, "ipv6", false);
+ bool has_ipv4 = qemu_opt_get(opts, "ipv4");
+ bool has_ipv6 = qemu_opt_get(opts, "ipv6");
+ bool ipv4 = qemu_opt_get_bool(opts, "ipv4", false);
+ bool ipv6 = qemu_opt_get_bool(opts, "ipv6", false);
saddr = g_new0(SocketAddress, 1);
if (websocket) {
@@ -3550,8 +3552,10 @@ void vnc_display_open(const char *id, Error **errp)
saddr->u.inet->has_to = true;
saddr->u.inet->to = to + 5900;
}
- saddr->u.inet->ipv4 = saddr->u.inet->has_ipv4 = has_ipv4;
- saddr->u.inet->ipv6 = saddr->u.inet->has_ipv6 = has_ipv6;
+ saddr->u.inet->ipv4 = ipv4;
+ saddr->u.inet->has_ipv4 = has_ipv4;
+ saddr->u.inet->ipv6 = ipv6;
+ saddr->u.inet->has_ipv6 = has_ipv6;
if (vs->ws_enabled) {
wsaddr->type = SOCKET_ADDRESS_KIND_INET;
@@ -3563,8 +3567,10 @@ void vnc_display_open(const char *id, Error **errp)
wsaddr->u.inet->has_to = true;
wsaddr->u.inet->to = to;
}
- wsaddr->u.inet->ipv4 = wsaddr->u.inet->has_ipv4 = has_ipv4;
- wsaddr->u.inet->ipv6 = wsaddr->u.inet->has_ipv6 = has_ipv6;
+ wsaddr->u.inet->ipv4 = ipv4;
+ wsaddr->u.inet->has_ipv4 = has_ipv4;
+ wsaddr->u.inet->ipv6 = ipv6;
+ wsaddr->u.inet->has_ipv6 = has_ipv6;
}
}
} else {
--
1.8.3.1
- [Qemu-devel] [PULL 0/6] Convert qemu-socket to use QAPI exclusively, update MAINTAINERS, Gerd Hoffmann, 2016/01/20
- [Qemu-devel] [PULL 1/6] add MAINTAINERS entry for qemu socket code, Gerd Hoffmann, 2016/01/20
- [Qemu-devel] [PULL 6/6] vnc: distiguish between ipv4/ipv6 omitted vs set to off,
Gerd Hoffmann <=
- [Qemu-devel] [PULL 2/6] sockets: remove use of QemuOpts from header file, Gerd Hoffmann, 2016/01/20
- [Qemu-devel] [PULL 3/6] sockets: remove use of QemuOpts from socket_listen, Gerd Hoffmann, 2016/01/20
- [Qemu-devel] [PULL 4/6] sockets: remove use of QemuOpts from socket_connect, Gerd Hoffmann, 2016/01/20
- [Qemu-devel] [PULL 5/6] sockets: remove use of QemuOpts from socket_dgram, Gerd Hoffmann, 2016/01/20
- Re: [Qemu-devel] [PULL 0/6] Convert qemu-socket to use QAPI exclusively, update MAINTAINERS, Peter Maydell, 2016/01/21