Skip to content

Conversation

@agroce
Copy link
Contributor

@agroce agroce commented Jul 29, 2021

MarcoFalke and practicalswift here's an Eclipser guide, reconstructed from their documentation and my docker history getting it up and running. It might be good if someone confirmed it actually works for them in a fresh ubuntu 20.04.

@DrahtBot DrahtBot added the Docs label Jul 29, 2021
Copy link
Contributor

@practicalswift practicalswift left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested ACK d899d36a146822c0d6e90755f343ddc45904f84f modulo missing ./autogen.sh :)

Thanks for doing this: Eclipser is a very nice and interesting fuzzer, and the addition of a symbolic execution inspired fuzzer to the Bitcoin Core "fuzzing ensemble" is awesome :)

Recommended reading for other reviewers: "What Makes Eclipser Special?" (Trail of Bits) and the Eclipser paper.

This quote from the blog post referenced above describes why "the more the merrier" typically holds when it comes to choosing fuzzers:

The research literature and practical wisdom of fuzzer use repeatedly show that, even when a fuzzer is good enough to “beat” other fuzzers (and thus get a paper published at ICSE), it will always have instances where it performs worse than an “old,” “outdated” fuzzer. In fuzzing, diversity is not just helpful, it’s essential, if you really want the best chance to find every last bug. No fuzzer will be best for all programs under test, or for all bugs in a given real-world program.

@maflcko
Copy link
Member

maflcko commented Jul 30, 2021

Concept ACK will test after #22585 (comment)

@agroce
Copy link
Contributor Author

agroce commented Jul 30, 2021

What I'd really like is to get tool-based ensemble fuzzing up and running, but my experiments right now are showing the tools may be not-ready-for-prime-time. The DeepState ensembler (sort of) works right now, but porting doesn't seem like a good approach.

@jonatack
Copy link
Member

Concept ACK. I might test with Debian.

@practicalswift
Copy link
Contributor

Tested ACK 4f9784fc2d51843ebc05e81c31a508bee5b7e370

@maflcko
Copy link
Member

maflcko commented Aug 6, 2021

@agroce
Copy link
Contributor Author

agroce commented Aug 6, 2021

Will squash in the AM

@agroce
Copy link
Contributor Author

agroce commented Aug 6, 2021

squashed & ready to go

@practicalswift
Copy link
Contributor

ACK 6e1150e

@laanwj laanwj merged commit 29b030b into bitcoin:master Oct 1, 2021
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Oct 1, 2021
6e1150e fuzz: add guide to fuzzing with Eclipser v1.x (Alex Groce)

Pull request description:

  MarcoFalke and practicalswift here's an Eclipser guide, reconstructed from their documentation and my docker history getting it up and running.  It might be good if someone confirmed it actually works for them in a fresh ubuntu 20.04.

ACKs for top commit:
  practicalswift:
    ACK 6e1150e

Tree-SHA512: ca855932fd7a2c1d1005d572ab5fabc26f42d779f9baf279783f08a43dd72ec60f57239135d30c2a82781e593626fec2c96bb19fb91e1b777cef2d83a54eba35
@bitcoin bitcoin locked and limited conversation to collaborators Oct 30, 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.

6 participants