All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrange" <berrange@redhat.com>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] Let user set vnc password from command line or file
Date: Sun, 6 Jan 2008 16:44:42 +0000	[thread overview]
Message-ID: <20080106164442.GA21083@redhat.com> (raw)
In-Reply-To: <4780B166.6010701@tiscali.it>

On Sun, Jan 06, 2008 at 11:45:58AM +0100, Leandro Dardini wrote:
> Here is a little patch to let the user specify the vnc server password 
> in the command line (yes, it is visible with the ps command) or via an 
> external file (better choice). This adds two new arguments, 
> -vnc-password and -vnc-password-file. The file containing the vnc 
> password is in clear.

Providing a password which is clearly visible on the command line to all
users of a machine is unacceptable.  This capability was *explicitly* left 
out when I did the original VNC password support in QEMU.

Providing a password via a file is reasonable, but this should be made
more general than just VNC. QCow2 disks files can be encrypted and require
passwords too. So I think it'd be more useful to have a '-passwd-file file'
arg, and in that file have 1 line per (device, password) pair. 

eg Assuming use of VNC, and 2 qcow encrypted disks one might have:

      vnc: 123456
      hda: mysecret
      hdb: othersecret

Perhaps 'hda' should be the more general syntax used by the -drive param
instead, or even the qcow file name...

Regards,
Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 

      reply	other threads:[~2008-01-06 16:44 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-06 10:45 [Qemu-devel] [PATCH] Let user set vnc password from command line or file Leandro Dardini
2008-01-06 16:44 ` Daniel P. Berrange [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080106164442.GA21083@redhat.com \
    --to=berrange@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.