Include a comment on secp256k1_ecdsa_sign explaining low-s. #205

pull gmaxwell wants to merge 1 commits into bitcoin-core:master from gmaxwell:lows_comment changing 1 files +27 −1
  1. gmaxwell commented at 2:45 am on February 2, 2015: contributor
    This was suggested by DavidEGrayson.
  2. gmaxwell commented at 2:45 am on February 2, 2015: contributor
  3. Include a comment on secp256k1_ecdsa_sign explaining low-s.
    This was suggested by DavidEGrayson.
    6cc8425c91
  4. DavidEGrayson commented at 2:59 am on February 2, 2015: none
    Thanks, that comment looks good to me! I am glad it talks about flipping the sign of S instead of modifying the nonce.
  5. droark cross-referenced this on Feb 2, 2015 from issue Some comments explaining some of the constants in the code. by gmaxwell
  6. DavidEGrayson cross-referenced this on Feb 4, 2015 from issue secp256k1_ecdsa_sign: document the low S values by DavidEGrayson
  7. sipa commented at 1:05 am on February 13, 2015: contributor
    ACK
  8. sipa merged this on Feb 13, 2015
  9. sipa closed this on Feb 13, 2015

  10. sipa referenced this in commit 122a1ecf30 on Feb 13, 2015

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-11-21 21:15 UTC

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