fuzz: reset connman state in p2p targets #35808

pull HowHsu wants to merge 1 commits into bitcoin:master from HowHsu:fuzz-connman changing 2 files +2 −0

HowHsu commented at 11:23 AM on July 26, 2026: contributor
Resets ConnmanTestMsg after StopNodes() in cmpctblock and p2p_handshake, matching the other reused-connman fuzz targets and preventing sticky CConnman state from leaking between corpus inputs.

Before this, deterministic-fuzz-coverage showed single inputs were stable, but all-input directory runs were not:
```
cmpctblock, src/net.cpp:4172
- Branch (4172:9): [True: 1.21k, False: 33.0k]
+ Branch (4172:9): [True: 613, False: 33.6k]
- Branch (4172:72): [True: 901, False: 311]
+ Branch (4172:72): [True: 497, False: 116]
```
```
p2p_handshake, src/net.cpp:4172
- Branch (4172:9): [True: 98, False: 1.67k]
+ Branch (4172:9): [True: 743, False: 1.03k]
- Branch (4172:72): [True: 90, False: 8]
+ Branch (4172:72): [True: 612, False: 131]
```
After the cmpctblock reset, deterministic-fuzz-coverage ... cmpctblock 8 passed.

fuzz: reset connman state in p2p targets

Clear ConnmanTestMsg state after stopping test nodes in cmpctblock and
p2p_handshake so sticky CConnman state does not leak between fuzz inputs.

fd403f9474

DrahtBot added the label Fuzzing on Jul 26, 2026
DrahtBot commented at 11:23 AM on July 26, 2026: contributor



The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/35808.



Reviews

See the guideline and AI policy for information on the review process. A summary of reviews will appear here.

Contributors

Labels