diff options
| author | Jeff Garzik <[email protected]> | 2014-06-11 23:20:37 -0400 |
|---|---|---|
| committer | Jeff Garzik <[email protected]> | 2014-06-11 23:20:37 -0400 |
| commit | 88df548dde280efac8eb33520f8192885455bc03 (patch) | |
| tree | d826d410d48a90e879882bd89e0224a47f0e16f7 /src/base58.cpp | |
| parent | Merge pull request #4328 (diff) | |
| download | discoin-88df548dde280efac8eb33520f8192885455bc03.tar.xz discoin-88df548dde280efac8eb33520f8192885455bc03.zip | |
base58: add paranoid return value checks
Diffstat (limited to 'src/base58.cpp')
| -rw-r--r-- | src/base58.cpp | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/src/base58.cpp b/src/base58.cpp index 597570388..1bd64684e 100644 --- a/src/base58.cpp +++ b/src/base58.cpp @@ -114,9 +114,8 @@ std::string EncodeBase58Check(const std::vector<unsigned char>& vchIn) { } bool DecodeBase58Check(const char* psz, std::vector<unsigned char>& vchRet) { - if (!DecodeBase58(psz, vchRet)) - return false; - if (vchRet.size() < 4) + if (!DecodeBase58(psz, vchRet) || + (vchRet.size() < 4)) { vchRet.clear(); return false; @@ -154,8 +153,8 @@ void CBase58Data::SetData(const std::vector<unsigned char> &vchVersionIn, const bool CBase58Data::SetString(const char* psz, unsigned int nVersionBytes) { std::vector<unsigned char> vchTemp; - DecodeBase58Check(psz, vchTemp); - if (vchTemp.size() < nVersionBytes) { + bool rc58 = DecodeBase58Check(psz, vchTemp); + if ((!rc58) || (vchTemp.size() < nVersionBytes)) { vchData.clear(); vchVersion.clear(); return false; |