fuzz: fix timeout in crypter target #30575

pull brunoerg wants to merge 1 commits into bitcoin:master from brunoerg:2024-07-fuzz-crypter changing 1 files +17 −22
  1. brunoerg commented at 12:51 PM on August 2, 2024: contributor

    Fixes #30503

    • Move SetKeyFromPassphrase to out of LIMITED_WHILE
    • Remove SetKey calls since it is already called internally by other functions.
    • Reduce number of iterations (100 is enough, no need for 10,000).
  2. fuzz: fix timeout in crypter target
    Move `SetKeyFromPassphrase` to out of LIMITED_WHILE,
    remove `SetKey` calls since it is already called
    internally by other functions and reduce the number
    of iterations.
    bfd3c29e4f
  3. DrahtBot commented at 12:51 PM on August 2, 2024: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--006a51241073e994b41acfe9ec718e94-->

    Code Coverage

    For detailed information about the code coverage, see the test coverage report.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK maflcko, dergoegge

    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 Aug 2, 2024
  5. maflcko commented at 1:02 PM on August 2, 2024: member

    Didn't test, but

    review ACK bfd3c29e4f7942b49986ce0efa08481bae190b7e 📆

    <details><summary>Show signature</summary>

    Signature:

    untrusted comment: signature from minisign secret key on empty file; verify via: minisign -Vm "${path_to_any_empty_file}" -P RWTRmVTMeKV5noAMqVlsMugDDCyyTSbA3Re5AkUrhvLVln0tSaFWglOw -x "${path_to_this_whole_four_line_signature_blob}"
    RUTRmVTMeKV5npGrKx1nqXCw5zeVHdtdYURB/KlyA/LMFgpNCs+SkW9a8N95d+U4AP1RJMi+krxU1A3Yux4bpwZNLvVBKy0wLgM=
    trusted comment: review ACK bfd3c29e4f7942b49986ce0efa08481bae190b7e 📆
    8nW8HYI5E7hw31lTTDtksUirgPjUN7LIo0snPfoS5cgkBIZrLhwWzsbZGZ5NbO2i8xDslKzwpkZVTIQaQoiECQ==
    

    </details>

  6. maflcko added this to the milestone 28.0 on Aug 5, 2024
  7. dergoegge approved
  8. dergoegge commented at 12:45 PM on August 5, 2024: member

    utACK bfd3c29e4f7942b49986ce0efa08481bae190b7e

  9. fanquake merged this on Aug 5, 2024
  10. fanquake closed this on Aug 5, 2024

  11. bitcoin locked this on Aug 5, 2025

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-16 06:13 UTC

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