fuzz: enable BDB in the MSAN CI jobs #24703

issue fanquake opened this issue on March 29, 2022
  1. fanquake commented at 9:15 AM on March 29, 2022: member

    Currently, our MSAN and MSAN fuzz CI jobs build depends without BDB (i.e NO_BDB=1). Ideally, these jobs would be building Bitcoin Core with BDB enabled (via depends). Previously BDB was being compiled in one of the jobs, but not actually used.

    Re-enabling BDB may require using newer (clang) tools; see #23008.

    Testing / debugging is possible by running the CI locally. i.e FILE_ENV="./ci/test/00_setup_env_native_msan.sh" ./ci/test_run_all.sh.

    If BDB is re-enabled in our MSAN CI, it may also be possible to use it in our OSS-FUZZ job, see: https://github.com/google/oss-fuzz/blob/1abbd8bf089660da96367b0370855f99e3aa2df6/projects/bitcoin-core/build.sh#L32.

    Useful skills:

    • CI
    • Sanitizers / Fuzzing

    Want to work on this issue?

    For guidance on contributing, please read CONTRIBUTING.md before opening your pull request.

  2. fanquake added the label Tests on Mar 29, 2022
  3. fanquake added the label good first issue on Mar 29, 2022
  4. MarcoFalke commented at 9:20 AM on March 29, 2022: member

    I don't think there are any bdb fuzz tests, so this will have no benefit for fuzzing. And given that bdb will be removed, this will also have limited benefit for the other stuff?

  5. fanquake commented at 9:22 AM on March 29, 2022: member

    And given that bdb will be removed, this will also have limited benefit for the other stuff?

    I'm less interested in the fuzzing, and more in the MSAN job. If we don't care that BDB isn't included in that build, because it's going away in the future, I'm happy to close this, and note why it's not enabled in the CI jobs themselves. Otherwise it is unclear why certain dependencies are enabled vs not.

  6. MarcoFalke commented at 9:24 AM on March 29, 2022: member

    Yeah, no objection to a comment why it is disabled.

  7. MarcoFalke referenced this in commit 565aac2e6f on Mar 29, 2022
  8. MarcoFalke closed this on Mar 29, 2022

  9. fanquake removed the label good first issue on May 18, 2022
  10. fanquake locked this on May 18, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-17 09:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me