WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] ioemu: need probe backing file format

To: xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: Re: [Xen-devel] [PATCH] ioemu: need probe backing file format
From: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Date: Tue, 18 Aug 2009 17:23:28 +0100
Delivery-date: Tue, 18 Aug 2009 09:24:22 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <fac556d20908171942o458416a8je2f14f76e7cec285@xxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Newsgroups: chiark.mail.xen.devel
References: <fac556d20908171942o458416a8je2f14f76e7cec285@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Wei Kong writes ("[Xen-devel] [PATCH] ioemu: need probe backing file format"):
> Revert changeset 17646
> Due to qcow2 also has backing file.

I'm not sure what you mean exactly or what you are trying to achieve,
but guessing slightly:

Do you mean that you have a qcow2 image which has a backing file other
than in raw format (eg, another cow image) ?  If so that is sadly not
supported in Xen; the version of qemu we currently have has no way of
recording the backing file format and format probing is a security
vulnerability, as has been discussed several times.

> -        ret = bdrv_open2(bs->backing_hd, backing_filename, open_flags,
> &bdrv_raw);
> +        ret = bdrv_open(bs->backing_hd, backing_filename, open_flags,
> &bdrv_raw);

In any case this is wrong because bdrv_open has one fewer arguments
than bdrv_open2 ...

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread] Current Thread [Next in Thread>