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.

Sign up for GitHub

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

Jump to bottom

Add 1st fuzzing #700

Merged
merged 9 commits into from
Jan 6, 2025
Merged

Add 1st fuzzing #700

merged 9 commits into from
Jan 6, 2025

Conversation

cedelavergne-ledger
Copy link
Contributor

@ledger-wiz-cspm-secret-detection
Copy link

ledger-wiz-cspm-secret-detection bot commented Dec 16, 2024 •
edited
Loading

Wiz Scan Summary

Scanner Findings
Data Finding Sensitive Data
Secret Finding Secrets
IaC Misconfiguration IaC Misconfigurations 1 High 1 Medium
Total 1 High 1 Medium

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

@cedelavergne-ledger cedelavergne-ledger force-pushed the cev/add_fuzzing branch 7 times, most recently from 43106d2 to 1130130 Compare December 17, 2024 09:14
@cedelavergne-ledger cedelavergne-ledger marked this pull request as ready for review December 17, 2024 09:43
@bboilot-ledger bboilot-ledger force-pushed the cev/add_fuzzing branch 3 times, most recently from 1775e30 to a5afb73 Compare December 27, 2024 14:58
@bboilot-ledger
Copy link
Contributor

bboilot-ledger commented Dec 27, 2024 •
edited
Loading

Hello 👋
I have added the other fuzzers done during review and merged everything in one harness. Also cleaned up few things in the cmake.
Added a script to run the fuzzer locally and compute the global coverage.
A few bugs reported by the fuzzer have been fixed (I am not using the memory fuzzer much but clusterfuzzlite runs it everytime).
Also improved the mocks and added a local build with MemSan.

@cedelavergne-ledger cedelavergne-ledger merged commit 906f640 into develop Jan 6, 2025
174 checks passed
@cedelavergne-ledger cedelavergne-ledger deleted the cev/add_fuzzing branch January 6, 2025 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ledger-wiz-cspm-secret-detection ledger-wiz-cspm-secret-detection[bot] ledger-wiz-cspm-secret-detection[bot] left review comments

@bboilot-ledger bboilot-ledger bboilot-ledger left review comments

@apaillier-ledger apaillier-ledger apaillier-ledger approved these changes

@tdejoigny-ledger tdejoigny-ledger Awaiting requested review from tdejoigny-ledger

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cedelavergne-ledger @bboilot-ledger @apaillier-ledger