[v2] Fix some memory leaks

[libvirt PATCH v2 0/5] Fix some memory leaks

Tim Wiederhake posted 5 patches 3 years ago

Diff against v1
Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20210419115415.19145-1-twiederh@redhat.com

src/conf/domain_conf.c   |  2 +-
src/libxl/xen_common.c   | 28 +++++++++++++++-------------
tools/virsh-checkpoint.c |  5 +++--
tools/virsh-snapshot.c   |  2 +-
4 files changed, 20 insertions(+), 17 deletions(-)

Expand all Fold all

[libvirt PATCH v2 0/5] Fix some memory leaks

Posted by Tim Wiederhake 3 years ago

Issues were found by llvm's asan and ubsan sanitizers.

V1: https://listman.redhat.com/archives/libvir-list/2021-April/msg00640.html

Changes since V1:
* Edited patch #1 to use two g_autofree variables instead of VIR_FREE'ing
  a reused variable
* Removed "virQEMUCapsSetHostModel: Fix memory leak", I will address this
  issue seperately
* Added fixes for two more memory leaks and one issue found by ubsan
  (passing NULL to qsort)

Tim Wiederhake (5):
  xenParseHypervisorFeatures: Fix memory leak
  virDomainFeaturesDefParse: Fix memory leak
  cmdCheckpointList: Fix memory leak
  cmdSnapshotList: Fix memory leak
  virshCheckpointListCollect: Do not pass NULL to qsort

 src/conf/domain_conf.c   |  2 +-
 src/libxl/xen_common.c   | 28 +++++++++++++++-------------
 tools/virsh-checkpoint.c |  5 +++--
 tools/virsh-snapshot.c   |  2 +-
 4 files changed, 20 insertions(+), 17 deletions(-)

-- 
2.26.3