fuzz: Fix implicit-integer-sign-change in wallet/fees fuzz target

MarcoFalke · MarcoFalke · commit faa05d1965b0 · 2023-06-20T12:05:09.000+02:00
diff --git a/src/wallet/test/fuzz/fees.cpp b/src/wallet/test/fuzz/fees.cpp
@@ -56,7 +56,7 @@ FUZZ_TARGET_INIT(wallet_fees, initialize_setup)
         coin_control.m_feerate = CFeeRate{ConsumeMoney(fuzzed_data_provider, /*max=*/COIN)};
     }
     if (fuzzed_data_provider.ConsumeBool()) {
-        coin_control.m_confirm_target = fuzzed_data_provider.ConsumeIntegral<unsigned int>();
+        coin_control.m_confirm_target = fuzzed_data_provider.ConsumeIntegralInRange<unsigned int>(0, 999'000);
     }
 
     FeeCalculation fee_calculation;

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ FUZZ_TARGET_INIT(wallet_fees, initialize_setup)`
`56`	`56`	`coin_control.m_feerate = CFeeRate{ConsumeMoney(fuzzed_data_provider, /max=/COIN)};`
`57`	`57`	`}`
`58`	`58`	`if (fuzzed_data_provider.ConsumeBool()) {`
`59`		`- coin_control.m_confirm_target = fuzzed_data_provider.ConsumeIntegral<unsigned int>();`
	`59`	`+ coin_control.m_confirm_target = fuzzed_data_provider.ConsumeIntegralInRange<unsigned int>(0, 999'000);`
`60`	`60`	`}`
`61`	`61`
`62`	`62`	`FeeCalculation fee_calculation;`