Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

build: enable CIFuzz#1490

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
DavidKorczynski wants to merge1 commit intocoveragepy:main
base:main
Choose a base branch
Loading
fromDavidKorczynski:cifuzz-oss-fuzz

Conversation

@DavidKorczynski
Copy link

AddCIFuzz workflow action to have fuzzers build and run on each PR.
This is a service offered by OSS-Fuzz, on which coveragepy already runs.

CIFuzz can help detect catch regressions and fuzzing build issues early, and has a variety of features (see the URL above). In the current PR the fuzzer is built on a pull request and will run for 300 seconds.

Signed-off-by: David Korczynskidavid@adalogics.com

Signed-off-by: David Korczynski <david@adalogics.com>
@nedbat
Copy link
Member

Thanks! There was also some mention some place of moving the fuzzing code itself into this repo so that we could extend it. How do we make that happen?

@DavidKorczynski
Copy link
Author

DavidKorczynski commentedNov 18, 2022
edited
Loading

There was also some mention some place of moving the fuzzing code itself into this repo so that we could extend it. How do we make that happen?

Move this codehttps://github.com/google/oss-fuzz/blob/master/projects/coveragepy/fuzz_parse.py into this library. We can move this anytime now -- feel free to make a PR on OSS-Fuzz that remove it and moves it up here. We should adjusthttps://github.com/google/oss-fuzz/blob/master/projects/coveragepy/build.sh andhttps://github.com/google/oss-fuzz/blob/master/projects/coveragepy/Dockerfile accordingly, but this is likely just switching a link.

In order to buildfuzz_parse.py in the OSS-Fuzz environment, we usehttps://github.com/google/oss-fuzz/blob/master/projects/coveragepy/build.sh It would be great to continue usingcompile_python_fuzzer for the OSS-Fuzz environment since it makes it possible to get the coverage reports (thanks to this repo :) !) as well as introspector reports (still early stage for Python:https://github.com/ossf/fuzz-introspector).

If we place the fuzzer intest/fuzzers/ of this repo, then we can adjust the build script on OSS-Fuzz to use:

for fuzzer in $(find ./test/fuzzers/ -name 'fuzz_*.py'); do  compile_python_fuzzer $fuzzerdone

The source folder in thefind command is the only thing changed. That will make it possible to simply put new fuzzers intest/fuzzers/* and OSS-Fuzz will start running them continuously automatically.

I can also make a PR that does the above -- let me know if you'd like me to and I'll do this tomorrow Friday.

@ProsperousHeart
Copy link
Contributor

ProsperousHeart commentedMar 20, 2023
edited
Loading

Are this and#1497 still valid@nedbat ?

@nedbat
Copy link
Member

@DavidKorczynski Sorry I've let this and#1497 sit for a few years. Are these still current? I'm a tiny bit reluctant to take on new actions, but also am baffled when I get a fuzzing report from the centralized runs that Google does.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@DavidKorczynski@nedbat@ProsperousHeart

[8]ページ先頭

©2009-2025 Movatter.jp