Page MenuHomePhabricator

[backport#18206] tests: Add fuzzing harness for bloom filter classes (CBloomFilter + CRollingBloomFilter)
ClosedPublic

Authored by majcosta on Aug 5 2020, 09:59.

Details

Summary

Add fuzzing harness for rolling bloom filter class CRollingBloomFilter (practicalswift)
Add fuzzing harness for bloom filter class CBloomFilter (practicalswift)

Add fuzzing harness for bloom filter classes (`CBloomFilter` + `CRollingBloomFilter`).

https://github.com/bitcoin/bitcoin/pull/18206/files


Backport of Core PR18206

Test Plan
cmake -GNinja .. -DENABLE_SANITIZERS="address;fuzzer" -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++
ninja bitcoin-fuzzers
./src/test/fuzz/bloom_filter
./src/test/fuzz/rolling_bloom_filter

../configure --enable-fuzz --with-sanitizers=fuzzer,address --disable-wallet --disable-bench --with-utils=no --with-daemon=no --with-libs=no --with-gui=no --with-seeder=no --enable-deprecated-build-system --without-miniupnpc --disable-zmq
make
./test/fuzz/test_runner.py -l DEBUG [path_to_corpus]

Diff Detail

Event Timeline

majcosta requested review of this revision.Aug 5 2020, 09:59
jasonbcox requested changes to this revision.Aug 5 2020, 22:22
jasonbcox added a subscriber: jasonbcox.
jasonbcox added inline comments.
src/Makefile.test.include
100 ↗(On Diff #22754)

This change is broken and autotools needs to be added to the test plan.

This revision now requires changes to proceed.Aug 5 2020, 22:22
jasonbcox requested changes to this revision.Aug 6 2020, 21:24
jasonbcox added inline comments.
src/Makefile.test.include
100

ordering is still wrong. this should come after test/fuzz/FuzzedDataProvider.h \

This revision now requires changes to proceed.Aug 6 2020, 21:24

actually fixed it this time

This revision is now accepted and ready to land.Aug 6 2020, 21:35
This revision was landed with ongoing or failed builds.Aug 6 2020, 21:36
This revision was automatically updated to reflect the committed changes.