Skip to content

Commit

Permalink
Generally disable leak sanitizer in tests
Browse files Browse the repository at this point in the history
Our tests have the unfortunate tendency to not clean up their resources;
as a consequence the leaksan check creates ton of (non-actionable)
warnings.
Our main focus for now are catching non-leak related issues via ASAN
(e.g. use-after-free).

The special casing for the cmake API tests stays in place in case we
want to re-enable leaksan checks for most tests again.

Task-number: QTQAINFRA-5315
Change-Id: I3e50ebf06ebd373f0ce0a4bb790ec8d38c344515
Reviewed-by: Qt CI Bot <[email protected]>
Reviewed-by: Dimitrios Apostolou <[email protected]>
Reviewed-by: Tor Arne Vestbø <[email protected]>
  • Loading branch information
Inkane committed Nov 30, 2022
1 parent 41c5178 commit ae7985b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion coin/instructions/cmake_run_ctest_enforce_exit_code.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ instructions:
instructions:
- type: EnvironmentVariable
variableName: ASAN_OPTIONS
variableValue: "malloc_context_size=100"
variableValue: "malloc_context_size=100:detect_leaks=0"
- type: EnvironmentVariable
# Override qt-testrunner as we don't want to gather test statistics
# because many tests FAIL when built with ASAN.
Expand Down

0 comments on commit ae7985b

Please sign in to comment.