Add comment on length checks when parsing ECDSA sigs #990

pull real-or-random wants to merge 1 commits into bitcoin-core:master from real-or-random:202110-readlen-comment changing 1 files +4 −3
  1. real-or-random commented at 10:11 am on October 17, 2021: contributor

    I claim the check can be removed but I don’t want to touch this stable and well-tested code.

    On the way, we fix grammar in another comment.

  2. Add comment on length checks when parsing ECDSA sigs
    I claim the check can be removed but I don't want to touch this
    stable and well-tested code.
    
    On the way, we fix grammar in another comment.
    e02f313b1f
  3. sipa commented at 5:43 pm on May 11, 2023: contributor
    ACK e02f313b1f251ccb363ae1ac24016d87c1be9009
  4. real-or-random added the label documentation on May 11, 2023
  5. RandyMcMillan commented at 5:48 pm on May 11, 2023: none
    ACK e02f313
  6. real-or-random removed the label user-documentation on Dec 7, 2023
  7. real-or-random added the label refactor/smell on Dec 7, 2023
  8. real-or-random commented at 8:26 am on December 7, 2023: contributor
    Merging this because it’s just a change in a comment. If we ever change the actual code, we should get more eyes on it.
  9. real-or-random merged this on Dec 7, 2023
  10. real-or-random closed this on Dec 7, 2023


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: 2024-10-31 06:15 UTC

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