Skip to content

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