mirror/bitcoin
1
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

net: Avoid logging AlreadyHaveTx when disconnecting misbehaving peer

Browse Source 
Can be reviewed with --color-moved=dimmed-zebra
This commit is contained in:
MarcoFalke committed 2021-09-20 09:25:02 +02:00
1 parent 226731ac11
commit fa2662c293
1 file changed
+6 -5
Download Patch File Download Diff File Expand all files Collapse all files
src/net_processing.cpp
+6 -5
View File
@@ -2949,16 +2949,17 @@ void PeerManagerImpl::ProcessMessage(CNode& pfrom, const std::string& msg_type,
best_block = &inv.hash; best_block = &inv.hash;
} }
} else if (inv.IsGenTxMsg()) { } else if (inv.IsGenTxMsg()) {
if (reject_tx_invs) {
LogPrint(BCLog::NET, "transaction (%s) inv sent in violation of protocol, disconnecting peer=%d\n", inv.hash.ToString(), pfrom.GetId());
pfrom.fDisconnect = true;
return;
}
const GenTxid gtxid = ToGenTxid(inv); const GenTxid gtxid = ToGenTxid(inv);
const bool fAlreadyHave = AlreadyHaveTx(gtxid); const bool fAlreadyHave = AlreadyHaveTx(gtxid);
LogPrint(BCLog::NET, "got inv: %s %s peer=%d\n", inv.ToString(), fAlreadyHave ? "have" : "new", pfrom.GetId()); LogPrint(BCLog::NET, "got inv: %s %s peer=%d\n", inv.ToString(), fAlreadyHave ? "have" : "new", pfrom.GetId());
pfrom.AddKnownTx(inv.hash); pfrom.AddKnownTx(inv.hash);
if (reject_tx_invs) { if (!fAlreadyHave && !m_chainman.ActiveChainstate().IsInitialBlockDownload()) {
LogPrint(BCLog::NET, "transaction (%s) inv sent in violation of protocol, disconnecting peer=%d\n", inv.hash.ToString(), pfrom.GetId());
pfrom.fDisconnect = true;
return;
} else if (!fAlreadyHave && !m_chainman.ActiveChainstate().IsInitialBlockDownload()) {
AddTxAnnouncement(pfrom, gtxid, current_time); AddTxAnnouncement(pfrom, gtxid, current_time);
} }
} else { } else {