diff options
| author | fanquake <fanquake@gmail.com> | 2022-06-23 16:13:22 +0100 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2022-06-23 16:18:07 +0100 |
| commit | e05564d706f06ccbebb89b4e6d76adb6db9aa820 (patch) | |
| tree | 26e97458e0ff4460ecc4f8e0891754289b7b90d2 | |
| parent | 01e9e2d1ca7fc08d65663b398c71ecec6a01f686 (diff) | |
| parent | e7a9133766a2dc357a79d38cd47231cc99cdefb7 (diff) | |
| download | bitcoin-e05564d706f06ccbebb89b4e6d76adb6db9aa820.tar.xz | |
Merge bitcoin/bitcoin#25446: p2p: Set CNode::m_relays_txs=true when receiving BIP37 filters
e7a9133766a2dc357a79d38cd47231cc99cdefb7 [net processing] Set CNode::m_relays_txs=true when receiving BIP37 filters (dergoegge)
Pull request description:
This line was accidentally removed in https://github.com/bitcoin/bitcoin/pull/22778.
Receiving a `filterload` message implies that we should relay txs to the sender (`CNode::m_relays_txs = true`). `CNode::m_relays_txs` is only used for the inbound eviction logic, so removing the line might have slightly changed the eviction behaviour but nothing else.
ACKs for top commit:
laanwj:
Code review ACK e7a9133766a2dc357a79d38cd47231cc99cdefb7
vasild:
ACK e7a9133766a2dc357a79d38cd47231cc99cdefb7
Tree-SHA512: 19c5df0f579f707c6c7900d12a6b71ac69e802be64f7d2fdcc40ac714c918dc4c17def164592f8836cc105a03daefefca3ca5e10423145eca8db4348c27c9cfc
| -rw-r--r-- | src/net_processing.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/net_processing.cpp b/src/net_processing.cpp index 751a03f01c..cbba1d0aa2 100644 --- a/src/net_processing.cpp +++ b/src/net_processing.cpp @@ -4075,6 +4075,7 @@ void PeerManagerImpl::ProcessMessage(CNode& pfrom, const std::string& msg_type, tx_relay->m_relay_txs = true; } pfrom.m_bloom_filter_loaded = true; + pfrom.m_relays_txs = true; } return; } |