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.

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

Add [email protected] to auto_ccs for image-png project. #12557

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

anforowicz
Copy link

@anforowicz anforowicz commented Oct 2, 2024

OSS-Fuzz maintainers - can you PTAL?

Trustworthiness of this request:

  • This is a memory-safe, Rust-based project (no unsafe in png and fdeflate crates, although there is some unsafe in other dependencies). This argues that most issues discovered by fuzzing should not be security issues, merely correctness/functional issues (or at most Denial-of-Service issues when a timeout happens for some inputs).
  • I am contributing to the png project as a Chromium Security engineer, so I can hopefully claim some of the trust/karma that may be attributed to the Chromium Security team.

Motivation for this request:

Disclaimer: in the long-term the ownership of png / Skia / Chromium integration may shift, but I think it still makes sense to add me for now

/cc @fintelia

Copy link

github-actions bot commented Oct 2, 2024

anforowicz is a new contributor to projects/image-png. The PR must be approved by known contributors before it can be merged. The past contributors are: fintelia, silvergasp

@vitorguidi
Copy link
Contributor

@fintelia @silvergasp can I go ahead and merge?

@nathaniel-brough
Copy link
Contributor

I don't have any objections personally. This seems reasonable to me. But I'll leave this up to @fintelia as he is the maintainer.

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