mirror of
https://github.com/dashpay/dash.git
synced 2024-12-27 13:03:17 +01:00
Advance pindexLastCommonBlock for blocks in chainActive
This prevents an edge case where a block downloaded and pruned in-between successive calls to FindNextBlocksToDownload could cause the block to be unnecessarily re-requested.
This commit is contained in:
parent
dbd8550231
commit
3e9143386a
@ -433,13 +433,14 @@ void FindNextBlocksToDownload(NodeId nodeid, unsigned int count, std::vector<CBl
|
|||||||
|
|
||||||
// Iterate over those blocks in vToFetch (in forward direction), adding the ones that
|
// Iterate over those blocks in vToFetch (in forward direction), adding the ones that
|
||||||
// are not yet downloaded and not in flight to vBlocks. In the mean time, update
|
// are not yet downloaded and not in flight to vBlocks. In the mean time, update
|
||||||
// pindexLastCommonBlock as long as all ancestors are already downloaded.
|
// pindexLastCommonBlock as long as all ancestors are already downloaded, or if it's
|
||||||
|
// already part of our chain (and therefore don't need it even if pruned).
|
||||||
BOOST_FOREACH(CBlockIndex* pindex, vToFetch) {
|
BOOST_FOREACH(CBlockIndex* pindex, vToFetch) {
|
||||||
if (!pindex->IsValid(BLOCK_VALID_TREE)) {
|
if (!pindex->IsValid(BLOCK_VALID_TREE)) {
|
||||||
// We consider the chain that this peer is on invalid.
|
// We consider the chain that this peer is on invalid.
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
if (pindex->nStatus & BLOCK_HAVE_DATA) {
|
if (pindex->nStatus & BLOCK_HAVE_DATA || chainActive.Contains(pindex)) {
|
||||||
if (pindex->nChainTx)
|
if (pindex->nChainTx)
|
||||||
state->pindexLastCommonBlock = pindex;
|
state->pindexLastCommonBlock = pindex;
|
||||||
} else if (mapBlocksInFlight.count(pindex->GetBlockHash()) == 0) {
|
} else if (mapBlocksInFlight.count(pindex->GetBlockHash()) == 0) {
|
||||||
|
Loading…
Reference in New Issue
Block a user