[libvirt] [PATCH] qemu: fix failure to start with spice graphics and no tls
Eric Blake
eblake at redhat.com
Tue Apr 30 19:57:13 UTC 2013
On 04/30/2013 12:13 PM, Laine Stump wrote:
> Commit eca3fdf inadvertantly caused a failure to start for any domain
s/inadvertantly/inadvertently/
> 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 | 41 ++++++++++++++++++++++++++---------------
> 1 file changed, 26 insertions(+), 15 deletions(-)
>
ACK.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 621 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130430/344c2ff7/attachment-0001.sig>
More information about the libvir-list
mailing list