Partially revert #27780, because moving the duplicate coinbase check out of the was_valid branch leads to non-bug crashes in the fuzz target.
For context and further explanation see: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=59516
fuzz: Partially revert #27780 #27810
pull dergoegge wants to merge 1 commits into bitcoin:master from dergoegge:2023-06-fuzz-supply-fix changing 1 files +5 −5-
dergoegge commented at 1:45 PM on June 3, 2023: member
-
[fuzz] Only check duplicate coinbase script when block was valid 71200ac390
-
DrahtBot commented at 1:45 PM on June 3, 2023: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK MarcoFalke If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
- DrahtBot added the label Tests on Jun 3, 2023
- fanquake requested review from maflcko on Jun 5, 2023
-
maflcko commented at 1:44 PM on June 7, 2023: member
nice lgtm ACK 71200ac390fe5c10f088cbe8053b010b515757b1
- DrahtBot removed review request from maflcko on Jun 7, 2023
- fanquake merged this on Jun 7, 2023
- fanquake closed this on Jun 7, 2023
- sidhujag referenced this in commit 21c16ecde3 on Jun 7, 2023
- bitcoin locked this on Jun 6, 2024
Labels