Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(216)

Unified Diff: tools/valgrind.supp

Issue 488343003: Update valgrind suppression file for new FontConfig code. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/valgrind.supp
diff --git a/tools/valgrind.supp b/tools/valgrind.supp
index 40a9b19ea33564f7ca0d9079a6269d5e181ffbdc..1f3e6d9ede8cba949503aaa9be7ba9dc6afe0ade 100644
--- a/tools/valgrind.supp
+++ b/tools/valgrind.supp
@@ -82,28 +82,22 @@
obj:/usr/lib*/libGL.so*
}
{
+ #Fixed by FontConfig 2.9.0
+ #http://cgit.freedesktop.org/fontconfig/commit/?id=1c475d5c8cb265ac939d6b9e097666e300162511
font_config_bug_1
Memcheck:Addr4
fun:FcConfigFileExists
}
{
+ #https://bugs.freedesktop.org/show_bug.cgi?id=8215
+ #https://bugs.freedesktop.org/show_bug.cgi?id=8428
+ #FcPattern uses 'intptr_t elts_offset' instead of 'FcPatternEltPtr elts',
+ #which confuses valgrind.
font_config_bug_2
Memcheck:Leak
- fun:malloc
- fun:FcFontSetCreate
-}
-{
- font_config_bug_3
- Memcheck:Leak
- fun:realloc
- fun:FcFontSetAdd
-}
-{
- font_config_bug_4
- Memcheck:Leak
- fun:malloc
+ fun:*alloc
fun:FcPatternObjectInsertElt
- fun:FcPatternObjectAddWithBinding
+ fun:Fc*Add*
}
{
zlib_bug_1
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698