I'd guess that any bug should be discoverable within 10 ops. However, 900 seems also better than no limit at all, which causes timeouts such as https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=50892
fuzz: Avoid timeout in bitdeque fuzz target #26012
pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:2209-fuzz-timeout-🏺 changing 2 files +4 −5-
MarcoFalke commented at 1:40 PM on September 5, 2022: member
-
fuzz: Avoid timeout in bitdeque fuzz target fa642286b8
-
MarcoFalke commented at 1:44 PM on September 5, 2022: member
Flame graph on my machine for the oss-fuzz timeout:

- DrahtBot added the label Tests on Sep 5, 2022
-
fuzz: Remove no-op call to get() fa4ba04c15
- fanquake requested review from sipa on Sep 22, 2022
- fanquake requested review from sdaftuar on Sep 22, 2022
-
sipa commented at 1:49 PM on September 22, 2022: member
ACK fa4ba04c157b83b827f7541fa007710bd6211fe7
No need for the fuzzing loop to continue forever.
- fanquake merged this on Sep 22, 2022
- fanquake closed this on Sep 22, 2022
-
fanquake commented at 7:01 PM on September 22, 2022: member
@MarcoFalke are you going to backport this?
- sidhujag referenced this in commit ee9b22a362 on Sep 23, 2022
- MarcoFalke referenced this in commit 2a56cc1f58 on Sep 23, 2022
- MarcoFalke deleted the branch on Sep 23, 2022
- bitcoin locked this on Sep 23, 2023
Contributors
Labels