Skip to content

fuzz: register PeerManager in process_message(s)#34788

Merged
fanquake merged 1 commit intobitcoin:masterfrom
Crypt-iQ:03102026/pm_fuzz_peerman
Mar 11, 2026
Merged

fuzz: register PeerManager in process_message(s)#34788
fanquake merged 1 commit intobitcoin:masterfrom
Crypt-iQ:03102026/pm_fuzz_peerman

Conversation

@Crypt-iQ
Copy link
Contributor

This lets CValidationInterface callbacks be hit (just BlockChecked using the corpus from qa-assets). Also remove no-op SyncWithValidationInterfaceQueue since there are no validation interfaces registered in ResetChainman.

This lets CValidationInterface callbacks be hit. Also remove
no-op SyncWithValidationInterfaceQueue since there are no validation
interfaces registered in ResetChainman.
@DrahtBot
Copy link
Contributor

DrahtBot commented Mar 10, 2026

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

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK marcofleon, dergoegge

If your review is incorrectly listed, please copy-paste <!--meta-tag:bot-skip--> into the comment that the bot should ignore.

@fanquake
Copy link
Member

cc @dergoegge @marcofleon

Copy link
Contributor

@marcofleon marcofleon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code review ACK b503768

Copy link
Member

@dergoegge dergoegge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK b503768

@fanquake fanquake merged commit b8c84ec into bitcoin:master Mar 11, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants