Skip to content

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Jul 14, 2021

Without a size limit on the input data, the runtime is unbounded. Fix this by picking an upper bound on the maximum number of fuzz operations.

Should fix https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=35981

@DrahtBot DrahtBot added the Tests label Jul 14, 2021
@maflcko maflcko changed the title fuzz: Speed up prevector fuzz target fuzz: Limit max ops in prevector fuzz target Jul 15, 2021
@practicalswift
Copy link
Contributor

cr ACK faafda2

@maflcko maflcko merged commit 2735e11 into bitcoin:master Jul 25, 2021
@maflcko maflcko deleted the 2107-fuzzPrev branch July 25, 2021 10:43
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Jul 28, 2021
faafda2 fuzz: Speed up prevector fuzz target (MarcoFalke)

Pull request description:

  Without a size limit on the input data, the runtime is unbounded. Fix this by picking an upper bound on the maximum number of fuzz operations.

  Should fix https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=35981

ACKs for top commit:
  practicalswift:
    cr ACK faafda2

Tree-SHA512: 1bf166c4a99a8ce88bdc030cd6a32ce1da5251b73873772e0e9c001ec2bacafebb183f7c8c88806d0ab633aada2cff8b78791f5c9c0c6f2cc8ef5f0875c4b2ef
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 17, 2022
faafda2 fuzz: Speed up prevector fuzz target (MarcoFalke)

Pull request description:

  Without a size limit on the input data, the runtime is unbounded. Fix this by picking an upper bound on the maximum number of fuzz operations.

  Should fix https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=35981

ACKs for top commit:
  practicalswift:
    cr ACK faafda2

Tree-SHA512: 1bf166c4a99a8ce88bdc030cd6a32ce1da5251b73873772e0e9c001ec2bacafebb183f7c8c88806d0ab633aada2cff8b78791f5c9c0c6f2cc8ef5f0875c4b2ef
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants