diff options
| author | Pieter Wuille <pieter.wuille@gmail.com> | 2014-11-05 10:53:59 -0800 |
|---|---|---|
| committer | Pieter Wuille <pieter.wuille@gmail.com> | 2014-11-10 05:03:24 -0800 |
| commit | f4e0aefadcc9f37611a6d5a13e3f7b4b9a37bf76 (patch) | |
| tree | 2850672169ed04c051d3d70f65b3d29aadfc8027 /src/ecwrapper.cpp | |
| parent | 5406f61373fe93326ab1f546e4da9f4528236cc7 (diff) | |
| download | bitcoin-f4e0aefadcc9f37611a6d5a13e3f7b4b9a37bf76.tar.xz | |
Do signature-s negation inside the tests
To avoid the need for libsecp256k1 to expose such functionality.
Diffstat (limited to 'src/ecwrapper.cpp')
| -rw-r--r-- | src/ecwrapper.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/ecwrapper.cpp b/src/ecwrapper.cpp index ebaa350264..3377dce0c1 100644 --- a/src/ecwrapper.cpp +++ b/src/ecwrapper.cpp @@ -193,7 +193,7 @@ bool CECKey::SetPubKey(const unsigned char* pubkey, size_t size) { return o2i_ECPublicKey(&pkey, &pubkey, size) != NULL; } -bool CECKey::Sign(const uint256 &hash, std::vector<unsigned char>& vchSig, bool lowS) { +bool CECKey::Sign(const uint256 &hash, std::vector<unsigned char>& vchSig) { vchSig.clear(); ECDSA_SIG *sig = ECDSA_do_sign((unsigned char*)&hash, sizeof(hash), pkey); if (sig == NULL) @@ -205,7 +205,7 @@ bool CECKey::Sign(const uint256 &hash, std::vector<unsigned char>& vchSig, bool BIGNUM *halforder = BN_CTX_get(ctx); EC_GROUP_get_order(group, order, ctx); BN_rshift1(halforder, order); - if (lowS && BN_cmp(sig->s, halforder) > 0) { + if (BN_cmp(sig->s, halforder) > 0) { // enforce low S values, by negating the value (modulo the order) if above order/2. BN_sub(sig->s, order, sig->s); } |