doc: Add a brief overview of fuzzing/Bitcoin Core fuzzing #23482

pull agroce wants to merge 1 commits into bitcoin:master from agroce:patch-3 changing 1 files +4 −0
  1. agroce commented at 6:10 PM on November 10, 2021: contributor
    • Google's repo
    • Our report
    • John's advice on fuzz-friendly development
  2. Add a brief overview of fuzzing/Bitcoin Core fuzzing
    - Google's repo
    - Our report
    - John's advice on fuzz-friendly development
    6cac99ac07
  3. agroce commented at 6:10 PM on November 10, 2021: contributor
  4. DrahtBot added the label Docs on Nov 10, 2021
  5. laanwj added the label Tests on Nov 10, 2021
  6. MarcoFalke renamed this:
    Add a brief overview of fuzzing/Bitcoin Core fuzzing
    doc: Add a brief overview of fuzzing/Bitcoin Core fuzzing
    on Nov 10, 2021
  7. MarcoFalke commented at 6:18 PM on November 10, 2021: member

    ACK 6cac99ac0713b41df45dd5a0c4283d5ad25d2917

  8. in doc/fuzzing.md:24 in 6cac99ac07
      18 | @@ -19,6 +19,10 @@ $ FUZZ=process_message src/test/fuzz/fuzz
      19 |  There is also a runner script to execute all fuzz targets. Refer to
      20 |  `./test/fuzz/test_runner.py --help` for more details.
      21 |  
      22 | +## Overview of Bitcoin Core fuzzing
      23 | +
      24 | +[Google](https://github.com/google/fuzzing/) has a good overview of fuzzing in general, with contributions from key architects of some of the most-used fuzzers. [This paper](https://agroce.github.io/bitcoin_report.pdf) includes an external overview of the status of Bitcoin Core fuzzing, as of summer 2021.  [John Regehr](https://blog.regehr.org/archives/1687) provides good advice on writing code that assists fuzzers in finding bugs, which is useful for developers to keep in mind.
    


    laanwj commented at 7:04 PM on November 10, 2021:

    Might want to start with the link to the report, as that's the section title, move the general advice later.


    agroce commented at 7:23 PM on November 10, 2021:

    Might be a good idea; I went this way since the report assumes a good bit of fuzzing background, perhaps, and the Google link would be a better place to start from to understand it, but the ordering is a bit weird this way, maybe.

  9. brunoerg commented at 7:27 PM on November 10, 2021: member

    Concept ACK. I have been looking for docs to learn more about fuzzing, this is a good start.

  10. ghost commented at 8:03 PM on November 10, 2021: none

    Concept ACK

  11. shaavan approved
  12. shaavan commented at 12:22 PM on November 11, 2021: contributor

    ACK 6cac99ac0713b41df45dd5a0c4283d5ad25d2917

    The added documentation looks good and is quite helpful. Checked that the links are correct and direct to the right locations. There are also no grammatical mistakes in the added sentences (checked with Grammarly premium). Thanks for this great work, @agroce!

  13. MarcoFalke merged this on Nov 11, 2021
  14. MarcoFalke closed this on Nov 11, 2021

  15. sidhujag referenced this in commit 644581e487 on Nov 11, 2021
  16. DrahtBot locked this on Nov 11, 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-22 18:14 UTC

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