fuzz: Add upgrade/downgrade simulation to chanmon_consistency and fix chacha20 build

Author: Atishyy27

fuzz/Cargo.toml

@@ -23,6 +23,7 @@ lightning-invoice = { path = "../lightning-invoice" }
 lightning-liquidity = { path = "../lightning-liquidity" }
 lightning-rapid-gossip-sync = { path = "../lightning-rapid-gossip-sync" }
 lightning-persister = { path = "../lightning-persister", features = ["tokio"]}
+lightning_0_2 = { package = "lightning", version = "0.2.0", features = ["_test_utils"] }
 bech32 = "0.11.0"
 bitcoin = { version = "0.32.4", features = ["secp-lowmemory"] }
 tokio = { version = "~1.35", default-features = false, features = ["rt-multi-thread"] }

fuzz/src/chanmon_consistency.rs

@@ -130,6 +130,28 @@ impl FeeEstimator for FuzzEstimator {
 	}
 }
 
+impl lightning_0_2::chain::chaininterface::FeeEstimator for FuzzEstimator {
+	fn get_est_sat_per_1000_weight(
+		&self, conf_target: lightning_0_2::chain::chaininterface::ConfirmationTarget,
+	) -> u32 {
+		match conf_target {
+            lightning_0_2::chain::chaininterface::ConfirmationTarget::MaximumFeeEstimate
+            | lightning_0_2::chain::chaininterface::ConfirmationTarget::UrgentOnChainSweep => {
+                MAX_FEE
+            },
+            lightning_0_2::chain::chaininterface::ConfirmationTarget::ChannelCloseMinimum
+            | lightning_0_2::chain::chaininterface::ConfirmationTarget::AnchorChannelFee
+            | lightning_0_2::chain::chaininterface::ConfirmationTarget::MinAllowedAnchorChannelRemoteFee
+            | lightning_0_2::chain::chaininterface::ConfirmationTarget::MinAllowedNonAnchorChannelRemoteFee
+            | lightning_0_2::chain::chaininterface::ConfirmationTarget::OutputSpendingFee => 253,
+            lightning_0_2::chain::chaininterface::ConfirmationTarget::NonAnchorChannelFee => {
+                let val = self.ret_val.load(atomic::Ordering::Relaxed);
+                cmp::min(val, MAX_FEE)
+            },
+        }
+	}
+}
+
 impl FuzzEstimator {
 	fn feerate_sat_per_kw(&self) -> FeeRate {
 		let feerate = self.ret_val.load(atomic::Ordering::Acquire);
@@ -182,6 +204,14 @@ impl BroadcasterInterface for TestBroadcaster {
 	}
 }
 
+impl lightning_0_2::chain::chaininterface::BroadcasterInterface for TestBroadcaster {
+	fn broadcast_transactions(&self, txs: &[&bitcoin::Transaction]) {
+		for tx in txs {
+			self.txn_broadcasted.borrow_mut().push((*tx).clone());
+		}
+	}
+}
+
 struct ChainState {
 	blocks: Vec<(Header, Vec<Transaction>)>,
 	confirmed_txids: HashSet<Txid>,
@@ -290,31 +320,31 @@ impl TestChainMonitor {
 			latest_monitors: Mutex::new(new_hash_map()),
 		}
 	}
+	fn do_watch_channel_bytes(
+		&self, channel_id_bytes: [u8; 32], monitor_id: u64, serialized_monitor: Vec<u8>,
+	) {
+		let channel_id = ChannelId(channel_id_bytes);
+		let state = LatestMonitorState {
+			persisted_monitor_id: monitor_id,
+			persisted_monitor: serialized_monitor,
+			pending_monitors: Vec::new(),
+		};
+		let mut latest_monitors = self.latest_monitors.lock().unwrap();
+		if latest_monitors.insert(channel_id, state).is_some() {
+			panic!("Already had monitor pre-watch_channel");
+		}
+	}
 }
