]> git.ipfire.org Git - thirdparty/libvirt.git/commit
qemu: fix failure to start with spice graphics and no tls
authorLaine Stump <laine@laine.org>
Tue, 30 Apr 2013 18:04:59 +0000 (14:04 -0400)
committerLaine Stump <laine@laine.org>
Tue, 30 Apr 2013 22:20:53 +0000 (18:20 -0400)
commitf6966b627703e165d2f6821d6558d8000feee294
tree5073540d0ca7fe1e7c24bb5e5e62dca669a09a0f
parent7fecc8e36f9c1ea1010664c81c2622798144750c
qemu: fix failure to start with spice graphics and no tls

Commit eca3fdf inadvertantly caused a failure to start for any domain
with the following in its config:

    <graphics type='spice' autoport='yes'/>

The problem is that when tlsPort == 0 and defaultMode == "any" (which
is the default for defaultMode), this would be flagged in the code as
"needTLSPort", and if there was then no spice tls config, the new
error+fail would happen.

This patch checks for the case of defaultMode == "any", and in that
case simply doesn't allocate a TLS port (since that's probably not
what the user wanted, and it would have failed later anyway.). It does
leave the error in place for cases when the user specifically asked to
use tls in one way or another, though.
src/qemu/qemu_process.c