[libvirt] [PATCH v3] storage: Avoid memory leak on metadata fetching

Michal Privoznik mprivozn at redhat.com
Thu Jul 14 14:42:36 UTC 2011


On 14.07.2011 16:28, Eric Blake wrote:
> On 07/14/2011 08:15 AM, Michal Privoznik wrote:
>> Getting metadata on storage allocates a memory (path) which need to
>> be freed after use otherwise it gets leaked. This means after use of
>> virStorageFileGetMetadataFromFD or virStorageFileGetMetadata one
>> must call virStorageFileFreeMetadata to free it. This function frees
>> structure internals and structure itself.
>> ---
>> diff to v2:
>> -Jrika's&  Eric's review taken in
>>
>> diff to v1:
>> -Eric's review suggestions taken in
>>
>>   cfg.mk                           |    1 +
>>   src/conf/domain_conf.c           |   18 +++++++++---
>>   src/libvirt_private.syms         |    1 +
>>   src/storage/storage_backend_fs.c |   54 +++++++++++++++++++++----------------
>>   src/util/storage_file.c          |   18 ++++++++++++
>>   src/util/storage_file.h          |    2 +
>>   6 files changed, 66 insertions(+), 28 deletions(-)
>
> ACK - looks like you got all my points addressed.
>
Thanks, pushed.

Michal




More information about the libvir-list mailing list