diff options
| author | MarcoFalke <[email protected]> | 2020-07-14 15:26:05 +0200 |
|---|---|---|
| committer | MarcoFalke <[email protected]> | 2020-07-14 15:26:40 +0200 |
| commit | 1a655e89cb4dbdf6c9b47626aff2f06fa4fa4dee (patch) | |
| tree | 5c3a2b75bb283a132cbb8092ca01c929ab92800c /src/interfaces/node.cpp | |
| parent | Merge #19268: doc: Add non-thread-safe note to FeeFilterRounder::round() (diff) | |
| parent | [net/net processing] check banman pointer before dereferencing (diff) | |
| download | discoin-1a655e89cb4dbdf6c9b47626aff2f06fa4fa4dee.tar.xz discoin-1a655e89cb4dbdf6c9b47626aff2f06fa4fa4dee.zip | |
Merge #19514: [net/net processing] check banman pointer before dereferencing
ca3585a483ca5f6fc4cc54fd1530f89d13e5b7b0 [net/net processing] check banman pointer before dereferencing (John Newbery)
Pull request description:
Although we currently don't do this, it should be possible to create a
CConnman or PeerLogicValidation without a Banman instance. Therefore
always check that banman exists before dereferencing the pointer.
Also add comments to the m_banman members of CConnman and
PeerLogicValidation to document that these may be nullptr.
ACKs for top commit:
jonatack:
ACK ca3585a
theStack:
ACK https://github.com/bitcoin/bitcoin/commit/ca3585a483ca5f6fc4cc54fd1530f89d13e5b7b0
Tree-SHA512: 726401c8921b9a502029ead34ae797473a1bc359d6e4e58dcbe3e25b70dde40bb100723be467fd3e2bf418892c493911998226de19c9d529d72034e3be26be48
Diffstat (limited to 'src/interfaces/node.cpp')
0 files changed, 0 insertions, 0 deletions