fuzz: Add support for fuzzing #1716
Conversation
|
I admit that I'm not sure what I've broken with the CI. I don't think I touched anything to do with the rp2040 configs. I'll just double-check in case there is something I missed. |
|
Ok, I don't think I've missed anything, perhaps it's a flake? |
silvergasp
force-pushed
the
master
branch
6 times, most recently
from
115ce67
to
8e390b3
Compare
|
I think I fixed the ci flake for the rp2040 in #1731. |
silvergasp
force-pushed
the
master
branch
9 times, most recently
from
779061d
to
cbefcb3
Compare
Sorry for super later response, sometime CI just break due to network/bandwidth issue since I have tried to run too many concurrent jobs (each for 1 supported mcu). It happens now and then, and often is fixed by re-run the failed workflow. I guess #1731 isn't related, sorry for not be able to response to this question of your in time. I am learning fuzzing and basically try to catch up with your PR and making a good progress. The more I learn about fuzzing, the more I want to integrate it (i.e merging this PR). Thank you very much for making such a brilliant PR. Hopefully we could merge this soon enough (for real this time). |
|
Not a problem, feel free to ask any questions here |
Adds support for fuzzing with basic cdc fuzzer.
There was a problem hiding this comment.
superb !! thank you very much for your time and effort (as well as your patient). Fuzzing is a realy useful/helpful testing for detecting memory overflow/leak issues. Which help to improve tinyusb a lot in the long run. And it indeed catch an overflow issue with tusb fifo. Thank you very much for introducing and adding harness.
PS: I have move fuzz into test/fuzz and integrate build fuzzer into pre-commit since it is quick to run on linux host, so it kind of make sense to bundle them together to save concurrent ci instances.
|
Not a problem, glad I could help. I'll put up a few more fuzz harnesses for the remaining device classes shortly. |
thanks again :) |
Adds support for fuzzing with basic a cdc fuzzer.
Describe the PR
This fuzz harness adds the necessary build infrastructure for fuzzing tinyusb. As a proof of concept there is a cdc fuzz harness written in c++. The intention of this PR is to improve security and robustness of the tinyusb stack by tracking down bugs and performance problems using fuzz testing.