diff options
author | W. J. van der Laan <laanwj@protonmail.com> | 2021-05-12 17:20:21 +0200 |
---|---|---|
committer | W. J. van der Laan <laanwj@protonmail.com> | 2021-05-12 17:39:39 +0200 |
commit | 6b49d88a5dda97fdbabe363fd7d3c4f1ce29082a (patch) | |
tree | 6dddc09ab5d40ddc6dd1eafb8c347083a8c214ed /REVIEWERS | |
parent | d2ec37221a5636650573e0f451f39c9fd563e935 (diff) | |
parent | 9c891b64ffd14bc8216dbd5eb60816043af265b6 (diff) | |
download | bitcoin-6b49d88a5dda97fdbabe363fd7d3c4f1ce29082a.tar.xz |
Merge bitcoin/bitcoin#21905: net: initialize nMessageSize to uint32_t max
9c891b64ffd14bc8216dbd5eb60816043af265b6 net: initialize nMessageSize to max uint32_t instead of -1 (eugene)
Pull request description:
nMessageSize is uint32_t and is set to -1. This will warn with `-fsanitize=implicit-integer-sign-change` when V1TransportDeserializer calls into the ctor. This pull initializes nMessageSize to `numeric_limits<uint32_t>::max()` instead and removes the ubsan suppression.
ACKs for top commit:
laanwj:
Code review ACK 9c891b64ffd14bc8216dbd5eb60816043af265b6
promag:
Code review ACK 9c891b64ffd14bc8216dbd5eb60816043af265b6.
Tree-SHA512: f05173d9553a01d207a5a7f8ff113d9e11354c50b494a67d44d3931c151581599a9da4e28f40edd113f4698ea9115e6092b2a5b7329c841426726772076c1493
Diffstat (limited to 'REVIEWERS')
0 files changed, 0 insertions, 0 deletions