feat: add fuzzing harnesses #925
Conversation
|
This should work now and hopefully pass all tests, once this gets merged I can open a PR (which is ready locally) on the OSS-Fuzz side of things |
|
@facontidavide, I thought I had the pre-commit fixed, looks like I forgot to push the changes to the corpus files. Please re-run the workflow, it should be all good now :). Also, if you're about to merge this, drop a comment in google/oss-fuzz#13008 to drive this effort forward, thanks! |
|
@facontidavide kind reminder to get this merged, Google approved of the integration and its just a matter of getting this PR in now and you dropping an |
|
yes, this is on my radar |
|
Merged |
|
This honestly add too much noise to the current CMakeList.txt. |
|
Yeah sure thats actually what I should have done. I'll push something for that in a bit! |
|
thanks!!! |
This is an initial integration for BehaviorTree.CPP ([website](https://www.behaviortree.dev/), [repo](https://github.com/BehaviorTree/BehaviorTree.CPP)) a popular library that allows to e.g., structure the switching between different tasks in an autonomous agent, such as a robot or a virtual entity in a computer game. This PR can be reviewed but needs to wait until the BehaviorTree/BehaviorTree.CPP#925 PR is merged and the harnesses land upstream. Also, @facontidavide or @miccol either of you as a core-maintainer need to agree on this onboarding of the project here, so if you do, please just ACK this. --------- Co-authored-by: cktii <mytupre@gmail.com>
I still have a bunch of other crashes, esp. for the type-system and casting. I figured why not try to up-stream the fuzzing efforts to have it saved in a central place.
While we're at it, I'd like to add this project to OSS-Fuzz and would prepare an appropriate PR for this next. If it gets approved, it would mean continuous fuzzing/testing for BehaviorTree.CPP and bug reports would come your way without e.g. me having to file anything.
That said, to get that one merged over there, I will need your approval from one of the maintainers here. Eying @facontidavide or @miccol. What do you think? Would that be something you guys would be interested in?