arm (and others): fix some memory leaks

[PATCH v2 1/9] scripts: Move lsan_suppressions.txt out of oss-fuzz subdir

Posted by Peter Maydell 1 month, 1 week ago

The oss-fuzz code uses an lsan_suppressions file to suppress certain
leak-sanitizer cases that are known issues or not our code's bug.
This is useful more widely than just for the fuzzer harness: if you
want to build QEMU with the leak sanitizer enabled and run 'make
check' then you will want to suppress some bogus leak reports.

Move the file up a directory.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
v2: just move the file so we keep git log --follow happy;
fix up reference in tests/docker/test-fuzz
---
 .gitlab-ci.d/buildtest.yml                   | 2 +-
 scripts/{oss-fuzz => }/lsan_suppressions.txt | 0
 tests/docker/test-fuzz                       | 2 +-
 3 files changed, 2 insertions(+), 2 deletions(-)
 rename scripts/{oss-fuzz => }/lsan_suppressions.txt (100%)

diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml
index 1b656b9eb0..7548057c54 100644
--- a/.gitlab-ci.d/buildtest.yml
+++ b/.gitlab-ci.d/buildtest.yml
@@ -626,7 +626,7 @@ build-oss-fuzz:
     IMAGE: fedora
   script:
     - mkdir build-oss-fuzz
-    - export LSAN_OPTIONS=suppressions=scripts/oss-fuzz/lsan_suppressions.txt
+    - export LSAN_OPTIONS=suppressions=scripts/lsan_suppressions.txt
     - CC="clang" CXX="clang++" CFLAGS="-fsanitize=address"
       ./scripts/oss-fuzz/build.sh
     - export ASAN_OPTIONS="fast_unwind_on_malloc=0"
diff --git a/scripts/oss-fuzz/lsan_suppressions.txt b/scripts/lsan_suppressions.txt
similarity index 100%
rename from scripts/oss-fuzz/lsan_suppressions.txt
rename to scripts/lsan_suppressions.txt
diff --git a/tests/docker/test-fuzz b/tests/docker/test-fuzz
index 7e506ae1f6..d2bdc8afba 100755
--- a/tests/docker/test-fuzz
+++ b/tests/docker/test-fuzz
@@ -18,7 +18,7 @@ cd "$BUILD_DIR"
 cp -a $QEMU_SRC .
 cd src
 mkdir build-oss-fuzz
-export LSAN_OPTIONS=suppressions=scripts/oss-fuzz/lsan_suppressions.txt
+export LSAN_OPTIONS=suppressions=scripts/lsan_suppressions.txt
 env CC="clang" CXX="clang++" CFLAGS="-fsanitize=address" ./scripts/oss-fuzz/build.sh
 export ASAN_OPTIONS="fast_unwind_on_malloc=0"
 for fuzzer in $(find ./build-oss-fuzz/DEST_DIR/ -executable -type f | grep -v slirp); do
-- 
2.43.0

Re: [PATCH v2 1/9] scripts: Move lsan_suppressions.txt out of oss-fuzz subdir

Posted by Yodel Eldar 1 month, 1 week ago

On 02/03/2026 03:22, Peter Maydell wrote:
> The oss-fuzz code uses an lsan_suppressions file to suppress certain
> leak-sanitizer cases that are known issues or not our code's bug.
> This is useful more widely than just for the fuzzer harness: if you
> want to build QEMU with the leak sanitizer enabled and run 'make
> check' then you will want to suppress some bogus leak reports.
> 
> Move the file up a directory.
> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
> v2: just move the file so we keep git log --follow happy;
> fix up reference in tests/docker/test-fuzz
> ---
>   .gitlab-ci.d/buildtest.yml                   | 2 +-
>   scripts/{oss-fuzz => }/lsan_suppressions.txt | 0
>   tests/docker/test-fuzz                       | 2 +-
>   3 files changed, 2 insertions(+), 2 deletions(-)
>   rename scripts/{oss-fuzz => }/lsan_suppressions.txt (100%)

Reviewed-by: Yodel Eldar <yodel.eldar@yodel.dev>

Re: [PATCH v2 1/9] scripts: Move lsan_suppressions.txt out of oss-fuzz subdir

Posted by Thomas Huth 1 month, 1 week ago

On 02/03/2026 10.22, Peter Maydell wrote:
> The oss-fuzz code uses an lsan_suppressions file to suppress certain
> leak-sanitizer cases that are known issues or not our code's bug.
> This is useful more widely than just for the fuzzer harness: if you
> want to build QEMU with the leak sanitizer enabled and run 'make
> check' then you will want to suppress some bogus leak reports.
> 
> Move the file up a directory.
> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
> v2: just move the file so we keep git log --follow happy;
> fix up reference in tests/docker/test-fuzz
> ---
>   .gitlab-ci.d/buildtest.yml                   | 2 +-
>   scripts/{oss-fuzz => }/lsan_suppressions.txt | 0
>   tests/docker/test-fuzz                       | 2 +-
>   3 files changed, 2 insertions(+), 2 deletions(-)
>   rename scripts/{oss-fuzz => }/lsan_suppressions.txt (100%)

Reviewed-by: Thomas Huth <thuth@redhat.com>