fuzz: Make encryption tests for CCrypter roundtrip #34527

pull Chand-ra wants to merge 1 commits into bitcoin:master from Chand-ra:crypter changing 1 files +13 −3
  1. Chand-ra commented at 7:06 pm on February 6, 2026: none

    Add verification logic to the crypter fuzz target by decrypting the resulting ciphertext from a successful call to Encrypt() or EncryptSecret() and verifying that the output matches the original plaintext.

    This ensures that the encryption/decryption cycle preserves data correctness, rather than just checking for execution crashes.

  2. fuzz: Make encryption tests for `CCrypter` roundtrip
    Add verification logic to the `crypter` fuzz target by decrypting
    the resulting ciphertext from a successful call to `Encrypt()` or
    `EncryptSecret()` and verifying that the output matches the
    original plaintext.
    
    This ensures that the encryption/decryption cycle preserves data
    correctness, rather than just checking for execution crashes.
    4a2f787010
  3. DrahtBot added the label Fuzzing on Feb 6, 2026
  4. DrahtBot commented at 7:06 pm on February 6, 2026: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

  5. Chand-ra commented at 7:12 pm on February 6, 2026: none

    Unsure if changes like this are needed. The oracles added are inexpensive enough to not affect fuzzing performance, and they do increase chances of catching a future regression, but there’s no increase in coverage so it might be code churn after all.

    Marking as draft for now.


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-02-10 21:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me