fuzz: fix vector size problem in system fuzzer #18917

pull brakmic wants to merge 1 commits into bitcoin:master from brakmic:fix-system-fuzzer changing 1 files +1 −1

brakmic commented at 6:25 PM on May 8, 2020: contributor

This PR fixes a problem with vector resizing in system fuzzer (case 7 there). Originally, this problem was discussed in PR https://github.com/bitcoin/bitcoin/pull/18908
fuzz: fix vector size problem in system fuzzer
Co-authored-by: Pieter Wuille <pieter.wuille@gmail.com>
095bc9a106
MarcoFalke commented at 6:38 PM on May 8, 2020: member

ACK 095bc9a10691505c3d0fdacb6caeb62bfdcf1732

cc @practicalswift
DrahtBot added the label Tests on May 8, 2020
practicalswift commented at 7:55 PM on May 8, 2020: contributor

ACK 095bc9a10691505c3d0fdacb6caeb62bfdcf1732

Thanks a lot for fixing this mistake of mine @brakmic – the intention was reserve but apparently I wrote resize. Sorry! :)
brakmic commented at 8:03 PM on May 8, 2020: contributor

ACK 095bc9a

Thanks a lot for fixing this mistake of mine @brakmic – the intention was reserve but apparently I wrote rewrite. Sorry! :)

It wasn't me. It was @sipa I only wanted to avoid UB without touching your fuzzer. :)

Cheers!
MarcoFalke merged this on May 8, 2020
MarcoFalke closed this on May 8, 2020
brakmic deleted the branch on May 8, 2020
Fabcien referenced this in commit eb81b068cd on Jan 26, 2021
DrahtBot locked this on Feb 15, 2022

Contributors

Labels