This is a follow-up to #27171, adding a check that the sigop-limit vsize logic is also respected for {ancestor,descendant}size calculation (as suggested in #27171#pullrequestreview-1331143909). For simplicity, we use a one-parent-one-child cluster here and only check for the case that the sigop-limit equivalent size is larger than the serialized vsize.
test: check that sigop limit also affects ancestor/descendant size (27171 follow-up) #27265
pull theStack wants to merge 1 commits into bitcoin:master from theStack:202303-test-check_sigops_for_ancestordescendant_sizes changing 1 files +29 −0-
theStack commented at 6:48 PM on March 15, 2023: contributor
-
test: check that sigop limit also affects ancestor/descendant size 6d24d1ef2b
-
DrahtBot commented at 6:48 PM on March 15, 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 glozow 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 Mar 15, 2023
- maflcko requested review from glozow on Mar 16, 2023
-
glozow commented at 12:06 PM on March 18, 2023: member
code review ACK 6d24d1ef2be7a86ddd798c4966d705e72013b6af, thanks for taking!
- fanquake merged this on Mar 19, 2023
- fanquake closed this on Mar 19, 2023
- sidhujag referenced this in commit 11890d713a on Mar 19, 2023
- theStack deleted the branch on Mar 19, 2023
- bitcoin locked this on Mar 18, 2024
Labels