From 55a6216a5dc2d4a6d49f28c73fa4f0fc7a7b5e79 Mon Sep 17 00:00:00 2001 From: Marius Vlad Date: Tue, 30 May 2023 10:55:22 +0300 Subject: [PATCH] gitlab-ci/leak-sanitizer.supp: Suppress entire libfontconfig As we can't really use `cleanup_after_cairo()` in our headless backend, add more leak suppression entry, found while running the entire test suite. FcConfigSubstituteWithPat was already added (which seems to be main source) but found a couple of more while running the entire suite. The issue is that we need to set fast unwind to zero in order track those leaks but that would result into some massive runtime penalty and we get into the issue of timing out some of the tests. So rather than doing that, just add the entire libconfig library and be done with it. With it, this removes any pango/cairo former symbols as that seems to catch all other leaks as well. Signed-off-by: Marius Vlad --- .gitlab-ci/leak-sanitizer.supp | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/.gitlab-ci/leak-sanitizer.supp b/.gitlab-ci/leak-sanitizer.supp index a919405f..769827dd 100644 --- a/.gitlab-ci/leak-sanitizer.supp +++ b/.gitlab-ci/leak-sanitizer.supp @@ -1,11 +1,6 @@ # AddressSanitizer memory leak suppressions -# Cairo internal leaks from weston-keyboard -leak:cairo_select_font_face -leak:cairo_text_extents - -# Pango thread-global state (not destroyable?) -leak:pango_language_get_default - -# This leaks in Debian's fontconfig/Xwayland setup? -leak:FcConfigSubstituteWithPat +# This leaks in Debian's fontconfig/Xwayland setup. We add the entire +# fontconfig library because turning off fast unwind -- required to catch other +# originating leaks from fontconfig; would stall our tests timing them out. +leak:libfontconfig