mbox series

[0/6] qtest: Fix some memory issues

Message ID 20241209204427.17763-1-farosas@suse.de (mailing list archive)
Headers show
Series qtest: Fix some memory issues | expand

Message

Fabiano Rosas Dec. 9, 2024, 8:44 p.m. UTC
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(-)

Comments

Fabiano Rosas Dec. 17, 2024, 6:13 p.m. UTC | #1
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.