[libvirt] PATCH: Support VNC password for QEMU guests
Jim Meyering
jim at meyering.net
Wed Jan 28 18:46:34 UTC 2009
"Daniel P. Berrange" <berrange at redhat.com> wrote:
> On Tue, Jan 20, 2009 at 11:08:56PM +0000, Daniel P. Berrange wrote:
>> This patch adds support for using the monitor interface to set the VNC
>> password
>>
>> (qemu) change vnc password
>> Password: ********
>>
>> A minor tricky thing is that we can't just send the command and password
>> all in one go, we must wait for the 'Password' prompt before sending the
>> password.
>>
>> When doing this I noticed that virsh dumpxml has no way to request a
>> secure XML dump (required to see the password element), nor did the
>> virsh edit command set the SECURE or INACTIVE flags when changing
>> the XML.
>>
>> qemu_conf.c | 45 ++++++++++++-----------
>> qemu_driver.c | 112 ++++++++++++++++++++++++++++++++++++++++++++--------------
>> virsh.c | 30 ++++++++++-----
>> 3 files changed, 131 insertions(+), 56 deletions(-)
...
> + int flags = 0;
> + int inactive = vshCommandOptBool(cmd, "inactive");
> + int secure = vshCommandOptBool(cmd, "secure");
> +
> + if (inactive)
> + flags |= VIR_DOMAIN_XML_INACTIVE;
> + if(secure)
> + flags |= VIR_DOMAIN_XML_SECURE;
ACK.
My only reservation is that this new --secure option currently means
"also dump sensitive info" (passwords), which is sometimes
_in_secure. So how about naming it --all instead?
More information about the libvir-list
mailing list