[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Don't require encryption password for 'qemu-img
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH] Don't require encryption password for 'qemu-img info' command |
Date: |
Mon, 10 Sep 2012 11:52:07 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Sep 10, 2012 at 12:44:52PM +0200, Kevin Wolf wrote:
> Am 04.09.2012 16:23, schrieb Kevin Wolf:
> > Am 31.08.2012 19:30, schrieb Eric Blake:
> >> On 08/31/2012 10:26 AM, Daniel P. Berrange wrote:
> >>> From: "Daniel P. Berrange" <address@hidden>
> >>>
> >>> The encryption password is only required if I/O is going to be
> >>> performed on a disk image. The 'qemu-img info' command merely
> >>> reports metadata, so it should not ask for a decryption password
> >>>
> >>> Signed-off-by: Daniel P. Berrange <address@hidden>
> >>> ---
> >>> qemu-img.c | 21 +++++++++++----------
> >>> 1 file changed, 11 insertions(+), 10 deletions(-)
> >>>
> >>> diff --git a/qemu-img.c b/qemu-img.c
> >>> index b41e670..3dd4ec7 100644
> >>> --- a/qemu-img.c
> >>> +++ b/qemu-img.c
> >>> @@ -221,7 +221,8 @@ static int print_block_option_help(const char
> >>> *filename, const char *fmt)
> >>>
> >>> static BlockDriverState *bdrv_new_open(const char *filename,
> >>> const char *fmt,
> >>> - int flags)
> >>> + int flags,
> >>> + int requireIO)
> >>
> >> Since you only pass 0 or 1 for requireIO, why not make it bool?
> >
> > Yes, please. bool require_io, to be precise. The qemu coding style
> > doesn't use camelCase here.
> >
> >> For
> >> that matter, why not make it part of 'flags', where the default is to
> >> require decryption, and a new flag BDV_O_NO_IO can be added when opening
> >> an image for no I/O and used to avoid the decryption?
> >
> > Because the block layer isn't really interested in it, this should be
> > kept inside qemu-img.
>
> Oh, Dan, seems you've been dropped from the CC list somehow. Are you
> going to send a v2?
Yes, I'll send an update soon.
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|