From 283a7a7e607ee9743c69b1d79c9abe73db5e3c47 Mon Sep 17 00:00:00 2001 From: Remi Gacogne Date: Tue, 2 Apr 2024 12:33:43 +0200 Subject: [PATCH] ci: Enable LeakSanitizer during dnsdist and recursor unit tests We need to fix some one-time allocations in the authoritative server that are reported as leaked memory before we can enabled it there. See: - https://github.com/PowerDNS/pdns/pull/14028 - https://github.com/PowerDNS/pdns/pull/14029 There is also a leak in the remotebackend unit tests that I will investigate after https://github.com/PowerDNS/pdns/pull/13960 has been merged. --- .github/workflows/build-and-test-all.yml | 2 -- 1 file changed, 2 deletions(-) diff --git a/.github/workflows/build-and-test-all.yml b/.github/workflows/build-and-test-all.yml index ba0669bfb2..4cf2364687 100644 --- a/.github/workflows/build-and-test-all.yml +++ b/.github/workflows/build-and-test-all.yml @@ -122,7 +122,6 @@ jobs: container: image: ghcr.io/powerdns/base-pdns-ci-image/debian-12-pdns-base:master env: - ASAN_OPTIONS: detect_leaks=0 SANITIZERS: ${{ matrix.sanitizers }} UBSAN_OPTIONS: "print_stacktrace=1:halt_on_error=1:suppressions=${{ env.REPO_HOME }}/build-scripts/UBSan.supp" UNIT_TESTS: yes @@ -207,7 +206,6 @@ jobs: container: image: ghcr.io/powerdns/base-pdns-ci-image/debian-12-pdns-base:master env: - ASAN_OPTIONS: detect_leaks=0 SANITIZERS: ${{ matrix.sanitizers }} UBSAN_OPTIONS: "print_stacktrace=1:halt_on_error=1:suppressions=${{ env.REPO_HOME }}/build-scripts/UBSan.supp" UNIT_TESTS: yes -- 2.47.2