[libvirt] [PATCH] qemu: fix failure to start with spice graphics and no tls

[Eric Blake]

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>