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

Fuzzing: Add more bug detectors #1253

Merged
merged 1 commit into from Dec 31, 2022
Merged

Fuzzing: Add more bug detectors #1253

merged 1 commit into from Dec 31, 2022

Conversation

AdamKorcz
Copy link
Contributor

@AdamKorcz AdamKorcz commented Dec 21, 2022

Signed-off-by: AdamKorcz adam@adalogics.com

Summary

Adds more bug detectors to the Rekor fuzzers. Currently 4 bug detectors come with the "instrumentation" project. It is still early stage and is being added to OSS-Fuzz projects one by one. Containerd, Docker, Istio, Golang are other projects that this is integrated with.

The "instrumentation" project adds a series of checks in the target code. It analyzes the project, modifies its AST and overwrites the source files with the updated AST. Since this is all done inside the OSS-Fuzz build container, the changes to the source tree are only applied for fuzz testing.

Release Note

Documentation

Signed-off-by: AdamKorcz <adam@adalogics.com>
@AdamKorcz AdamKorcz requested a review from a team as a code owner December 21, 2022 15:58
@codecov-commenter
Copy link

Codecov Report

Merging #1253 (bb58c74) into main (4273c66) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1253      +/-   ##
==========================================
+ Coverage   63.15%   63.19%   +0.03%     
==========================================
  Files          82       82              
  Lines        7670     7670              
==========================================
+ Hits         4844     4847       +3     
+ Misses       2212     2210       -2     
+ Partials      614      613       -1     
Flag Coverage Δ
e2etests 47.74% <ø> (ø)
unittests 42.04% <ø> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/types/helm/v0.0.1/entry.go 47.47% <0.00%> (+1.26%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Contributor

@naveensrinivasan naveensrinivasan left a comment

Choose a reason for hiding this comment

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

What does the bug detector do? Could you please add some information as to what it does?

@AdamKorcz
Copy link
Contributor Author

What does the bug detector do? Could you please add some information as to what it does?

Added to the PR message.

@bobcallaway bobcallaway merged commit 526caed into sigstore:main Dec 31, 2022
@github-actions github-actions bot added this to the v1.1.0 milestone Dec 31, 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.

None yet

4 participants