Add Murch’s fuzz seed in January 2024 #164

pull murchandamus wants to merge 1 commits into bitcoin-core:main from murchandamus:2024-01-murch changing 20060 files +30737 −0
  1. murchandamus commented at 12:34 pm on January 3, 2024: contributor
  2. maflcko commented at 12:37 pm on January 3, 2024: contributor
    lgtm, see also #163 :)
  3. murchandamus commented at 12:42 pm on January 3, 2024: contributor

    Nice. I did mine just with the regular merge script:

    Run mini_miner with args […]/qa-assets/src/test/fuzz/fuzz -rss_limit_mb=8000 -set_cover_merge=1 -shuffle=0 -prefer_small=1 -use_value_profile=0 ../qa-assets/fuzz_seed_corpus/mini_miner ../qa-assets-active-fuzzing/fuzz_seed_corpus/mini_miner

  4. dergoegge commented at 2:17 pm on January 3, 2024: member
    Could you redo the merge on top of #163? (to avoid duplicates)
  5. murchandamus commented at 2:43 pm on January 3, 2024: contributor
    Sure, gimme a few hours.
  6. Add fuzz seeds 70728884c6
  7. murchandamus force-pushed on Jan 3, 2024
  8. murchandamus commented at 6:44 pm on January 3, 2024: contributor

    I rebased on #163 and used @maflcko’s approach:

    0mv ../qa-assets/fuzz_seed_corpus /tmp/fuzz_inputs_upstream
    1./test/fuzz/test_runner.py --par $( nproc ) -l DEBUG ../qa-assets/fuzz_seed_corpus/ --m_dir ../qa-assets-active-fuzzing/fuzz_seed_corpus/ -x wallet_notifications -x utxo_total_supply --m_dir /tmp/upstream_fuzz_seed_corpus -x wallet_notifications -x utxo_total_supply
    2git restore -- ./fuzz_seed_corpus
    3git add ./fuzz_seed_corpus
    4git commit -S -m "Add fuzz seeds"
    

    The test_runner script then used -set_cover_merge=1 -shuffle=0 -prefer_small=1 -use_value_profile=0 under the hood.

  9. maflcko commented at 7:45 am on January 4, 2024: contributor
    nit: The -x in your command only needs to be specified once. In fact, I doubt it works, given that it should be a comma separated list, according to --help?
  10. maflcko commented at 7:47 am on January 4, 2024: contributor
    I guess there is no need to exclude wallet_notifications anymore, since there shouldn’t be any timeouts anymore. Only utxo_total_supply (https://github.com/bitcoin-core/qa-assets/issues/158) times out for you, right? So the command accidentally did the right thing, even though it was written not entirely correct.
  11. maflcko commented at 7:48 am on January 4, 2024: contributor
    lgtm
  12. dergoegge merged this on Jan 4, 2024
  13. dergoegge closed this on Jan 4, 2024


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/qa-assets. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-10-30 01:25 UTC

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