diff options
| author | Adam Weiss <[email protected]> | 2014-12-18 18:28:29 -0500 |
|---|---|---|
| committer | Adam Weiss <[email protected]> | 2014-12-23 02:19:34 -0500 |
| commit | c90770430d7c1eb7ece2d4ddb987b0f2210fd86f (patch) | |
| tree | d9662d96933a58a8dc7e475a3cf23541577cbb72 /src/main.cpp | |
| parent | Merge pull request #5484 (diff) | |
| download | discoin-c90770430d7c1eb7ece2d4ddb987b0f2210fd86f.tar.xz discoin-c90770430d7c1eb7ece2d4ddb987b0f2210fd86f.zip | |
DOS: Respect max per-peer blocks in flight limit
Don't allow immediate inv driven block downloads if
a peer already has MAX_BLOCKS_IN_TRANSIT_PER_PEER
active downloads. Prevents bogus inv spam from
blowing up block transfer tracking data structures.
Diffstat (limited to 'src/main.cpp')
| -rw-r--r-- | src/main.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/main.cpp b/src/main.cpp index 9c038f90f..accdfd10b 100644 --- a/src/main.cpp +++ b/src/main.cpp @@ -3574,7 +3574,9 @@ bool static ProcessMessage(CNode* pfrom, string strCommand, CDataStream& vRecv, // doing this will result in the received block being rejected as an orphan in case it is // not a direct successor. pfrom->PushMessage("getheaders", chainActive.GetLocator(pindexBestHeader), inv.hash); - if (chainActive.Tip()->GetBlockTime() > GetAdjustedTime() - Params().TargetSpacing() * 20) { + CNodeState *nodestate = State(pfrom->GetId()); + if (chainActive.Tip()->GetBlockTime() > GetAdjustedTime() - Params().TargetSpacing() * 20 && + nodestate->nBlocksInFlight < MAX_BLOCKS_IN_TRANSIT_PER_PEER) { vToFetch.push_back(inv); // Mark block as in flight already, even though the actual "getdata" message only goes out // later (within the same cs_main lock, though). |