[libvirt] [PATCH] qemu: honor allowDiskFormatProbing when parsing command line
Eric Blake
eblake at redhat.com
Mon Apr 29 16:16:44 UTC 2013
On 04/29/2013 07:56 AM, Ján Tomko wrote:
> My commit 024e9af broke this.
> ---
> src/qemu/qemu_command.c | 3 ---
> diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
> index d23bdfc..d161b56 100644
> --- a/src/qemu/qemu_command.c
> +++ b/src/qemu/qemu_command.c
> @@ -8540,7 +8540,6 @@ qemuParseCommandLineDisk(virDomainXMLOptionPtr xmlopt,
> def->bus = VIR_DOMAIN_DISK_BUS_IDE;
> def->device = VIR_DOMAIN_DISK_DEVICE_DISK;
> def->type = VIR_DOMAIN_DISK_TYPE_FILE;
> - def->format = VIR_STORAGE_FILE_AUTO;
>
> for (i = 0 ; i < nkeywords ; i++) {
> if (STREQ(keywords[i], "file")) {
> @@ -9788,8 +9787,6 @@ virDomainDefPtr qemuParseCommandLine(virCapsPtr qemuCaps,
> if (VIR_ALLOC(disk) < 0)
> goto no_memory;
>
> - disk->format = VIR_STORAGE_FILE_AUTO;
> -
ACK. Using VIR_STORAGE_FILE_AUTO would be a security hole if it
contradicts the user's configuration. Thanks for fixing this pre-release.
--
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/20130429/4c409d4d/attachment-0001.sig>
More information about the libvir-list
mailing list