Skip to content

Conversation

@ennamarie19
Copy link

Following up on #1198, this PR adds a fuzzing harnesses and a build script to integrate OSS-Fuzz into the project.

Further, this adds the necessary workflow pipeline to automatically fuzz all pull requests into the stable and develop branches. Until this is merged into stable, that specific pipeline will fail (as it must both be merged in here and into Google's respective OSSFuzz repo before builds start succeeding and producing results / uncovering bugs).

Added pipeline

Fix path in build script

Formatting

Add

Updated changelog to reflect PR number
@jsvine
Copy link
Owner

jsvine commented Jan 10, 2025

Thanks, @ennamarie19. Perhaps I misunderstood, but I was under the impression that fuzzing could be supported in a synced fork (to avoid adding to this repository's infrastructural complexity)?: #1198 (comment)

@ennamarie19
Copy link
Author

ennamarie19 commented Jan 10, 2025 via email

@jsvine
Copy link
Owner

jsvine commented Jan 13, 2025

Great, thanks!

@ennamarie19
Copy link
Author

@jsvine I just submitted for OSSFuzz' consideration without upstream merge, would very much appreciate if you could leave a comment on the PR supporting the out-of-repo fuzzing (to get ahead of any confirmation needs Google may want)

The PR can be found here. Thank you!

DonggeLiu pushed a commit to google/oss-fuzz that referenced this pull request Jan 27, 2025
This pull request integrates the Dockerfile needed to build the fuzzers
for pdfplumber.

Note: The fuzzers were NOT merged upstream following discussion with the
project maintainer
[here](jsvine/pdfplumber#1245 (comment))
and with the precedence for out-of-repo fuzzers established
[here](#12467)

---------

Co-authored-by: bcapuano <[email protected]>
Co-authored-by: Bailey Capuano <[email protected]>
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