diff options
| author | fanquake <fanquake@gmail.com> | 2021-10-08 11:37:43 +0800 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2021-10-08 12:13:48 +0800 |
| commit | 97e2435ac43f1a633acfcba734b1a1405ec48bfb (patch) | |
| tree | c2a62f5ee7532f65ffb6c85ed47e447f17ada94f /src/util/url.h | |
| parent | 99e53967c85c20a166803f94ebb127f1d7ee9198 (diff) | |
| parent | 44452110f0fa7cc1bcb941a3c7b5db4b492a7b9c (diff) | |
Merge bitcoin/bitcoin#23053: [fuzz] Use public methods in addrman fuzz tests
44452110f0fa7cc1bcb941a3c7b5db4b492a7b9c [fuzz] Update comment in FillAddrman() (John Newbery)
640476eb0e17fd4c64d4840ceab229642f1d79d9 [fuzz] Make Fill() a free function in fuzz/addrman.cpp (John Newbery)
90ad8ad61a38dbb1f247a5f3d5f649a856d9938a [fuzz] Make RandAddr() a free function in fuzz/addrman.cpp (John Newbery)
491975c596ebce93ae8de192c9ef171f002fac7c [fuzz] Pass FuzzedDataProvider& into Fill() in addrman fuzz tests (John Newbery)
56303e382e26ac7096c09152c66894dc3bb4d1fd [fuzz] Create a FastRandomContext in addrman fuzz tests (John Newbery)
Pull request description:
#22974 improved the performance of `CAddrMan::Good()` significantly so that it could be used directly in the fuzz tests, instead of those tests reaching directly into addrman's internal data structures.
This PR continues the work of making the fuzz tests only use `CAddrMan`'s public methods and pulls the `Fill()` and `RandAddr()` methods from `CAddrManDeterministic` into free functions, since they no longer need access to `CAddrMan` internals.
ACKs for top commit:
theuni:
utACK 44452110f0fa7cc1bcb941a3c7b5db4b492a7b9c. Agree the failure seems unrelated, looks like some startup race.
mzumsande:
ACK 44452110f0fa7cc1bcb941a3c7b5db4b492a7b9c
vasild:
ACK 44452110f0fa7cc1bcb941a3c7b5db4b492a7b9c
Tree-SHA512: fcf994e1dedd0012b77f632720b6423d51ceda4eb85c9efe572f2a1150117f9e511114a5206738dd94409137287577f3b01a9998f5237de845410d3d96e7cb7f
Diffstat (limited to 'src/util/url.h')
0 files changed, 0 insertions, 0 deletions