From 4763b51bca86fb9e49175619a47cdbef34feaf99 Mon Sep 17 00:00:00 2001 From: John Newbery Date: Thu, 25 Jun 2020 15:39:32 -0400 Subject: [PATCH] ProcessOrphanTx: remove useless setMisbehaving set This starts empty, and is only added to if we're about to exit the function (so we never read from it). --- src/net_processing.cpp | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/net_processing.cpp b/src/net_processing.cpp index 45cb29065e..40783e210c 100644 --- a/src/net_processing.cpp +++ b/src/net_processing.cpp @@ -2048,7 +2048,6 @@ void PeerManager::ProcessOrphanTx(std::set& orphan_work_set, std::list< { AssertLockHeld(cs_main); AssertLockHeld(g_cs_orphans); - std::set setMisbehaving; while (!orphan_work_set.empty()) { const uint256 orphanHash = *orphan_work_set.begin(); @@ -2065,7 +2064,6 @@ void PeerManager::ProcessOrphanTx(std::set& orphan_work_set, std::list< // that relayed the previous transaction). TxValidationState orphan_state; - if (setMisbehaving.count(fromPeer)) continue; if (AcceptToMemoryPool(m_mempool, orphan_state, porphanTx, &removed_txn, false /* bypass_limits */, 0 /* nAbsurdFee */)) { LogPrint(BCLog::MEMPOOL, " accepted orphan tx %s\n", orphanHash.ToString()); RelayTransaction(orphanHash, porphanTx->GetWitnessHash(), m_connman); @@ -2081,14 +2079,12 @@ void PeerManager::ProcessOrphanTx(std::set& orphan_work_set, std::list< break; } else if (orphan_state.GetResult() != TxValidationResult::TX_MISSING_INPUTS) { if (orphan_state.IsInvalid()) { - // Punish peer that gave us an invalid orphan tx - if (MaybePunishNodeForTx(fromPeer, orphan_state)) { - setMisbehaving.insert(fromPeer); - } LogPrint(BCLog::MEMPOOL, " invalid orphan tx %s from peer=%d. %s\n", orphanHash.ToString(), fromPeer, orphan_state.ToString()); + // Maybe punish peer that gave us an invalid orphan tx + MaybePunishNodeForTx(fromPeer, orphan_state); } // Has inputs but not accepted to mempool // Probably non-standard or insufficient fee