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

Added fuzzer #279

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added fuzzer #279

wants to merge 1 commit into from

Conversation

AdamKorcz
Copy link

Dear maintainers and contributors of ejdb,

I have worked on the security of your project and set it up such that it can be fuzzed. As such, I thought it could be useful to set ejdb up on the OSS-Fuzz infrastructure, which will allow the current fuzzer to run on Googles infrastructure continuously. If a bug is discovered, the maintainers receive an email with a link to a detailed bug report. This is done free of charge, although there is an expectation that the bugs are fixed so the fuzzers can go on and find new bugs.

The current fuzzer will be enough to integrate with OSS-Fuzz and looking forward if you are happy to receive contributions in this domain then I would be glad to put more efforts into fuzzing ejdb.

If you would like this then please merge in and I will set things up on the OSS-Fuzz end. To complete this initial setup the only thing I need is an email address from you which will receive the bug reports - notice this email address will be public on the OSS-Fuzz repo.

@DavidKorczynski
Copy link

I was wondering if there were updates on this? If you would like for the Ejdb to be fuzzed we would be happy to put more efforts into it - if not then we can just close this PR! @adamansky perhaps?

@adamansky
Copy link
Member

adamansky commented May 6, 2020

@DavidKorczynski

I was wondering if there were updates on this? If you would like for the Ejdb to be fuzzed we would be happy to put more efforts into it - if not then we can just close this PR! @adamansky perhaps?

Hi! Thank you for this patch! I will apply changes anyway but need more tight integration with cmake and test cases. So I need some spare time to do that.

@AdamKorcz
Copy link
Author

Are there updates on this PR? A route can be to run the fuzzer in the CI with ClusterfuzzLite.

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