Commit 1ccbc285 authored by Anthony PERARD's avatar Anthony PERARD Committed by malc

qemu-sockets: Fix parsing of the inet option 'to'.

Having a qemu command line argument like "-vnc 127.0.0.1:0,to=99" is broken.
This have been break with commit 879e45c7.
Signed-off-by: default avatarAnthony PERARD <anthony.perard@citrix.com>
Signed-off-by: default avatarStefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: default avatarmalc <av1474@comtv.ru>
parent ecf51c9a
......@@ -529,8 +529,9 @@ static InetSocketAddress *inet_parse(const char *str, Error **errp)
optstr = str + pos;
h = strstr(optstr, ",to=");
if (h) {
if (1 != sscanf(str, "%d%n", &to, &pos) ||
(str[pos] != '\0' && str[pos] != ',')) {
h += 4;
if (sscanf(h, "%d%n", &to, &pos) != 1 ||
(h[pos] != '\0' && h[pos] != ',')) {
error_setg(errp, "error parsing to= argument");
goto fail;
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment