Skip to content

Conversation

murchandamus
Copy link
Contributor

After the package_rbf fuzz target was updated, the old fuzz inputs produced sub-optimal coverage, so I put together some new fuzz inputs for the updated target.

The old fuzz inputs after bitcoin/bitcoin#30300 was merged:

#1188	DONE   cov: 3651 ft: 20819 corp: 388/13927Kb lim: 1041407 exec/s: 132 rss: 618Mb

After fuzzing a bit and creating a fresh set_cover_merge from old and new:

#195	DONE   cov: 3676 ft: 21559 corp: 193/15247Kb lim: 796083 exec/s: 24 rss: 580Mb

I decided to merge the result from the set_cover_merge rather than replacing the old fuzz inputs, so that the fuzz test continues to work for older versions of the master branch and the latest major release.

@maflcko
Copy link
Contributor

maflcko commented Jun 21, 2024

lgtm

@dergoegge dergoegge merged commit f76936c into bitcoin-core:main Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants