diff options
author | Pan Nengyuan <pannengyuan@huawei.com> | 2020-02-27 09:29:50 +0800 |
---|---|---|
committer | Max Reitz <mreitz@redhat.com> | 2020-03-11 12:42:30 +0100 |
commit | fc124ea1dbfac2a1d5b851596831c9da746b7b66 (patch) | |
tree | 4d4a219616e234bd68fa70722e7ba1d7f54d52e1 /qemu-img.c | |
parent | 4aebf0f0da4a57204f568deed14661cb37b4ac30 (diff) | |
download | qemu-fc124ea1dbfac2a1d5b851596831c9da746b7b66.zip qemu-fc124ea1dbfac2a1d5b851596831c9da746b7b66.tar.gz qemu-fc124ea1dbfac2a1d5b851596831c9da746b7b66.tar.bz2 |
qemu-img: free memory before re-assign
collect_image_check() is called twice in img_check(), the filename/format will be alloced without free the original memory.
It is not a big deal since the process will exit anyway, but seems like a clean code and it will remove the warning spotted by asan.
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Pan Nengyuan <pannengyuan@huawei.com>
Message-Id: <20200227012950.12256-3-pannengyuan@huawei.com>
Signed-off-by: Max Reitz <mreitz@redhat.com>
Diffstat (limited to 'qemu-img.c')
-rw-r--r-- | qemu-img.c | 2 |
1 files changed, 2 insertions, 0 deletions
@@ -817,6 +817,8 @@ static int img_check(int argc, char **argv) check->corruptions_fixed); } + qapi_free_ImageCheck(check); + check = g_new0(ImageCheck, 1); ret = collect_image_check(bs, check, filename, fmt, 0); check->leaks_fixed = leaks_fixed; |