fuzz: Limit max insertions in timedata fuzz test #21909

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2105-fuzzMedian changing 1 files +3 −1
  1. MarcoFalke commented at 6:58 AM on May 11, 2021: member

    It is debatable whether a size of the median filter other than 200 (the only size used in production) should be fuzzed. For now add a minimal patch to cap the max insertions. Otherwise the complexity is N^2 log(N), where N is the size of the fuzz input.

    Hopefully fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=34167

  2. fuzz: Limit max insertions in timedata fuzz test fa95555a49
  3. MarcoFalke requested review from practicalswift on May 11, 2021
  4. DrahtBot added the label Tests on May 11, 2021
  5. practicalswift commented at 12:12 PM on May 11, 2021: contributor

    cr ACK fa95555a491dc01952703f476836e607ac34eab4: patch looks correct

  6. practicalswift commented at 12:13 PM on May 11, 2021: contributor

    @MarcoFalke Don't forget to make the issues public before posting links to the OSS-Fuzz bug tracker, otherwise reviewers won't be able to view them :)

  7. MarcoFalke merged this on May 11, 2021
  8. MarcoFalke closed this on May 11, 2021

  9. MarcoFalke deleted the branch on May 11, 2021
  10. sidhujag referenced this in commit d6be2b7865 on May 11, 2021
  11. gwillen referenced this in commit 0c0d7cf29c on Jun 1, 2022
  12. PastaPastaPasta referenced this in commit c0821bb518 on Jul 17, 2022
  13. DrahtBot locked this on Aug 18, 2022


practicalswift

Labels

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-17 06:14 UTC

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