oss-fuzz icon indicating copy to clipboard operation
oss-fuzz copied to clipboard

gVisor opts out of the oss-fuzz.

Open milantracy opened this issue 1 year ago • 6 comments

milantracy avatar Feb 07 '24 19:02 milantracy

milantracy is a new contributor to projects/gvisor. The PR must be approved by known contributors before it can be merged. The past contributors are: avagin, kyakdan, 0roman

github-actions[bot] avatar Feb 07 '24 19:02 github-actions[bot]

i don't see an option to request reviews, @avagin @kyakdan @0roman do you have a chance to take a look?

milantracy avatar Feb 07 '24 19:02 milantracy

@milantracy pls add a commit message that explains why gvisor has to be opted out.

avagin avatar Feb 08 '24 17:02 avagin

@milantracy pls add a commit message that explains why gvisor has to be opted out.

@avagin done. Force push a new commit at the PR to overwrite the original commit message.

milantracy avatar Feb 08 '24 19:02 milantracy

Hi,

Would it make sense at all to just remove the public mailing list from here: https://github.com/google/oss-fuzz/blob/abe93d11506b18c55a7d20a5f982ed6fa332151e/projects/gvisor/project.yaml#L5?

Of course, this is your team's decision but we'd also be happy to chat through any ways we can make this more useful for you.

It also seems a bit contradictory if the findings are both not useful and should not be leaked to the public :)

oliverchang avatar Feb 12 '24 09:02 oliverchang

thank you Oliver, the team added gVisor to oss-fuzz because we thought it might check a box for compliance or something to have more (and public) fuzzing of gVisor. It appears that security value oss-fuzz brings to the project is very minimal, there are a bunch of false bug reports like https://github.com/google/gvisor/issues/8357 and https://github.com/google/gvisor/issues/7440

so the team decided that we shall just remove the gVisor config from oss-fuzz.

milantracy avatar Feb 12 '24 23:02 milantracy

could anyone stamp the pr :/

milantracy avatar Feb 18 '24 09:02 milantracy