[libvirt] [PATCH] qemu: fix domxml-to-native failing when spice_tls is not enabled

Ján Tomko jtomko at redhat.com
Wed Jul 9 09:08:45 UTC 2014


On 06/27/2014 04:37 PM, Jincheng Miao wrote:
> The default graphics channel mode is 'any', so as to defaultMode attribute.
> If defaultMode and channel mode are all the default value 'any',
> qemuConnectDomainXMLToNative will set TLSPort.
> But in qemuBuildGraphicsSPICECommandLine, if spice_tls is not enabled, libvirtd
> will report an error to tell the user that spice TLS is disabled in qemu.conf.
> 
> So qemuConnectDomainXMLToNative should check spice_tls is enabled,
> then decide to allocate an tlsPort number to this graphics.
> 
> If user specified defaultMode is 'secure', qemuConnectDomainXMLToNative
> could allocate tlsPort, and then let qemuBuildGraphicsSPICECommandLine reports
> the spice_tls disabled error.
> 
> The related bug is:
> https://bugzilla.redhat.com/show_bug.cgi?id=1113868
> 
> Signed-off-by: Jincheng Miao <jmiao at redhat.com>
> ---
>  src/qemu/qemu_driver.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 

ACK, I will push this later.

Jan


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140709/dcde588d/attachment-0001.sig>


More information about the libvir-list mailing list