test/fuzz: added fuzz target addrdb #35330

pull kevkevinpal wants to merge 1 commits into bitcoin:master from kevkevinpal:fuzzAddrDB changing 1 files +77 −0

kevkevinpal commented at 12:53 PM on May 20, 2026: contributor

Summary

This adds the fuzz target addrdb and adds coverage for LoadAddrman, DumpAnchors and ReadAnchors.

I'm not seeing any coverage on this currently, so I figured it might be worth adding some.

test/fuzz: added fuzz target addrdb

This adds the fuzz target addrdb and adds coverage for LoadAddrman,
DumpAnchors and ReadAnchors.

0a21bfa61d

DrahtBot commented at 12:53 PM on May 20, 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/35330.



Reviews

See the guideline for information on the review process. A summary of reviews will appear here.
maflcko commented at 2:55 PM on May 20, 2026: member

Is this worth it? The functions are thin wrappers around ser-code. I think it could make more sense to have functional coverage for them for end-users, and fuzz coverage of the ser-code, but I think both already exist?
kevkevinpal commented at 1:16 PM on May 21, 2026: contributor

Is this worth it? The functions are thin wrappers around ser-code. I think it could make more sense to have functional coverage for them for end-users, and fuzz coverage of the ser-code, but I think both already exist?

Gotcha, no problem, I can close this PR if there is no need for this coverage
kevkevinpal closed this on May 21, 2026

Contributors

Summary