diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2021-02-19 12:58:30 +0100 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2021-02-19 12:58:41 +0100 |
| commit | 09eb46c9430bb549cf4ffc3815a951ecdb90fef7 (patch) | |
| tree | 66f61c6e354107a9fd75eb7362c8a7e4007f46a4 /src/test | |
| parent | 6a680a6236e07c93f28514fe05fbb64a29cb2ec8 (diff) | |
| parent | 3e68efa615968e0c9d68a7f197c7852478f6be78 (diff) | |
| download | bitcoin-09eb46c9430bb549cf4ffc3815a951ecdb90fef7.tar.xz | |
Merge #21187: Net processing: Only call PushAddress() from net_processing
3e68efa615968e0c9d68a7f197c7852478f6be78 [net] Move checks from GetLocalAddrForPeer to caller (John Newbery)
d21d2b264cd77c027a06f68289cf4c3f177d1ed0 [net] Change AdvertiseLocal to GetLocalAddrForPeer (John Newbery)
Pull request description:
This is the first part of #21186. It slightly disentangles addr handling in net/net_processing by making it explicit that net_processing is responsible for pushing addr records into `vAddrToSend`.
ACKs for top commit:
MarcoFalke:
re-ACK 3e68efa615968e0c9d68a7f197c7852478f6be78 🍅
Tree-SHA512: 9af50c41f5a977e2e277f24a589db38e2980b353401def5e74b108ac5f493d9b5d6b1b8bf15323a4d66321495f04bc271450fcef7aa7d1c095f051a4f8e9b15f
Diffstat (limited to 'src/test')
| -rw-r--r-- | src/test/net_tests.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/net_tests.cpp b/src/test/net_tests.cpp index f52905e1ef..1c7c35528e 100644 --- a/src/test/net_tests.cpp +++ b/src/test/net_tests.cpp @@ -691,7 +691,7 @@ BOOST_AUTO_TEST_CASE(ipv4_peer_with_ipv6_addrMe_test) pnode->SetAddrLocal(addrLocal); // before patch, this causes undefined behavior detectable with clang's -fsanitize=memory - AdvertiseLocal(&*pnode); + GetLocalAddrForPeer(&*pnode); // suppress no-checks-run warning; if this test fails, it's by triggering a sanitizer BOOST_CHECK(1); |