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 cargo-fuzz test harness for the qos_crypto crate #441

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

cr-tk
Copy link
Collaborator

@cr-tk cr-tk commented Mar 8, 2024

Summary & Motivation (Problem vs. Solution)

Add some fuzzing test coverage of basic qos_crypto functions on Shamir's Secret Sharing functionality, mainly our wrappers around vsss-rs.

How I Tested These Changes

Running the fuzz tests locally.
No change to production code.

Pre merge check list

Does not apply.

@cr-tk cr-tk added the enhancement New feature or request label Mar 8, 2024
@cr-tk cr-tk self-assigned this Mar 8, 2024
@cr-tk
Copy link
Collaborator Author

cr-tk commented Apr 25, 2024

This fuzzer harness is on hold, since the SSS code it tests is supposed to get replaced soon.

@cr-tk cr-tk force-pushed the christian/qos_crypto_fuzz1 branch from 7b338fb to dfe845a Compare November 26, 2024 14:09
@cr-tk cr-tk marked this pull request as ready for review November 26, 2024 14:11
@cr-tk cr-tk requested a review from a team as a code owner November 26, 2024 14:11
@cr-tk
Copy link
Collaborator Author

cr-tk commented Nov 26, 2024

I have reworked, extended and rebased the changes against the newest main. PR description is updated as well.

This is ready for review and merge.

@r-n-o r-n-o merged commit 39e8b41 into main Nov 27, 2024
6 checks passed
@r-n-o r-n-o deleted the christian/qos_crypto_fuzz1 branch November 27, 2024 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants