LBRYCommunity/lbrycrd
1
0
Fork 0
Code Issues 29 Pull requests 1 Projects Releases 32 Packages Wiki Activity

refactor getValueForClaim, fix off by one error in the conditional statement that could cause segfault

Browse source
This commit is contained in:
Victor Shyba 2018-08-20 10:33:36 -03:00 committed by kay kurokawa
parent 7c3ef9b635
commit 368a7389f4
No known key found for this signature in database
GPG key ID: AA15748C1D2B529F
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/rpc/claimtrie.cpp
View file

@ -77,7 +77,7 @@ UniValue getclaimsintrie(const UniValue& params, bool fHelp)
__func__, itClaims->outPoint.hash.GetHex());
claim.push_back(Pair("error", "No value found for claim"));
}
else if (coin->vout.size() < itClaims->outPoint.n || coin->vout[itClaims->outPoint.n].IsNull())
else if (!coin->IsAvailable(itClaims->outPoint.n))
{
LogPrintf("%s: the specified txout of %s appears to have been spent\n", __func__, itClaims->outPoint.hash.GetHex());
claim.push_back(Pair("error", "Txout spent"));
@ -153,7 +153,7 @@ bool getValueForClaim(const COutPoint& out, std::string& sValue)
__func__, out.hash.GetHex());
return true;
}
if (coin->vout.size() < out.n || coin->vout[out.n].IsNull())
if(!coin->IsAvailable(out.n))
{
LogPrintf("%s: the specified txout of %s appears to have been spent\n", __func__, out.hash.GetHex());
return true;