[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 10/10] qcow2: Don't ignore qcow2_alloc_clusters
From: |
Christoph Hellwig |
Subject: |
Re: [Qemu-devel] [PATCH 10/10] qcow2: Don't ignore qcow2_alloc_clusters return value |
Date: |
Tue, 19 Jan 2010 19:58:31 +0100 |
User-agent: |
Mutt/1.3.28i |
On Mon, Jan 18, 2010 at 01:11:36PM +0100, Kevin Wolf wrote:
> @@ -55,6 +55,9 @@ int qcow2_grow_l1_table(BlockDriverState *bs, int min_size)
>
> /* write new table (align to cluster) */
> new_l1_table_offset = qcow2_alloc_clusters(bs, new_l1_size2);
> + if (new_l1_table_offset < 0) {
> + return new_l1_table_offset;
> + }
I think the error return needs to free new_l1_table first.
- Re: [Qemu-devel] [PATCH 06/10] qcow2: Fix error handling in grow_refcount_table, (continued)
- [Qemu-devel] [PATCH 07/10] qcow2: Improve error handling in update_refcount, Kevin Wolf, 2010/01/18
- [Qemu-devel] [PATCH 05/10] block: Return original error codes in bdrv_pread/write, Kevin Wolf, 2010/01/18
- [Qemu-devel] [PATCH 08/10] qcow2: Allow updating no refcounts, Kevin Wolf, 2010/01/18
- [Qemu-devel] [PATCH 10/10] qcow2: Don't ignore qcow2_alloc_clusters return value, Kevin Wolf, 2010/01/18
- Re: [Qemu-devel] [PATCH 10/10] qcow2: Don't ignore qcow2_alloc_clusters return value,
Christoph Hellwig <=
- [Qemu-devel] [PATCH 09/10] qcow2: Don't ignore update_refcount return value, Kevin Wolf, 2010/01/18