-
-
Notifications
You must be signed in to change notification settings - Fork 86
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
separate fuzzers into different files #655
Conversation
Can you also update build.sh file? |
Sure, I have to run linter as well |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
@anonrig we should be able to see the results here, right? https://storage.googleapis.com/oss-fuzz-introspector/ada-url/inspector-report/20240513/fuzz_report.html Just wanted to confirm it is configured correctly. |
@CarlosEduR Yes but they run it on a cronjob. The GitHub workflow is merely for local development and does not cover the whole fuzzer. Basically, oss-fuzz runs a docker file (implemented in https://github.com/google/oss-fuzz/tree/master/projects/ada-url) which executes |
@CarlosEduR Can you also disable memory sanitizer in a different pull-request if you don't mind? |
sure, @CarlosEduR! From the cifuzz.yml, right? |
@CarlosEduR they are not blocked. they just test a specific portion of the codebase ~0.1%. but this is good progress! |
thanks for explaining that @anonrig, good to know and glad to help! |
No description provided.