Add missing mempool lock for CalculateMemPoolAncestors

TheBlueMatt · TheBlueMatt · commit 003cd60cc369 · 2017-01-20T18:11:24.000-05:00
diff --git a/src/txmempool.cpp b/src/txmempool.cpp
@@ -173,6 +173,8 @@ void CTxMemPool::UpdateTransactionsFromBlock(const std::vector<uint256> &vHashes
 
 bool CTxMemPool::CalculateMemPoolAncestors(const CTxMemPoolEntry &entry, setEntries &setAncestors, uint64_t limitAncestorCount, uint64_t limitAncestorSize, uint64_t limitDescendantCount, uint64_t limitDescendantSize, std::string &errString, bool fSearchForParents /* = true */) const
 {
+    LOCK(cs);
+
     setEntries parentHashes;
     const CTransaction &tx = entry.GetTx();
 

Original file line number	Diff line number	Diff line change
`@@ -173,6 +173,8 @@ void CTxMemPool::UpdateTransactionsFromBlock(const std::vector<uint256> &vHashes`
`173`	`173`
`174`	`174`	`bool CTxMemPool::CalculateMemPoolAncestors(const CTxMemPoolEntry &entry, setEntries &setAncestors, uint64_t limitAncestorCount, uint64_t limitAncestorSize, uint64_t limitDescendantCount, uint64_t limitDescendantSize, std::string &errString, bool fSearchForParents /* = true */) const`
`175`	`175`	`{`
	`176`	`+ LOCK(cs);`
	`177`	`+`
`176`	`178`	`setEntries parentHashes;`
`177`	`179`	`const CTransaction &tx = entry.GetTx();`
`178`	`180`