Full checking of all loaded keys #786

pull sipa wants to merge 1 commits into bitcoin:master from sipa:checkkeys changing 2 files +15 −1
  1. sipa commented at 6:29 pm on January 26, 2012: member

    See #782: @piratelinux is right in saying that not everything is checked, as GetPubKey() returns the public key embedded in the PrivKey structure, and does not force recalculation.

    CKey::IsValid() performs this check now.

  2. Full checking of all loaded keys 91f43a33f8
  3. TheBlueMatt commented at 6:33 pm on January 26, 2012: member
    ACK
  4. gavinandresen commented at 3:32 am on January 27, 2012: contributor
    ACK
  5. luke-jr commented at 7:20 pm on January 28, 2012: member
    Does this remove the ability to intentionally remove/corrupt your private keys and run with read-only access?
  6. sipa commented at 2:40 am on January 29, 2012: member

    Yes…

    You can always encrypt it and throw away the passphrase, though.

  7. gavinandresen merged this on Jan 31, 2012
  8. gavinandresen closed this on Jan 31, 2012

  9. destenson referenced this in commit 2c3e480681 on Jun 26, 2016
  10. ptschip referenced this in commit abd2ec4fc0 on Oct 17, 2017
  11. DrahtBot locked this on Sep 8, 2021

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: 2024-10-05 01:12 UTC

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