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 *ring*. #1930

Merged
merged 1 commit into from
Nov 7, 2018
Merged

Add *ring*. #1930

merged 1 commit into from
Nov 7, 2018

Conversation

briansmith
Copy link
Contributor

ring is a Rust cryptography project that is a fork of BoringSSL. It is used by many Rust-based projects; see https://crates.io/crates/ring/reverse_dependencies for the tip of the iceberg. Many projects use ring through Rustls, the Rust TLS implementation.

@briansmith briansmith changed the title And *ring*. Add *ring*. Nov 7, 2018
@jonathanmetzman
Copy link
Contributor

We approve of adding this project.
Add the build.sh and Dockerfile when you can.

@kcc
Copy link
Contributor

kcc commented Nov 7, 2018

This will be the first project in Rust. Eager to see how it works.

@briansmith briansmith deleted the b/add-ring branch November 7, 2018 05:54
@briansmith
Copy link
Contributor Author

Thanks for your help!

This will be the first project in Rust. Eager to see how it works.

Do you have any tips for integrating Rust projects into oss-fuzz based on what you might have done internally with other projects, e.g. the Rust components of Fuchsia?

@jonathanmetzman
Copy link
Contributor

cc @mbarbella-chromium who might know more about the fuchsia efforts.

@Dor1s
Copy link
Contributor

Dor1s commented Feb 14, 2020

Any plans to enable fuzzing on this project? I think we have some running Rust fuzzers now.

@briansmith
Copy link
Contributor Author

Any plans to enable fuzzing on this project? I think we have some running Rust fuzzers now.

Yes, I am planning to add the fuzzing soon-ish now that everything is sorted out. Thanks!

@jonathanmetzman
Copy link
Contributor

Since this is a rust project, I'll repost my comment from #3382 (comment)

Rust fuzzing should 100% work and we are eager to support it.
But there is one sharp edge I know of that I haven't gotten around to investigating, you should probably look at this.
Also, some runtime changes were made by the the first rust project integrated, I'm not sure if those are generally needed: ec58a65 94d0c3a

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.

None yet

4 participants