Add missing infinity checks #110

pull sipa wants to merge 3 commits into bitcoin-core:master from sipa:noinf changing 5 files +51 −13
  1. sipa commented at 12:13 PM on November 18, 2014: contributor

    This adds infinity checks when serializing or recovering public keys, and a test for it.

    Either fix is enough to solve #109.

  2. Make secp256k1_eckey_pubkey_serialize fail for infinity bbe67d8b29
  3. Test whether recovered public keys are not infinity
    Fixes a bug discovered by Sergio Demian Lerner.
    4861f83686
  4. Add test that recovering infinity fails 3bf029d676
  5. sipa commented at 12:14 PM on November 18, 2014: contributor

    Ping @SergioDemianLerner.

  6. gmaxwell commented at 4:39 PM on November 18, 2014: contributor

    ACK

  7. sipa merged this on Nov 18, 2014
  8. sipa closed this on Nov 18, 2014
  9. sipa referenced this in commit ad2028f989 on Nov 18, 2014
Contributors
sipagmaxwell
Linked (view graph)
#109 Missing point-at-infinity check in int secp256k1_ecdsa_sig_recover() can cause non-deterministic results
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-05-19 06:53 UTC