diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2020-07-14 15:26:05 +0200 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2020-07-14 15:26:40 +0200 |
| commit | 1a655e89cb4dbdf6c9b47626aff2f06fa4fa4dee (patch) | |
| tree | 5c3a2b75bb283a132cbb8092ca01c929ab92800c /src/fs.cpp | |
| parent | 07c83ce0399a2811bdd764b0cce347ba3bdbe37c (diff) | |
| parent | ca3585a483ca5f6fc4cc54fd1530f89d13e5b7b0 (diff) | |
| download | bitcoin-1a655e89cb4dbdf6c9b47626aff2f06fa4fa4dee.tar.xz | |
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/fs.cpp')
0 files changed, 0 insertions, 0 deletions