[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH for-5.1 2/5] qom: Plug memory leak in "info qom-tree"
From: |
Thomas Huth |
Subject: |
Re: [PATCH for-5.1 2/5] qom: Plug memory leak in "info qom-tree" |
Date: |
Thu, 16 Jul 2020 11:05:44 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 |
On 15/07/2020 17.19, Li Qiang wrote:
> Markus Armbruster <armbru@redhat.com> 于2020年7月15日周三 上午12:05写道:
>>
>> Commit e8c9e65816 "qom: Make "info qom-tree" show children sorted"
>> created a memory leak, because I didn't realize
>> object_get_canonical_path_component()'s value needs to be freed.
>>
>> Reproducer:
>>
>> $ qemu-system-x86_64 -nodefaults -display none -S -monitor stdio
>> QEMU 5.0.50 monitor - type 'help' for more information
>> (qemu) info qom-tree
>>
>> This leaks some 4500 path components, 12-13 characters on average,
>> i.e. roughly 100kBytes depending on the allocator. A couple of
>> hundred "info qom-tree" here, a couple of hundred there, and soon
>> enough we're talking about real memory.
>>
>> Plug the leak.
>>
>> Fixes: e8c9e65816f5dbfe18ad3b2be938d0d8192d459a
>> Signed-off-by: Markus Armbruster <armbru@redhat.com>
Tested-by: Thomas Huth <thuth@redhat.com>
> Reviewed-by: Li Qiang <liq3ea@gmail.com>
>
> I have also send out this patch.
> I hope the maintainer will pick your patch.
Thanks a lot for your patch, too! Normally, I'd say "first come, first
serve" and suggest that we use your patch, but since Markus' patch
series has some more patches which would break due to the contextual
differences, I think it's slightly better this time to use Markus'
version of the patch.
Regards,
Thomas
- [PATCH for-5.1 0/5] qom: HMP command fixes, Markus Armbruster, 2020/07/14
- [PATCH for-5.1 4/5] qom: Document object_get_canonical_path() returns malloced string, Markus Armbruster, 2020/07/14
- [PATCH for-5.1 5/5] qom: Make info qom-tree sort children more efficiently, Markus Armbruster, 2020/07/14
- [PATCH for-5.1 2/5] qom: Plug memory leak in "info qom-tree", Markus Armbruster, 2020/07/14
- [PATCH for-5.1 1/5] qdev: Fix device_add DRIVER, help to print to monitor, Markus Armbruster, 2020/07/14
- [PATCH for-5.1 3/5] qom: Change object_get_canonical_path_component() not to malloc, Markus Armbruster, 2020/07/14