- Notifications
You must be signed in to change notification settings - Fork778
feat: add fuzzing harnesses#925
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
cktii commentedFeb 4, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
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 ingoogle/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 |
cktii commentedFeb 25, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Merged |
This honestly add too much noise to the currentCMakeList.txt. |
Yeah sure thats actually what I should have done. I'll push something for that in a bit! |
14589e5 intoBehaviorTree:masterUh oh!
There was an error while loading.Please reload this page.
thanks!!! |
This is an initial integration for BehaviorTree.CPP([website](https://www.behaviortree.dev/),[repo](https://github.com/BehaviorTree/BehaviorTree.CPP)) a popularlibrary that allows to e.g., structure the switching between differenttasks in an autonomous agent, such as a robot or a virtual entity in acomputer game.This PR can be reviewed but needs to wait until theBehaviorTree/BehaviorTree.CPP#925 PR is mergedand the harnesses land upstream.Also,@facontidavide or@miccol either of you as a core-maintainer needto agree on this onboarding of the project here, so if you do, pleasejust 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 toOSS-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?