fuzz: crypter timeout #30503

issue maflcko openend this issue on July 23, 2024
  1. maflcko commented at 10:22 am on July 23, 2024: member

    Found by oss-fuzz

    clusterfuzz-testcase-crypter-4690510954954752.bin.not.txt

    0FUZZ=crypter perf record -g --call-graph dwarf  ./src/test/fuzz/fuzz  ./clusterfuzz-testcase-crypter-4690510954954752
    1... half a minute ...
    2hotspot ./perf.data
    

    Screenshot from 2024-07-23 12-16-34

  2. maflcko added the label Tests on Jul 23, 2024
  3. maflcko commented at 10:23 am on July 23, 2024: member
    My recommendation would be to limit the number of iterations to something reasonable, but still enough to cover all cases.
  4. brunoerg commented at 2:48 pm on July 23, 2024: contributor

    My recommendation would be to limit the number of iterations to something reasonable, but still enough to cover all cases.

    Agreed, I just reproduced it and it executes 9176 ones. No need for this number.

    edit: besides executing 9176, it seems to call only SetKeyFromPassphrase which is an expensive function.

  5. fanquake closed this on Aug 5, 2024

  6. fanquake referenced this in commit d15d95c5cc on Aug 5, 2024


maflcko brunoerg

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: 2024-09-08 01:12 UTC

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