From 8caf00cd8155d50d634d652fd13bbc409dd49120 Mon Sep 17 00:00:00 2001 From: Alexey Gradoboev Date: Tue, 5 May 2026 14:20:42 +0300 Subject: [PATCH] fuzz: remove splicing blocks in chanmon_consistency_target Removed !cfg!(splicing) blocks to enable full coverage of splicing logic during fuzzing. Verified by running the fuzzer for 15 minutes before and after the changes. Fuzzing Validation (chanmon_consistency_target): Metric Before After Change cov 37824 38686 +862 ft 155421 168662 +13241 corp 1545 1851 +306 The increase in coverage confirms that the splicing logic is now reachable and being actively fuzzed. --- fuzz/Cargo.toml | 1 - fuzz/src/chanmon_consistency.rs | 24 ------------------------ 2 files changed, 25 deletions(-) diff --git a/fuzz/Cargo.toml b/fuzz/Cargo.toml index 8cafdd1f2fb..8da465b3fd0 100644 --- a/fuzz/Cargo.toml +++ b/fuzz/Cargo.toml @@ -43,5 +43,4 @@ check-cfg = [ "cfg(fuzzing)", "cfg(secp256k1_fuzz)", "cfg(hashes_fuzz)", - "cfg(splicing)", ] diff --git a/fuzz/src/chanmon_consistency.rs b/fuzz/src/chanmon_consistency.rs index d678d97918f..2c41ffece1c 100644 --- a/fuzz/src/chanmon_consistency.rs +++ b/fuzz/src/chanmon_consistency.rs @@ -2501,36 +2501,24 @@ pub fn do_test(data: &[u8], out: Out) { }, 0xa0 => { - if !cfg!(splicing) { - test_return!(); - } let cp_node_id = nodes[1].get_our_node_id(); let wallet = WalletSync::new(&wallets[0], Arc::clone(&loggers[0])); let feerate_sat_per_kw = fee_estimators[0].feerate_sat_per_kw(); splice_in(&nodes[0], &cp_node_id, &chan_a_id, &wallet, feerate_sat_per_kw); }, 0xa1 => { - if !cfg!(splicing) { - test_return!(); - } let cp_node_id = nodes[0].get_our_node_id(); let wallet = WalletSync::new(&wallets[1], Arc::clone(&loggers[1])); let feerate_sat_per_kw = fee_estimators[1].feerate_sat_per_kw(); splice_in(&nodes[1], &cp_node_id, &chan_a_id, &wallet, feerate_sat_per_kw); }, 0xa2 => { - if !cfg!(splicing) { - test_return!(); - } let cp_node_id = nodes[2].get_our_node_id(); let wallet = WalletSync::new(&wallets[1], Arc::clone(&loggers[1])); let feerate_sat_per_kw = fee_estimators[1].feerate_sat_per_kw(); splice_in(&nodes[1], &cp_node_id, &chan_b_id, &wallet, feerate_sat_per_kw); }, 0xa3 => { - if !cfg!(splicing) { - test_return!(); - } let cp_node_id = nodes[1].get_our_node_id(); let wallet = WalletSync::new(&wallets[2], Arc::clone(&loggers[2])); let feerate_sat_per_kw = fee_estimators[2].feerate_sat_per_kw(); @@ -2538,36 +2526,24 @@ pub fn do_test(data: &[u8], out: Out) { }, 0xa4 => { - if !cfg!(splicing) { - test_return!(); - } let cp_node_id = nodes[1].get_our_node_id(); let wallet = &wallets[0]; let feerate_sat_per_kw = fee_estimators[0].feerate_sat_per_kw(); splice_out(&nodes[0], &cp_node_id, &chan_a_id, wallet, feerate_sat_per_kw); }, 0xa5 => { - if !cfg!(splicing) { - test_return!(); - } let cp_node_id = nodes[0].get_our_node_id(); let wallet = &wallets[1]; let feerate_sat_per_kw = fee_estimators[1].feerate_sat_per_kw(); splice_out(&nodes[1], &cp_node_id, &chan_a_id, wallet, feerate_sat_per_kw); }, 0xa6 => { - if !cfg!(splicing) { - test_return!(); - } let cp_node_id = nodes[2].get_our_node_id(); let wallet = &wallets[1]; let feerate_sat_per_kw = fee_estimators[1].feerate_sat_per_kw(); splice_out(&nodes[1], &cp_node_id, &chan_b_id, wallet, feerate_sat_per_kw); }, 0xa7 => { - if !cfg!(splicing) { - test_return!(); - } let cp_node_id = nodes[1].get_our_node_id(); let wallet = &wallets[2]; let feerate_sat_per_kw = fee_estimators[2].feerate_sat_per_kw();