diff options
| -rwxr-xr-x | tools/valgrind/build-valgrind-for-chromium.sh | 4 | ||||
| -rw-r--r-- | tools/valgrind/leak.patch | 18 |
2 files changed, 22 insertions, 0 deletions
diff --git a/tools/valgrind/build-valgrind-for-chromium.sh b/tools/valgrind/build-valgrind-for-chromium.sh index 652fe0b..a1fef40 100755 --- a/tools/valgrind/build-valgrind-for-chromium.sh +++ b/tools/valgrind/build-valgrind-for-chromium.sh @@ -43,6 +43,10 @@ patch -p0 < "$THISDIR"/fork.patch #wget -O longlines.patch "https://bugs.kde.org/attachment.cgi?id=35174" patch -p0 < "$THISDIR"/longlines.patch +# Work around bug http://bugs.kde.org/186790 +# "Suppression counts do not include leak suppressions" +patch -p0 < "$THISDIR"/leak.patch + # Add feature bug https://bugs.kde.org/show_bug.cgi?id=201170 # "Want --show-possible option so I can ignore the bazillion possible leaks..." #wget -O possible.patch https://bugs.kde.org/attachment.cgi?id=35559 diff --git a/tools/valgrind/leak.patch b/tools/valgrind/leak.patch new file mode 100644 index 0000000..5526f11 --- /dev/null +++ b/tools/valgrind/leak.patch @@ -0,0 +1,18 @@ +Index: coregrind/m_main.c +=================================================================== +--- coregrind/m_main.c (revision 10461) ++++ coregrind/m_main.c (working copy) +@@ -2175,11 +2186,11 @@ + if (VG_(clo_track_fds)) + VG_(show_open_fds)(); + ++ VG_TDICT_CALL(tool_fini, 0/*exitcode*/); ++ + if (VG_(needs).core_errors || VG_(needs).tool_errors) + VG_(show_all_errors)(); + +- VG_TDICT_CALL(tool_fini, 0/*exitcode*/); +- + if (VG_(clo_xml)) { + VG_(message)(Vg_UserMsg, ""); + VG_(message)(Vg_UserMsg, "</valgrindoutput>"); |