Skip to content

fuzz: follow up for #22704 #23806

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

Merged
merged 1 commit into from
Dec 18, 2021

Conversation

stratospher
Copy link
Contributor

This PR addresses comments from #22704 to make the following changes in src/test/fuzz/crypto_diff_fuzz_chacha20.cpp:

  • replace memcmp() with ==
  • add a missing assert statement to compare the encrypted bytes

@laanwj laanwj added the Tests label Dec 17, 2021
@maflcko maflcko merged commit 98a2ddc into bitcoin:master Dec 18, 2021
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Dec 18, 2021
8f79831 Refactor the chacha20 differential fuzz test (stratospher)

Pull request description:

  This PR addresses [comments from bitcoin#22704](https://github.com/bitcoin/bitcoin/pull/22704/files#discussion_r771510963)  to make the following changes in `src/test/fuzz/crypto_diff_fuzz_chacha20.cpp`:

  - replace `memcmp()` with ==
  - add a missing assert statement to compare the encrypted bytes

Top commit has no ACKs.

Tree-SHA512: 02338460fb3a89e732558bf00f3aebf8f04daba194e03ae0e3339bb2ff6ba35d06841452585b739047a29f8ec64f36b1b4ce2dfa39a08f6ad44a6a937e7b3acb
@bitcoin bitcoin locked and limited conversation to collaborators Dec 18, 2022
@stratospher stratospher deleted the follow-up-diff-fuzz-chacha20 branch July 17, 2024 06:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
No open projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

3 participants