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 blogpost for etcd fuzzing audit #566

Merged
merged 9 commits into from
Mar 11, 2022
Merged

Add blogpost for etcd fuzzing audit #566

merged 9 commits into from
Mar 11, 2022

Conversation

AdamKorcz
Copy link
Contributor

@AdamKorcz AdamKorcz commented Mar 11, 2022

Adds the blogpost for the fuzzing audit performed by Ada Logics.

Hold this PR until the report has been merged here: etcd-io/etcd#13788

@spzala @DavidKorczynski @caniszczyk

---

In the last few months the team at [Ada Logics](https://adalogics.com) has worked on integrating continuous fuzzing into the etcd project. This was an effort focused on improving the security posture of etcd and ensuring a continued good experience for etcds users. The fuzzing integration involved enrolling etcd in the OSS-Fuzz project and writing a set of fuzzers that would bring the test coverage of etcd up to a mature level. In total, 18 fuzzers were written and 8 bugs were found, demonstrating the value the work has had for etcd both short term and long term. All fuzzers were implemented by way of go-fuzz and when running in OSS-Fuzz instrumented by way of libFuzzer, and as such, etcd uses state-of-the-art open source fuzzing capabilities.
The full report of the engagement can be found [here](https://github.com/etcd-io/etcd/blob/main/security/audit_fuzzer_adalogics_2022.pdf).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume it should also land in website, probably: https://github.com/etcd-io/website/tree/main/content/en/community or just blog

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ptabor thanks so much for the quick review. It was planned to add as a blog only. The community page seems to focus on community-wide ongoing collaborative events. It has a Twitter handler under Join the conversation, and we will tweet blog and report.

Copy link
Member

@spzala spzala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Thanks @AdamKorcz

---

In the last few months the team at [Ada Logics](https://adalogics.com) has worked on integrating continuous fuzzing into the etcd project. This was an effort focused on improving the security posture of etcd and ensuring a continued good experience for etcds users. The fuzzing integration involved enrolling etcd in the OSS-Fuzz project and writing a set of fuzzers that would bring the test coverage of etcd up to a mature level. In total, 18 fuzzers were written and 8 bugs were found, demonstrating the value the work has had for etcd both short term and long term. All fuzzers were implemented by way of go-fuzz and when running in OSS-Fuzz instrumented by way of libFuzzer, and as such, etcd uses state-of-the-art open source fuzzing capabilities.
The full report of the engagement can be found [here](https://github.com/etcd-io/etcd/blob/main/security/audit_fuzzer_adalogics_2022.pdf).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ptabor thanks so much for the quick review. It was planned to add as a blog only. The community page seems to focus on community-wide ongoing collaborative events. It has a Twitter handler under Join the conversation, and we will tweet blog and report.

@@ -0,0 +1,31 @@
---
title: etcd Integrates Continuous Fuzzing
spelling: cSpell:ignore Gyuho
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we need spelling: but it's okay, will remove it later if needed.

@spzala
Copy link
Member

spzala commented Mar 11, 2022

We will merge it after the report gets merged.

Copy link
Contributor

@nate-double-u nate-double-u left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've done a copy-edit pass, please see my suggestions inline. Otherwise, looks good to me!

@AdamKorcz
Copy link
Contributor Author

I've done a copy-edit pass, please see my suggestions inline. Otherwise, looks good to me!

Thank you for the thorough read-through.

@spzala
Copy link
Member

spzala commented Mar 11, 2022

@nate-double-u thanks for the review and valuable comments. @AdamKorcz thank you for quickly addressing Nate's comments.

@spzala spzala merged commit 9a00f79 into etcd-io:main Mar 11, 2022
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