dash/blackcoin-more
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge branch 'master' into 'master'

Browse Source 
Fix memory leak

See merge request blackcoin/blackcoin-more!10
This commit is contained in:
lateminer
2020-07-20 19:57:35 +00:00
parent 2884e9b970 efee370024
commit 8022a97986
1 changed files with 17 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
src/pos.cpp
View File

@@ -196,7 +196,14 @@ bool CheckKernel(CBlockIndex* pindexPrev, unsigned int nBits, uint32_t nTime, co
return false;
}
return CheckStakeKernelHash(pindexPrev, nBits, new CCoins(txPrev, pindexPrev->nHeight), prevout, nTime);
//CheckStakeKernalHash needs a pointer to coins in order to be used to validate the chain on disk/memort
//Using a pointer in this context is not needed, but is required by the function.
//Must ensure coins is deleted to prevent memory leak.
CCoins* coins = new CCoins(txPrev, pindexPrev->nHeight);
bool result = CheckStakeKernelHash(pindexPrev, nBits, coins, prevout, nTime);
delete coins;
return result;
} else {
//found in cache
const CStakeCache& stake = it->second;
@@ -206,7 +213,15 @@ bool CheckKernel(CBlockIndex* pindexPrev, unsigned int nBits, uint32_t nTime, co
return CheckKernel(pindexPrev, nBits, nTime, prevout);
}
*/
return CheckStakeKernelHash(pindexPrev, nBits, new CCoins(stake.txPrev, pindexPrev->nHeight), prevout, nTime);
//CheckStakeKernalHash needs a pointer to coins in order to be used to validate the chain on disk/memort
//Using a pointer in this context is not needed, but is required by the function.
//Must ensure coins is deleted to prevent memory leak.
CCoins* coins = new CCoins(stake.txPrev, pindexPrev->nHeight);
bool result = CheckStakeKernelHash(pindexPrev, nBits, coins, prevout, nTime);
delete coins;
return result;
}
}