diff options
author | fanquake <fanquake@gmail.com> | 2023-07-28 12:08:17 +0100 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2023-07-28 12:29:55 +0100 |
commit | 42a9110899a9020417f0397456168aea6ac6ade9 (patch) | |
tree | 73e010526d54551137713d9495c148571f56d679 /src/test/fuzz/netbase_dns_lookup.cpp | |
parent | 8aa77a77e6fa35b6c571996854a249999ac66bae (diff) | |
parent | 06199a995f20c55583f6948cfe99e608679fcdf1 (diff) |
Merge bitcoin/bitcoin#28162: refactor: Revert additional univalue check in ParseSighashString
06199a995f20c55583f6948cfe99e608679fcdf1 refactor: Revert addition of univalue sighash string check (TheCharlatan)
0b47c1621524a96b79cbdc3c45ee5ad36e7ba541 doc: Correct release-notes for sighashtype exceptions (TheCharlatan)
Pull request description:
This is a follow up for #28113.
The string type check is already done by the rpc parser / RPCHelpMan. Re-doing it is adding dead code. Instead, throwing an exception when the assumption does not hold is the already correct behavior. Pointed out in this [comment](https://github.com/bitcoin/bitcoin/pull/28113/files#r1274568557).
Also correct the release note for the correct sighashtype exception change. There is no change in the handling of non-string sighashtype arugments. Pointed out in this [comment](https://github.com/bitcoin/bitcoin/pull/28113/files#r1274567555).
ACKs for top commit:
MarcoFalke:
lgtm ACK 06199a995f20c55583f6948cfe99e608679fcdf1
jonatack:
Tested ACK 06199a995f20c55583f6948cfe99e608679fcdf1
stickies-v:
ACK 06199a995f20c55583f6948cfe99e608679fcdf1
Tree-SHA512: 3faa6b3d2247624c0973df8d79c09fbf1f90ffb99f1be484e359b528f485c31affea45976759bd206e4c81cbb54ebba5ad0ef4127d1deacbfe2a58153fcc94ee
Diffstat (limited to 'src/test/fuzz/netbase_dns_lookup.cpp')
0 files changed, 0 insertions, 0 deletions