[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/3] qcow2: Don't put invalid L2 table into cach
From: |
Benoît Canet |
Subject: |
Re: [Qemu-devel] [PATCH 1/3] qcow2: Don't put invalid L2 table into cache |
Date: |
Wed, 25 Sep 2013 16:47:59 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Le Wednesday 25 Sep 2013 à 16:37:18 (+0200), Max Reitz a écrit :
> In l2_allocate, the fail path is executed if qcow2_cache_flush fails.
> However, the L2 table has not yet been fetched from the L2 table cache.
> The qcow2_cache_put in the fail path therefore basically gives an
> undefined argument as the L2 table address (in this case).
>
> Signed-off-by: Max Reitz <address@hidden>
> ---
> block/qcow2-cluster.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/block/qcow2-cluster.c b/block/qcow2-cluster.c
> index 738ff73..f6d47c9 100644
> --- a/block/qcow2-cluster.c
> +++ b/block/qcow2-cluster.c
> @@ -188,7 +188,7 @@ static int l2_allocate(BlockDriverState *bs, int
> l1_index, uint64_t **table)
> {
> BDRVQcowState *s = bs->opaque;
> uint64_t old_l2_offset;
> - uint64_t *l2_table;
> + uint64_t *l2_table = NULL;
> int64_t l2_offset;
> int ret;
>
> @@ -265,7 +265,9 @@ static int l2_allocate(BlockDriverState *bs, int
> l1_index, uint64_t **table)
>
> fail:
> trace_qcow2_l2_allocate_done(bs, l1_index, ret);
> - qcow2_cache_put(bs, s->l2_table_cache, (void**) table);
> + if (l2_table != NULL) {
> + qcow2_cache_put(bs, s->l2_table_cache, (void**) table);
> + }
> s->l1_table[l1_index] = old_l2_offset;
> return ret;
> }
> --
> 1.8.3.1
>
>
Reviewed-by: Benoit Canet <address@hidden>