Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add initial wallet_bdb_parser corpus #140

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Sjors
Copy link
Member

@Sjors Sjors commented Aug 1, 2023

Generated on Ubuntu 23.04 using Clang 15.0.7 and libFuzzer
--with-sanitizers=fuzzer

Generated by running the wallet_bdb_parser target in bitcoin/bitcoin#26606 for about a day on AMD Ryzen 9 7950X.

I used test/fuzz/test_runner.py --m_dir which reduced the inputs from 1210 (200KB) to 116 (2 KB).

I wonder if it makes sense to manually generate some inputs based on real (testnet) wallets.

Generated on Ubuntu 23.04 using Clang 15.0.7.
--with-sanitizers=fuzzer
@Sjors
Copy link
Member Author

Sjors commented Aug 1, 2023

cc @achow101 @TheCharlatan

@maflcko
Copy link
Contributor

maflcko commented Aug 1, 2023

I wonder if it makes sense to manually generate some inputs based on real (testnet) wallets.

This should be trivially possible by removing the use of ConsumeRandomLengthByteVector/FuzzedDataProvider and replace it with outfile << Span{buffer};

3ae851ea6b6b5a15fa168c598b15aee1282d2b0f is a regtest wallet
90090898d8dd570e4003cfb42b0eaf97e05290ee produces a crash
@Sjors
Copy link
Member Author

Sjors commented Aug 2, 2023

Pushed a commit that contains a wallet and a crash. Let me know if it's non-trivial to reproduce.

@Sjors
Copy link
Member Author

Sjors commented Aug 3, 2023

bda20e7f3777a147f944da4a50b9147e68b7f4b0 produces a UndefinedBehaviorSanitizer for me with bitcoin/bitcoin@a7508fe.

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.

None yet

2 participants