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

OSS-Fuzz Integration #1029

Closed

Conversation

ennamarie19
Copy link
Contributor

Hello,
This is a follow-on from my Issue regarding OSS-Fuzz integration. This PR introduces the source for the fuzzer. I would greatly appreciate it being merged in!

I will continue monitoring OSS-Fuzz for bugs and fix any I can in future PRs. John McNamara also has access to the OSS-Fuzz dashboard to review any security-relevant crashes that may come up.

Please let me know if anything else is needed.

Thank you!

@jmcnamara
Copy link
Owner

Thanks.

Could you move it under the dev directory in the XlsxWriter source directory, i.e., dev/fuzzing.

If possible could you squash the commits into one. If not I can do it merge.

Also,could you explain how this will work in practice. Will I be able to find/verify issues locally or does it rely on OSS-Fuzz?

@ennamarie19
Copy link
Contributor Author

@jmcnamara I will move it over right now.

I will also look into squashing my commits.

As for finding the issues, a nightly build of a Dockerized instance of the project is run on Google's ClusterFuzz server and will report any bugs to you via the email listed in the project.yaml in the OSSFuzz repo. PRs can then be made to resolve the errors and, once merged, the bugs would be marked as fixed after a few days of the bug not re-appearing during fuzz-testing

@ennamarie19
Copy link
Contributor Author

@jmcnamara Ready for review!

@jmcnamara
Copy link
Owner

PRs can then be made to resolve the errors and, once merged, the bugs would be marked as fixed after a few days of the bug not re-appearing during fuzz-testing

Sounds good.

I will also look into squashing my commits.

I don't think that worked. I see 6 commits and there is duplicate code in the dev and xlsxwriter directories. Could you check again.

@ennamarie19 ennamarie19 closed this Nov 6, 2023
Copy link

sonarqubecloud bot commented Nov 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

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.

3 participants