qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH for-2.3 v2] qcow2: Fix header update with overri


From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH for-2.3 v2] qcow2: Fix header update with overridden backing file
Date: Tue, 07 Apr 2015 09:08:07 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0

On 04/07/2015 07:03 AM, Kevin Wolf wrote:
> In recent qemu versions, it is possible to override the backing file
> name and format that is stored in the image file with values given at
> runtime. In such cases, the temporary override could end up in the
> image header if the qcow2 header was updated, while obviously correct
> behaviour would be to leave the on-disk backing file path/format
> unchanged.
> 
> Fix this and add a test case for it.
> 
> Reported-by: Michael Tokarev <address@hidden>
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
> v2:
> - Now freeing instead of leaking the new strings in qcow2_close()
> 
>  block/qcow2.c              | 29 ++++++++++----
>  block/qcow2.h              |  6 +++
>  tests/qemu-iotests/130     | 95 
> ++++++++++++++++++++++++++++++++++++++++++++++
>  tests/qemu-iotests/130.out | 43 +++++++++++++++++++++
>  tests/qemu-iotests/group   |  1 +
>  5 files changed, 167 insertions(+), 7 deletions(-)
>  create mode 100755 tests/qemu-iotests/130
>  create mode 100644 tests/qemu-iotests/130.out

Reviewed-by: Eric Blake <address@hidden>

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]