fuzz: Remove unused TimeoutExpired catch in fuzz runner #32388

pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2504-fuzz-timeout changing 1 files +15 −18
  1. maflcko commented at 5:57 pm on April 30, 2025: member

    Currently, the way to check for libFuzzer is to search the stderr of the fuzz executable when passed -help=1 for the string libFuzzer. See also https://github.com/bitcoin/bitcoin/blob/14b8dfb2bd5e2ca2b7c0c9a7f7d50e1e60adf75c/contrib/devtools/deterministic-fuzz-coverage/src/main.rs#L90-L101

    The python test runner additionally includes a timeout catch, which was needed before the plain read_file fallback was implemented, see https://github.com/bitcoin/bitcoin/blob/14b8dfb2bd5e2ca2b7c0c9a7f7d50e1e60adf75c/src/test/fuzz/fuzz.cpp#L251.

    However, it is no longer needed and the printed error message would be wrong, so remove it.

    (side-note: On Windows the fuzz executable seems to time out when an assert is hit in a debug build, see #32341 (comment). However, no one is running fuzz debug on Windows. Also, the newly added debug logging is a preferable replacement in this case anyway.)

  2. fuzz: Remove unused TimeoutExpired catch in fuzz runner
    Can be reviewed with --ignore-all-space
    fa4804009c
  3. DrahtBot commented at 5:57 pm on April 30, 2025: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32388.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK marcofleon

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

  4. DrahtBot added the label Tests on Apr 30, 2025
  5. fanquake requested review from dergoegge on May 1, 2025
  6. fanquake requested review from marcofleon on May 1, 2025
  7. marcofleon commented at 4:36 pm on May 2, 2025: contributor
    crACK fa4804009ceba96926edd7f55ea22442ebdc665d


maflcko DrahtBot marcofleon


dergoegge

Labels
Tests


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: 2025-05-05 12:12 UTC

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