Page MenuHomePhabricator

[contrib] Add Valgrind suppressions file
ClosedPublic

Authored by Fabien on Mon, Aug 26, 16:46.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Commits
rABCdca8f7210c89: [contrib] Add Valgrind suppressions file
Summary
Includes known Valgrind warnings in our dependencies that cannot be
fixed in-tree.

Example use:

$ valgrind --suppressions=contrib/valgrind.supp src/test/test_bitcoin
$ valgrind --suppressions=contrib/valgrind.supp --leak-check=full \
      --show-leak-kinds=all src/test/test_bitcoin --log_level=test_suite

Backport of core PR11035
https://github.com/bitcoin/bitcoin/pull/11035/files

Test Plan

Build with debug enabled.

valgrind --suppressions=contrib/valgrind.supp src/test/test_bitcoin

Result on my machine:

==24741== HEAP SUMMARY:
==24741==     in use at exit: 1,050,552 bytes in 15 blocks
==24741==   total heap usage: 47,594,831 allocs, 47,594,816 frees,
5,662,452,623 bytes allocated
==24741==
==24741== LEAK SUMMARY:
==24741==    definitely lost: 0 bytes in 0 blocks
==24741==    indirectly lost: 0 bytes in 0 blocks
==24741==      possibly lost: 0 bytes in 0 blocks
==24741==    still reachable: 1,049,592 bytes in 11 blocks
==24741==         suppressed: 960 bytes in 4 blocks

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Fabien created this revision.Mon, Aug 26, 16:46
Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Aug 26, 16:46
deadalnix accepted this revision.Tue, Aug 27, 13:23
This revision is now accepted and ready to land.Tue, Aug 27, 13:23
This revision was automatically updated to reflect the committed changes.