tests: Add fuzzing harness for CConnman #20188

pull practicalswift wants to merge 1 commits into bitcoin:master from practicalswift:fuzzers-connman changing 3 files +196 −0

practicalswift commented at 8:13 PM on October 19, 2020: contributor

Add fuzzing harness for CConnman.

See doc/fuzzing.md for information on how to fuzz Bitcoin Core. Don't forget to contribute any coverage increasing inputs you find to the Bitcoin Core fuzzing corpus repo.

Happy fuzzing :)
DrahtBot added the label Build system on Oct 19, 2020
tests: Add fuzzing harness for CConnman 79ef8324d4
practicalswift force-pushed on Oct 20, 2020
fanquake removed the label Build system on Oct 21, 2020
fanquake added the label Tests on Oct 21, 2020
MarcoFalke commented at 10:25 AM on November 10, 2020: member

Concept ACK, will review
MarcoFalke commented at 6:15 PM on November 11, 2020: member

review ACK 79ef8324d4c85ed16a304e98805724b8a59022ac
MarcoFalke merged this on Nov 12, 2020
MarcoFalke closed this on Nov 12, 2020
sidhujag referenced this in commit f5e1ff04a3 on Nov 12, 2020
ajtowns commented at 12:59 PM on November 17, 2020: member

This is now failing when run in CI under valgrind -- https://cirrus-ci.com/task/6536629458305024 ; seems that it's trying to use CConnman without calling ::Start(), which means nMaxOutboundTotalBytesSentInCycle and nMaxOutboundCycleStartTime are uninitialized.

This wasn't caught in this PRs CI run, because there weren't any test cases in qa-assets for it to run against.
practicalswift deleted the branch on Apr 10, 2021
kittywhiskers referenced this in commit 133754a1c7 on Aug 11, 2022
DrahtBot locked this on Aug 18, 2022

Contributors

Labels