diff options
| author | Cory Fields <cory-nospam-@coryfields.com> | 2015-01-09 22:06:08 -0500 |
|---|---|---|
| committer | Cory Fields <cory-nospam-@coryfields.com> | 2015-01-09 22:06:55 -0500 |
| commit | 8dccba6a45db0466370726ed462b9da2eae43bce (patch) | |
| tree | 44083128b139ad0171401972140430fd99cb4d32 /src | |
| parent | dad7764a9dedb1143b6ea8bfd37f71f7ff72f518 (diff) | |
fail immediately on an empty signature
Diffstat (limited to 'src')
| -rw-r--r-- | src/ecwrapper.cpp | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/ecwrapper.cpp b/src/ecwrapper.cpp index 33dd9a9a29..0236e90c16 100644 --- a/src/ecwrapper.cpp +++ b/src/ecwrapper.cpp @@ -117,6 +117,9 @@ bool CECKey::SetPubKey(const unsigned char* pubkey, size_t size) { } bool CECKey::Verify(const uint256 &hash, const std::vector<unsigned char>& vchSig) { + if (vchSig.empty()) + return false; + // New versions of OpenSSL will reject non-canonical DER signatures. de/re-serialize first. unsigned char *norm_der = NULL; ECDSA_SIG *norm_sig = ECDSA_SIG_new(); |