qtest: Fix some memory issues

[PATCH 0/6] qtest: Fix some memory issues

Posted by Fabiano Rosas 4 months ago

I'm trying to get the qtests to run with asan/valgrind without any
warnings so I can run a test before every PR. Here are a few fixes.

CI run: https://gitlab.com/farosas/qemu/-/pipelines/1580782506

Fabiano Rosas (6):
  tests/qtest/migration: Do proper cleanup in the dirty_limit test
  tests/qtest/migration: Initialize buffer in probe_o_direct_support
  tests/qtest/bios-tables-test: Free tables at dump_aml_files
  tests/qtest/virtio-iommu-test: Don't pass uninitialized data into
    qtest_memwrite
  tests/qtest/qos-test: Plug a couple of leaks
  tests/qtest/test-x86-cpuid-compat: Free allocated memory

 tests/qtest/bios-tables-test.c      |  1 +
 tests/qtest/migration-helpers.c     |  1 +
 tests/qtest/migration-test.c        |  5 +++++
 tests/qtest/qos-test.c              | 35 ++++++++++++++++++++---------
 tests/qtest/test-x86-cpuid-compat.c |  4 ++++
 tests/qtest/virtio-iommu-test.c     |  4 ++--
 6 files changed, 38 insertions(+), 12 deletions(-)

-- 
2.35.3

Re: [PATCH 0/6] qtest: Fix some memory issues

Posted by Fabiano Rosas 3 months, 3 weeks ago

Fabiano Rosas <farosas@suse.de> writes:

> I'm trying to get the qtests to run with asan/valgrind without any
> warnings so I can run a test before every PR. Here are a few fixes.
>
> CI run: https://gitlab.com/farosas/qemu/-/pipelines/1580782506
>
> Fabiano Rosas (6):
>   tests/qtest/migration: Do proper cleanup in the dirty_limit test
>   tests/qtest/migration: Initialize buffer in probe_o_direct_support
>   tests/qtest/bios-tables-test: Free tables at dump_aml_files
>   tests/qtest/virtio-iommu-test: Don't pass uninitialized data into
>     qtest_memwrite
>   tests/qtest/qos-test: Plug a couple of leaks
>   tests/qtest/test-x86-cpuid-compat: Free allocated memory
>
>  tests/qtest/bios-tables-test.c      |  1 +
>  tests/qtest/migration-helpers.c     |  1 +
>  tests/qtest/migration-test.c        |  5 +++++
>  tests/qtest/qos-test.c              | 35 ++++++++++++++++++++---------
>  tests/qtest/test-x86-cpuid-compat.c |  4 ++++
>  tests/qtest/virtio-iommu-test.c     |  4 ++--
>  6 files changed, 38 insertions(+), 12 deletions(-)

Queuing 1-4 for qtest-next.