This fixes an oversight from #28196: in the p2p_transport_bidirectional_v2 fuzz test, when the desired garbage length is over 64 bytes, the code would actually use garbage length 0. Fix this.
fuzz: actually test garbage >64b in p2p transport test #29667
pull sipa wants to merge 1 commits into bitcoin:master from sipa:202403_bip324_fuzz_actual_garbage changing 1 files +1 −0-
sipa commented at 3:38 PM on March 17, 2024: member
-
fuzz: actually test garbage >64b in p2p transport test 626f8e398e
-
DrahtBot commented at 3:38 PM on March 17, 2024: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage
For detailed information about the code coverage, see the test coverage report.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK instagibbs, brunoerg If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #29625 (Several randomness improvements by sipa)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
- DrahtBot added the label Tests on Mar 17, 2024
-
instagibbs commented at 2:46 PM on March 18, 2024: member
- maflcko approved
-
maflcko commented at 2:50 PM on March 18, 2024: member
lgtm
- brunoerg approved
-
brunoerg commented at 11:30 AM on March 19, 2024: contributor
crACK 626f8e398e219b84907ccaad036f69177d39284c
- fanquake merged this on Mar 19, 2024
- fanquake closed this on Mar 19, 2024
- PastaPastaPasta referenced this in commit 62c39973e8 on Oct 24, 2024
- PastaPastaPasta referenced this in commit 054edd31a6 on Oct 24, 2024
- PastaPastaPasta referenced this in commit b70e091374 on Oct 24, 2024
- PastaPastaPasta referenced this in commit aaccc9ea51 on Oct 24, 2024
- bitcoin locked this on Mar 19, 2025