diff options
author | laanwj <126646+laanwj@users.noreply.github.com> | 2022-05-26 20:06:07 +0200 |
---|---|---|
committer | laanwj <126646+laanwj@users.noreply.github.com> | 2022-05-26 20:06:10 +0200 |
commit | c5e67be03bb06a5d7885c55db1f016fbf2333fe3 (patch) | |
tree | 8a4c86ae4ca081cbfe4ab19229a8ebab4f82e254 /test | |
parent | a0e8aff6051bfeaad38799058a2a46fa2e45911d (diff) | |
parent | c4c5b9ca6e98cf44309af73edf5559940a04d00f (diff) | |
download | bitcoin-c5e67be03bb06a5d7885c55db1f016fbf2333fe3.tar.xz |
Merge bitcoin/bitcoin#24032: Add defaults to vDeployments to avoid uninitialized variables
c4c5b9ca6e98cf44309af73edf5559940a04d00f consensus/params: set default values for BIP9Deployment (Anthony Towns)
Pull request description:
Adds default values for `vDeployments` in `consensus/params.h` so that undefined behaviour is avoided if a deployment is not initialized. Also adds a check in the unit tests to alert if this is happening, since even if it doesn't invoke undefined behaviour it's probably a mistake.
ACKs for top commit:
laanwj:
Code review ACK c4c5b9ca6e98cf44309af73edf5559940a04d00f
Tree-SHA512: 22d7ff86a817d9e9e67c47301fc3b7e9d5821c13565d7706711f113dea220eea29b413a7c8d029691009159cebc85a108d77cb52418734091c196bafb2b12423
Diffstat (limited to 'test')
0 files changed, 0 insertions, 0 deletions