diff options
author | Sebastian Falbesoner <sebastian.falbesoner@gmail.com> | 2022-09-05 00:30:54 +0200 |
---|---|---|
committer | Sebastian Falbesoner <sebastian.falbesoner@gmail.com> | 2022-09-05 00:30:54 +0200 |
commit | 644772b9efffda4dac01aff54042b3162079514d (patch) | |
tree | a91663da2279dd9d52552239d404e9b438735987 /contrib/verifybinaries | |
parent | 0ebd4db32b39cb7c505148f090df4b7ac778c307 (diff) |
message-capture-parser: fix AssertionError on parsing `headers` message
If a test framework message's field name is in the list of
`HASH_INT_VECTORS`, we currently assume that it _always_ has to contain
a vector of integers and throw otherwise (introduced in PR #25367,
commit 42bbbba7c83d1e2baad18b4c6f05bad1358eb117). However, that
assumption is too strict. In this concrete case, the (de)serialization
field name "headers" is used in two different message types, one for
`cfcheckpt` (where it is serialized as an integer vector), and another
time for `headers` (where it is serialized as a vector of
`CBlockHeader`s). Fix by adding the integer type check as additional
condition to the `HASH_INT_VECTORS` check rather than asserting.
Fixes #25954.
Diffstat (limited to 'contrib/verifybinaries')
0 files changed, 0 insertions, 0 deletions