Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initial fuzz_target.c #1273

Open
wants to merge 3 commits into
base: grass8
Choose a base branch
from
Open

Initial fuzz_target.c #1273

wants to merge 3 commits into from

Conversation

Shivam7-1
Copy link

@Shivam7-1 Shivam7-1 commented Dec 25, 2024

Hii @neteler @echoix
This is initial fuzz support file if it get merged here so next I will make PR in ossfuzz Repo to include other file like yaml,dockerfile etc
#1267 (comment)

Further Team can also Check the OSS-Fuzz documentation and Bug Disclosure Guidelines

Fuzz/fuzz_target.c Outdated Show resolved Hide resolved
Fuzz/fuzz_target.c Outdated Show resolved Hide resolved
Shivam7-1 and others added 2 commits December 25, 2024 18:20
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Member

@echoix echoix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I spot checked that test_calling_all_functions() exists. I myself have no experience with fuzzing.

Since this is the grass-addons repo, not the main repo, work isn't expected to be fully reviewed to the same level to allow more contributions, so if this is enough to get yourself started, I'm glad to let you explore this for us!

@Shivam7-1
Copy link
Author

Hii @echoix Thanks For Reviewing :)
So if possible can Team Merge this PR so next I will Create PR in ossfuzz to integrate this Project
Thanks

@echoix
Copy link
Member

echoix commented Dec 27, 2024

Were you able to build and use this in the meantime? Its a new folder and I don't see any changes to makefiles or anything.

How do you plan naming the PRs related to this? (In order to start with that one before merging)

@Shivam7-1
Copy link
Author

Shivam7-1 commented Dec 27, 2024

Hii @echoix
The build.sh,dockerfile and project.yaml will be on ossfuzz Project directory and this fuzz file would be in addon project Directory which is current project for this fuzz named folder has been created in Commit Or we can have all this file all together in ossfuzz project directory
So this is how it going to work here

For the naming of the PR in OSS-Fuzz, I will name it "grass-addons: initial integration" to reflect the integration of the grass-addons project into OSS-Fuzz. This will also make it clear that this is the first step in integrating the project, and I will ensure the necessary build system changes are included.

@Shivam7-1
Copy link
Author

Hii @echoix
Can you please Share maintainers name or email so who is in list will get notified of Alerts ?
So next I will make PR in ossfuzz
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants