Fix use of uninitialized memory in bench_verify #350

pull apoelstra wants to merge 1 commits into bitcoin-core:master from apoelstra:verify-fix changing 1 files +1 −0
  1. apoelstra commented at 1:43 PM on November 3, 2015: contributor

    No description provided.

  2. Fix use of uninitialized memory in bench_verify e3e9f218e2
  3. gmaxwell commented at 12:30 AM on November 4, 2015: contributor

    This actually may be my error. The API does not claim this is in/out.

    So, either the API doc should be updated (sigh), OR we should stop zeroizing (sigh) OR we should only zeroize 33 bytes (sigh). The first has the problem that its still slightly less consistent with the calls that have fixed size output (like the compact seralize) in that they do not have an in or out size.

  4. sipa cross-referenced this on Nov 5, 2015 from issue Turn secp256k1_ec_pubkey_serialize outlen to in/out by sipa
  5. apoelstra commented at 12:56 PM on November 5, 2015: contributor

    Closing in favor of #351.

  6. apoelstra closed this on Nov 5, 2015

  7. apoelstra deleted the branch on Jun 19, 2017

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/secp256k1. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-14 14:15 UTC

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