[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 0/7] Misc qcow2 corruption checks
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [PATCH v2 0/7] Misc qcow2 corruption checks |
Date: |
Fri, 3 Nov 2017 17:37:23 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 |
On 2017-11-03 15:18, Alberto Garcia wrote:
> This series contains a few checks that prevent QEMU from crashing
> under some scenarios with corrupted qcow2 images.
>
> The first patch solves the crash reported here:
>
> https://bugs.launchpad.net/qemu/+bug/1728615
>
> And the others solve similar crashes that I detected in the process of
> fixing this one.
>
> Regards,
>
> Berto
>
> v2:
> - Use goto fail in the l2_allocate() check [Max]
> - Add check and test case for allocation of compressed clusters [Max]
> - Add test case for header.refcount_table_offset == 0
> - Add overlap checks to qcow2_crypto_hdr_init_func() [Max]
>
> v1: https://lists.gnu.org/archive/html/qemu-block/2017-11/msg00010.html
> - Initial version
>
> Output of backport-diff against v1:
>
> Key:
> [----] : patches are identical
> [####] : number of functional differences between upstream/downstream patch
> [down] : patch is downstream-only
> The flags [FC] indicate (F)unctional and (C)ontextual differences,
> respectively
>
> 001/7:[----] [--] 'qcow2: Prevent allocating refcount blocks at offset 0'
> 002/7:[0003] [FC] 'qcow2: Prevent allocating L2 tables at offset 0'
> 003/7:[down] 'qcow2: Prevent allocating compressed clusters at offset 0'
> 004/7:[----] [-C] 'qcow2: Don't open images with
> header.refcount_table_clusters == 0'
> 005/7:[down] 'qcow2: Add iotest for an image with
> header.refcount_table_offset == 0'
> 006/7:[----] [--] 'qcow2: Add iotest for an empty refcount table'
> 007/7:[down] 'qcow2: Assert that the crypto header does not overlap other
> metadata'
>
> Alberto Garcia (7):
> qcow2: Prevent allocating refcount blocks at offset 0
> qcow2: Prevent allocating L2 tables at offset 0
> qcow2: Prevent allocating compressed clusters at offset 0
> qcow2: Don't open images with header.refcount_table_clusters == 0
> qcow2: Add iotest for an image with header.refcount_table_offset == 0
> qcow2: Add iotest for an empty refcount table
> qcow2: Assert that the crypto header does not overlap other metadata
>
> block/qcow2-cluster.c | 8 ++++++++
> block/qcow2-refcount.c | 15 +++++++++++++-
> block/qcow2.c | 7 +++++++
> tests/qemu-iotests/060 | 49
> ++++++++++++++++++++++++++++++++++++++++++++++
> tests/qemu-iotests/060.out | 39 ++++++++++++++++++++++++++++++++++++
> 5 files changed, 117 insertions(+), 1 deletion(-)
Thanks, applied to my block branch:
https://github.com/XanClic/qemu/commits/block
Max
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-block] [PATCH v2 3/7] qcow2: Prevent allocating compressed clusters at offset 0, (continued)
[Qemu-block] [PATCH v2 2/7] qcow2: Prevent allocating L2 tables at offset 0, Alberto Garcia, 2017/11/03
[Qemu-block] [PATCH v2 6/7] qcow2: Add iotest for an empty refcount table, Alberto Garcia, 2017/11/03
[Qemu-block] [PATCH v2 1/7] qcow2: Prevent allocating refcount blocks at offset 0, Alberto Garcia, 2017/11/03
[Qemu-block] [PATCH v2 5/7] qcow2: Add iotest for an image with header.refcount_table_offset == 0, Alberto Garcia, 2017/11/03
[Qemu-block] [PATCH v2 4/7] qcow2: Don't open images with header.refcount_table_clusters == 0, Alberto Garcia, 2017/11/03
Re: [Qemu-block] [PATCH v2 0/7] Misc qcow2 corruption checks,
Max Reitz <=