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.

Sign up for GitHub

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

Jump to bottom

Feat/improve fuzz perf #975

Closed
wants to merge 23 commits into from
Closed

Feat/improve fuzz perf #975

wants to merge 23 commits into from

Conversation

capuanob
Copy link
Contributor

@capuanob capuanob commented Jul 7, 2024 •
edited
Loading

Pull request

This PR seeks to address the discussion concerning low coverage in fuzzing in #949. As seen here, I believe the issue was needing to append _seed_corpus to each fuzzer name in order for OSS-Fuzz to ingest it into the fuzzing runtime.

How Has This Been Tested?

I built the fuzzers on my branch and confirmed that they were properly loading their initial corpora.

Checklist

  • I have read CONTRIBUTING.md.
  • I have added a concise human-readable description of the change to CHANGELOG.md.
  • I have tested that this fix is effective or that this feature works.
  • I have added docstrings to newly created methods and classes.
  • I have updated the README.md and the readthedocs documentation. Or verified that this is not necessary.

capuanob and others added 23 commits March 5, 2024 11:37
@capuanob
Pushing progress
4f4ce5a
@capuanob
Initial fuzzing integration
aa73014
@capuanob
Two proposed simple bug-fixes that were preventing fuzzing from advan…
4063d56 
…cing
@capuanob
Fixed build script
0f41ba6 
Updated build

Updated build

Updated build
@capuanob
Added CIFuzz workflow
2df272e
@capuanob
Removed atheris from Python 3.12 dependencies
9dc4949
@capuanob
Removed atheris from Python 3.12 dependencies
ef620c3
@capuanob
Updated changelog
db33a38
@capuanob
Removed corpus files
8cbc4f9
@pietermarsman
Update CHANGELOG.md
a0b5e06
@pietermarsman
Added fuzzing directory to noxfile.py
ec70d62
@pietermarsman
Fix flake8
c095594
@pietermarsman
Fix mypy
a946a7e
@pietermarsman
Rename test_one_input to fuzz_one_input so PyCharm is not thinking th…
a13bc78 
…ey are test cases
@pietermarsman
Undo fixes, so we can monitor them
cd4d715
@pietermarsman
Also fuzz None values
9b55c8d
@pietermarsman
Use relative imports, just like pdfminer.six package
dc2d032
@pietermarsman
Fix imports
971f402
@pietermarsman
Simplify pdf_utils.py by removing class, just functions
067f94f
@pietermarsman
Subclassing all internal exceptions to PSException such that a single…
2c371db 
… except can catch all
@pietermarsman
Merge branch 'master' into master
0cb509d
@capuanob
Correct name of generated corpus to be properly ingested by ClusterFuzz
f9cae1f
@capuanob
Added to changelog
ee5ef6c
@capuanob capuanob closed this Jul 7, 2024
@capuanob capuanob deleted the feat/improve-fuzz-perf branch July 7, 2024 22:59
Sign up for free to 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.
2 participants
@capuanob @pietermarsman