fuzz: Avoid timeout in EncodeBase58 #21922

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2105-fuzzRpcTimeout changing 1 files +3 −2

MarcoFalke commented at 7:27 PM on May 11, 2021: member

The complexity is O(N^2), so limit the size.

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

Oss-Fuzz testcase for rpc fuzzer: https://github.com/bitcoin/bitcoin/files/6461382/clusterfuzz-testcase-minimized-rpc-4831734974775296.log
fuzz: Avoid timeout in EncodeBase58 faa0d94a7d
MarcoFalke requested review from practicalswift on May 11, 2021
DrahtBot added the label Tests on May 11, 2021
practicalswift commented at 8:41 PM on May 11, 2021: contributor

cr ACK faa0d94a7d9cdd10e81ee231a7b06d4b14b37e13: patch looks correct
sipa commented at 9:04 PM on May 11, 2021: member

utACK faa0d94a7d9cdd10e81ee231a7b06d4b14b37e13
practicalswift approved
fanquake merged this on May 12, 2021
fanquake closed this on May 12, 2021
sidhujag referenced this in commit b90c01102f on May 12, 2021
MarcoFalke deleted the branch on May 12, 2021
MarcoFalke commented at 7:28 AM on May 12, 2021: member

See also #17511
gwillen referenced this in commit 2d5af8c458 on Jun 1, 2022
DrahtBot locked this on Aug 18, 2022

Contributors

Labels