Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Oct 3, 2020

Instead of the PP jungle, use a proper configure check

Fixes google/honggfuzz#336 (comment)

@practicalswift
Copy link
Contributor

Concept ACK

Thanks for fixing this!

@maflcko maflcko force-pushed the 2010-fuzzMainConfig branch from fab4827 to fae7a1c Compare October 4, 2020 15:51
@practicalswift
Copy link
Contributor

ACK fae7a1c

@maflcko maflcko merged commit b337bd7 into bitcoin:master Oct 8, 2020
@maflcko maflcko deleted the 2010-fuzzMainConfig branch October 8, 2020 11:39
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Oct 8, 2020
fae7a1c fuzz: Configure check for main function (MarcoFalke)

Pull request description:

  Instead of the PP jungle, use a proper configure check

  Fixes google/honggfuzz#336 (comment)

ACKs for top commit:
  practicalswift:
    ACK fae7a1c

Tree-SHA512: 2e55457d01f9ac598bb1e119d8b49dca55a28f88ec164cee6b5f071c29e9791f5a46cc8ee2b801b3a3faf906348da964ce32e7254da981c1104b9210a3508100
maflcko pushed a commit that referenced this pull request Feb 5, 2021
fac4be3 fuzz: Configure check for main function (take 2) (MarcoFalke)

Pull request description:

  Actually fix google/honggfuzz#336 (comment)

  Follow-up to #20065

  Steps to test: `honggfuzz` section in doc/fuzzing.md

ACKs for top commit:
  practicalswift:
    cr ACK fac4be3: patch looks correct!

Tree-SHA512: 893768c80439fe5d90b883ade89dc02f5bb80e27637916cf5575b6a9ed0b1c04942ff851342f5bbabb8666e6696715427feeb98f5301ad23c7b87b09e5872f97
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Feb 5, 2021
fac4be3 fuzz: Configure check for main function (take 2) (MarcoFalke)

Pull request description:

  Actually fix google/honggfuzz#336 (comment)

  Follow-up to bitcoin#20065

  Steps to test: `honggfuzz` section in doc/fuzzing.md

ACKs for top commit:
  practicalswift:
    cr ACK fac4be3: patch looks correct!

Tree-SHA512: 893768c80439fe5d90b883ade89dc02f5bb80e27637916cf5575b6a9ed0b1c04942ff851342f5bbabb8666e6696715427feeb98f5301ad23c7b87b09e5872f97
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error message "LLVMFuzzerTestOneInput is not defined" also when LLVMFuzzerTestOneInput is defined?

3 participants