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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]