[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/6] VMDK: add twoGbMaxExtentSparse support
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/6] VMDK: add twoGbMaxExtentSparse support |
Date: |
Thu, 4 Aug 2011 18:38:46 +0800 |
On Thu, Aug 4, 2011 at 6:34 PM, Stefan Hajnoczi <address@hidden> wrote:
> On Thu, Aug 4, 2011 at 4:09 AM, Fam Zheng <address@hidden> wrote:
>> +static void vmdk_free_last_extent(BlockDriverState *bs)
>> +{
>> + BDRVVmdkState *s = bs->opaque;
>> +
>> + if (s->num_extents == 0) {
>> + return;
>> + }
>> + s->num_extents--;
>> + s->extents = qemu_realloc(s->extents, s->num_extents *
>> sizeof(VmdkExtent));
>
> vmdk_free_extents() frees extent->l1_table, extent->l2_cache, and
> extent->l1_backup_table. Are they being leaked here?
No, it's only called after vmdk_init_tables fails, where no table is
actually allocated to the extent.
--
Best regards!
Fam Zheng
- [Qemu-devel] [PATCH v2 0/6] Add various VMDK subformats support, Fam Zheng, 2011/08/03
- [Qemu-devel] [PATCH v2 3/6] VMDK: separate vmdk_read_extent/vmdk_write_extent, Fam Zheng, 2011/08/03
- [Qemu-devel] [PATCH v2 4/6] VMDK: Opening compressed extent., Fam Zheng, 2011/08/03
- [Qemu-devel] [PATCH v2 6/6] VMDK: creating streamOptimized subformat, Fam Zheng, 2011/08/03
- [Qemu-devel] [PATCH v2 5/6] VMDK: read/write compressed extent, Fam Zheng, 2011/08/03