dinedal / textql

Execute SQL against structured text like CSV or TSV
MIT License
9.07k stars 300 forks source link

Added fuzzer for sqlparser #124

Closed AdamKorcz closed 4 years ago

AdamKorcz commented 4 years ago

This PR adds a fuzzer to the sql parser. It can be run both locally and on oss-fuzz's infrastructure, and I will be happy to setup an integration application to oss-fuzz to run the fuzzer continuously. oss-fuzz is a free service that is offered with an implied expectation that bugs are fixed by the maintainers. If a bug is found through oss-fuzz, a link to a detailed bug report is sent to the contact email addresses of textql. All I need to apply for integration are the email addresses for this list. The list is public and can be changed at any time.

dinedal commented 4 years ago

Maybe you forgot to add all the files to the commit?

AdamKorcz commented 4 years ago

No, I didn't forget to add all the files. The added test is enough to run both locally or through continuous fuzzing by means of oss-fuzz. Running the fuzzer continuously does require some infrastructure, but this does not have to be uploaded to the projects repository for now.

AdamKorcz commented 4 years ago

As mentioned, I will be glad to see if we can setup continuous fuzzing as well. For that I need an email address for potential bug reports, and in case you are familiar with any adopters that would help as well.