zecfoundation
/
zcashd
mirror of https://github.com/ZcashFoundation/zcashd.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

base58: add paranoid return value checks

This commit is contained in:
Jeff Garzik 2014-06-11 23:20:37 -04:00
parent 4800ccb823
commit 88df548dde
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/base58.cpp
View File

@ -114,9 +114,8 @@ std::string EncodeBase58Check(const std::vector<unsigned char>& vchIn) {
} }
bool DecodeBase58Check(const char* psz, std::vector<unsigned char>& vchRet) { bool DecodeBase58Check(const char* psz, std::vector<unsigned char>& vchRet) {
if (!DecodeBase58(psz, vchRet)) if (!DecodeBase58(psz, vchRet) ||
return false; (vchRet.size() < 4))
if (vchRet.size() < 4)
{ {
vchRet.clear(); vchRet.clear();
return false; 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) { bool CBase58Data::SetString(const char* psz, unsigned int nVersionBytes) {
std::vector<unsigned char> vchTemp; std::vector<unsigned char> vchTemp;
DecodeBase58Check(psz, vchTemp); bool rc58 = DecodeBase58Check(psz, vchTemp);
if (vchTemp.size() < nVersionBytes) { if ((!rc58) || (vchTemp.size() < nVersionBytes)) {
vchData.clear(); vchData.clear();
vchVersion.clear(); vchVersion.clear();
return false; return false;