fuzz: fuzz max_ret_len for DecodeBase58/DecodeBase58Check #31577

pull brunoerg wants to merge 1 commits into bitcoin:master from brunoerg:2024-12-fuzz-base-encode-decode changing 1 files +6 −3
  1. brunoerg commented at 9:02 pm on December 27, 2024: contributor
    DecodeBase58 and DecodeBase58Check are fuzzed with a fixed value for max_ret_len (100). However, different values are used for this parameter depending on the usage (see key_io and unit tests).
  2. fuzz: fuzz `max_ret_len` for `DecodeBase58`/`DecodeBase58Check`
    The value of `max_ret_len` is hardcoded (100). However, different
    values are used for this parameter depending on the usage.
    28187098c9
  3. DrahtBot commented at 9:02 pm on December 27, 2024: 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/31577.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

  4. DrahtBot added the label Tests on Dec 27, 2024
  5. l0rinc commented at 9:12 pm on December 27, 2024: contributor
    Would it make sense to extend #30746 instead? Edit: extended a32002f (#30746) with this.
  6. brunoerg commented at 9:17 pm on December 27, 2024: contributor

    Would it make sense to extend #30746 instead?

    Feel free to cherry-pick this change there, then I close this one.

  7. l0rinc commented at 9:18 pm on December 27, 2024: contributor
    Moved the change over, added you as a co-author - thanks
  8. brunoerg closed this on Dec 27, 2024


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-01-21 06:12 UTC

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