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

Protobuf DoS #2169

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Protobuf DoS #2169

wants to merge 3 commits into from

Conversation

DarkaMaul
Copy link

This (public) advisory follows two emails sent on August 9 and October 3rd.

The crate is affected by the same vulnerability as described in GHSA-735f-pc8j-v9w8

@jayvdb
Copy link
Contributor

jayvdb commented Dec 12, 2024

ping @stepancheg

@tnull
Copy link

tnull commented Dec 13, 2024

The crate is affected by the same vulnerability as described in GHSA-735f-pc8j-v9w8

Given that this affects multiple implementations across different languages, do we know whether other Rust protobuf implementations such as prost might also be affected?

@DarkaMaul
Copy link
Author

The crate is affected by the same vulnerability as described in GHSA-735f-pc8j-v9w8

Given that this affects multiple implementations across different languages, do we know whether other Rust protobuf implementations such as prost might also be affected?

From my understanding, prost is not affected by the same issue:

@tnull
Copy link

tnull commented Dec 13, 2024

From my understanding, prost is not affected by the same issue:

  • it does not support unknown fields
  • it performs a recursion depth limit check when parsing unknown fields

That makes sense. Thank you for clarifying!

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.

4 participants