-impl chain::Watch<TestChannelSigner> for TestChainMonitor {
+impl lightning::chain::Watch<TestChannelSigner> for TestChainMonitor {
 	fn watch_channel(
 		&self, channel_id: ChannelId, monitor: channelmonitor::ChannelMonitor<TestChannelSigner>,
 	) -> Result<chain::ChannelMonitorUpdateStatus, ()> {
 		let mut ser = VecWriter(Vec::new());
 		monitor.write(&mut ser).unwrap();
 		let monitor_id = monitor.get_latest_update_id();
 		let res = self.chain_monitor.watch_channel(channel_id, monitor);
-		let state = match res {
-			Ok(chain::ChannelMonitorUpdateStatus::Completed) => LatestMonitorState {
-				persisted_monitor_id: monitor_id,
-				persisted_monitor: ser.0,
-				pending_monitors: Vec::new(),
-			},
-			Ok(chain::ChannelMonitorUpdateStatus::InProgress) => LatestMonitorState {
-				persisted_monitor_id: monitor_id,
-				persisted_monitor: Vec::new(),
-				pending_monitors: vec![(monitor_id, ser.0)],
-			},
-			Ok(chain::ChannelMonitorUpdateStatus::UnrecoverableError) => panic!(),
-			Err(()) => panic!(),
-		};
-		if self.latest_monitors.lock().unwrap().insert(channel_id, state).is_some() {
-			panic!("Already had monitor pre-watch_channel");
+		if res == Ok(chain::ChannelMonitorUpdateStatus::Completed) {
+			self.do_watch_channel_bytes(channel_id.0, monitor_id, ser.0);
 		}
 		res
 	}
@@ -330,13 +360,11 @@ impl chain::Watch<TestChannelSigner> for TestChainMonitor {
 			.as_ref()
 			.map(|(_, data)| data)
 			.unwrap_or(&map_entry.persisted_monitor);
-		let deserialized_monitor =
-			<(BlockHash, channelmonitor::ChannelMonitor<TestChannelSigner>)>::read(
-				&mut &latest_monitor_data[..],
-				(&*self.keys, &*self.keys),
-			)
-			.unwrap()
-			.1;
+		let deserialized_monitor: (BlockHash, channelmonitor::ChannelMonitor<TestChannelSigner>) =
+			ReadableArgs::read(&mut &latest_monitor_data[..], (&*self.keys, &*self.keys))
+				.unwrap();
+
+		let deserialized_monitor = deserialized_monitor.1;
 		deserialized_monitor
 			.update_monitor(
 				update,
@@ -368,6 +396,42 @@ impl chain::Watch<TestChannelSigner> for TestChainMonitor {
 	}
 }
 
+impl lightning_0_2::chain::Watch<lightning_0_2::util::test_channel_signer::TestChannelSigner>
+	for TestChainMonitor
+{
+	fn watch_channel(
+		&self, channel_id: lightning_0_2::ln::types::ChannelId,
+		monitor: lightning_0_2::chain::channelmonitor::ChannelMonitor<
+			lightning_0_2::util::test_channel_signer::TestChannelSigner,
+		>,
+	) -> Result<lightning_0_2::chain::ChannelMonitorUpdateStatus, ()> {
+		let mut ser = Vec::new();
+		let block_hash = BlockHash::all_zeros();
+		let monitor_id = monitor.get_latest_update_id();
+		lightning_0_2::util::ser::Writeable::write(&(block_hash, monitor), &mut ser).unwrap();
+		self.do_watch_channel_bytes(channel_id.0, monitor_id, ser);
+
+		Ok(lightning_0_2::chain::ChannelMonitorUpdateStatus::Completed)
+	}
+	fn update_channel(
+		&self, _channel_id: lightning_0_2::ln::types::ChannelId,
+		_update: &lightning_0_2::chain::channelmonitor::ChannelMonitorUpdate,
+	) -> lightning_0_2::chain::ChannelMonitorUpdateStatus {
+		lightning_0_2::chain::ChannelMonitorUpdateStatus::Completed
+	}
+
+	fn release_pending_monitor_events(
+		&self,
+	) -> Vec<(
+		lightning_0_2::chain::transaction::OutPoint,
+		lightning_0_2::ln::types::ChannelId,
+		Vec<lightning_0_2::chain::channelmonitor::MonitorEvent>,
+		PublicKey,
+	)> {
+		Vec::new()
+	}
+}
+
 struct KeyProvider {
 	node_secret: SecretKey,
 	rand_bytes_id: atomic::AtomicU32,
@@ -385,6 +449,33 @@ impl EntropySource for KeyProvider {
 	}
 }
 
+impl lightning_0_2::sign::EntropySource for KeyProvider {
+	fn get_secure_random_bytes(&self) -> [u8; 32] {
+		lightning::sign::EntropySource::get_secure_random_bytes(self)
+	}
+}
+
+impl lightning_0_2::sign::SignerProvider for KeyProvider {
+	type EcdsaSigner = lightning_0_2::util::test_channel_signer::TestChannelSigner;
+
+	fn generate_channel_keys_id(&self, _inbound: bool, _user_channel_id: u128) -> [u8; 32] {
+		let id = self.rand_bytes_id.fetch_add(1, atomic::Ordering::Relaxed) as u8;
+		[id; 32]
+	}
+
+	fn derive_channel_signer(&self, _channel_keys_id: [u8; 32]) -> Self::EcdsaSigner {
+		unreachable!()
+	}
+
+	fn get_destination_script(&self, _channel_keys_id: [u8; 32]) -> Result<bitcoin::ScriptBuf, ()> {
+		unreachable!()
+	}
+
+	fn get_shutdown_scriptpubkey(&self) -> Result<lightning_0_2::ln::script::ShutdownScript, ()> {
+		unreachable!()
+	}
+}
+
 impl NodeSigner for KeyProvider {
 	fn get_node_id(&self, recipient: Recipient) -> Result<PublicKey, ()> {
 		let node_secret = match recipient {
@@ -436,7 +527,9 @@ impl NodeSigner for KeyProvider {
 	fn sign_gossip_message(
 		&self, msg: lightning::ln::msgs::UnsignedGossipMessage,
 	) -> Result<Signature, ()> {
-		let msg_hash = Message::from_digest(Sha256dHash::hash(&msg.encode()[..]).to_byte_array());
+		let mut ser = Vec::new();
+		lightning::util::ser::Writeable::write(&msg, &mut ser).unwrap();
+		let msg_hash = Message::from_digest(Sha256dHash::hash(&ser[..]).to_byte_array());
 		let secp_ctx = Secp256k1::signing_only();
 		Ok(secp_ctx.sign_ecdsa(&msg_hash, &self.node_secret))
 	}
@@ -999,12 +1092,40 @@ pub fn do_test<Out: Output + MaybeSend + MaybeSync>(
 			};
 			// Use a different value of `use_old_mons` if we have another monitor (only for node B)
 			// by shifting `use_old_mons` one in base-3.
-			use_old_mons /= 3;
-			let mon = <(BlockHash, ChannelMonitor<TestChannelSigner>)>::read(
-				&mut &serialized_mon[..],
-				(&**keys, &**keys),
-			)
-			.expect("Failed to read monitor");
+			let mut serialized_mon = serialized_mon;
+			if use_old_mons % 9 == 2 {
+				let old_mon_res: Result<
+					(BlockHash, lightning_0_2::chain::channelmonitor::ChannelMonitor<
+						lightning_0_2::util::test_channel_signer::TestChannelSigner
+					>),
+					_
+				> = lightning_0_2::util::ser::ReadableArgs::read(
+					&mut &serialized_mon[..],
+					(&**keys, &**keys),
+				);
+
+				if let Ok(old_mon) = old_mon_res {
+					let mut out = Vec::new();
+					let (block_hash, old_monitor) = old_mon;
+
+					if lightning_0_2::util::ser::Writeable::write(&(block_hash, old_monitor), &mut out).is_ok() {
+						serialized_mon = out;
+					} else {
+						return;
+					}
+				} else {
+					return;
+				}
+			}
+
+			let mon: (BlockHash, ChannelMonitor<TestChannelSigner>) =
+				match lightning::util::ser::ReadableArgs::read(
+					&mut &serialized_mon[..],
+					(&**keys, &**keys),
+				) {
+					Ok(m) => m,
+					Err(_) => return,
+				};
 			monitors.insert(channel_id, mon.1);
 			// Update the latest `ChannelMonitor` state to match what we just told LDK.
 			prev_state.persisted_monitor = serialized_mon;
@@ -1034,8 +1155,8 @@ pub fn do_test<Out: Output + MaybeSend + MaybeSync>(
 			channel_monitors: monitor_refs,
 		};
 
-		let manager =
-			<(BlockHash, ChanMan)>::read(&mut &ser[..], read_args).expect("Failed to read manager");
+		let manager: (BlockHash, ChanMan) =
+    		lightning::util::ser::ReadableArgs::read(&mut &ser[..], read_args).expect("Failed to read manager");
 		let res = (manager.1, chain_monitor.clone());
 		for (channel_id, mon) in monitors.drain() {
 			assert_eq!(