diff options
| author | merge-script <fanquake@gmail.com> | 2024-10-21 14:58:44 +0100 |
|---|---|---|
| committer | merge-script <fanquake@gmail.com> | 2024-10-21 14:58:44 +0100 |
| commit | d9f8dc645343a4432fca75c5fa2156e8cf06efae (patch) | |
| tree | 0c6429355f4da41dd19062aa24ab05948fa9c4d6 /test/util/test_runner.py | |
| parent | 563c4d292687d2fd7aa3d21eef7a8256cd645c21 (diff) | |
| parent | 86e2a6b749c7fecbd086b361806ac9f6e9426d79 (diff) | |
Merge bitcoin/bitcoin#31097: validation: Improve input script check error reporting
86e2a6b749c7fecbd086b361806ac9f6e9426d79 [test] A non-standard transaction which is also consensus-invalid should return the consensus error (Antoine Poinsot)
f859ff8a4e9c3aa23bf5be6eceb7099ca72b2290 [validation] Improve script check error reporting (dergoegge)
Pull request description:
An input script might be invalid for multiple reasons. For example, it might fail both a standardness check and a consensus check, which can lead to a `mandatory-script-verify-flag-failed` error being reported that includes the script error string from the standardness failure (e.g. `mandatory-script-verify-flag-failed (Using OP_CODESEPARATOR in non-witness script)`), which is confusing.
ACKs for top commit:
darosior:
re-ACK 86e2a6b749c7fecbd086b361806ac9f6e9426d79
ariard:
Re-Code Review ACK 86e2a6b7
instagibbs:
ACK 86e2a6b749c7fecbd086b361806ac9f6e9426d79
Tree-SHA512: 053939107c0bcd6643e9006b2518ddc3a6de47d2c6c66af71a04e8af5cf9ec207f19e54583b7a056efd77571edf5fd4f36c31ebe80d1f0777219c756c055eb42
Diffstat (limited to 'test/util/test_runner.py')
0 files changed, 0 insertions, 0 deletions