diff options
author | fanquake <fanquake@gmail.com> | 2023-02-24 12:16:37 +0000 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2023-02-24 12:16:37 +0000 |
commit | 84ca5b349ecc2ad083bb39352e5d5ae731fb1622 (patch) | |
tree | 67757295e0f96efce66b17dc16b9edc57aef5f88 /doc | |
parent | 6758bd7de75d65d08f9f588378b92a56cc79a49e (diff) |
doc: mention sanitizer suppressions in developer docs
Should be enough to close #17834.
Diffstat (limited to 'doc')
-rw-r--r-- | doc/developer-notes.md | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/doc/developer-notes.md b/doc/developer-notes.md index e2e54e13d3..f15df1bf73 100644 --- a/doc/developer-notes.md +++ b/doc/developer-notes.md @@ -560,8 +560,19 @@ address sanitizer, libtsan for the thread sanitizer, and libubsan for the undefined sanitizer. If you are missing required libraries, the configure script will fail with a linker error when testing the sanitizer flags. -The test suite should pass cleanly with the `thread` and `undefined` sanitizers, -but there are a number of known problems when using the `address` sanitizer. The +The test suite should pass cleanly with the `thread` and `undefined` sanitizers. You +may need to use a suppressions file, see `test/sanitizer_suppressions`. They may be +used as follows: +```bash +export LSAN_OPTIONS="suppressions=$(pwd)/test/sanitizer_suppressions/lsan" +export TSAN_OPTIONS="suppressions=$(pwd)/test/sanitizer_suppressions/tsan:halt_on_error=1:second_deadlock_stack=1" +export UBSAN_OPTIONS="suppressions=$(pwd)/test/sanitizer_suppressions/ubsan:print_stacktrace=1:halt_on_error=1:report_error_type=1" +``` + +See the CI config for more examples, and upstream documentation for more information +about any additional options. + +There are a number of known problems when using the `address` sanitizer. The address sanitizer is known to fail in [sha256_sse4::Transform](/src/crypto/sha256_sse4.cpp) which makes it unusable unless you also use `--disable-asm` when running configure. We would like to fix |