mirror/bitcoin
1
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge #20300: fuzz: Add missing ECC_Start to descriptor_parse test

Browse Source 
5cafe2b25c fuzz: Add missing ECC_Start to descriptor_parse test (Ivan Metlushko)

Pull request description:

  Fixes fuzzing harness.

  I also observed that the corpus for this test consists only of `xprv...` keys while we are using regtest parameters. So for proper fuzzing we need either A) to update the corpus and replace `xprv...` with `tprv...` B) switch to main net in the test

ACKs for top commit:
  MarcoFalke:
    review ACK 5cafe2b25c
  practicalswift:
    Tested ACK 5cafe2b25c

Tree-SHA512: 7415a98a445ce0f96219637d2362fecfc1191ad104f55d79ca92b0c92cde165e00646be5bf3fda956385e3cb22540eca457e575048493367cdf0e00a27d7cdb8
This commit is contained in:
MarcoFalke committed 2020-11-04 20:38:18 +01:00
2 parents deb2b27c0d 5cafe2b25c
commit 6760088015
1 file changed
+2 -1
Download Patch File Download Diff File Expand all files Collapse all files
src/test/fuzz/descriptor_parse.cpp
+2 -1
View File
@@ -11,7 +11,8 @@
void initialize() void initialize()
{ {
static const ECCVerifyHandle verify_handle; static const ECCVerifyHandle verify_handle;
SelectParams(CBaseChainParams::REGTEST); ECC_Start();
SelectParams(CBaseChainParams::MAIN);
} }
void test_one_input(const std::vector<uint8_t>& buffer) void test_one_input(const std::vector<uint8_t>& buffer)