Remove KeysInterface trait.
[rust-lightning] / fuzz / src / chanmon_consistency.rs
1 // This file is Copyright its original authors, visible in version control
2 // history.
3 //
4 // This file is licensed under the Apache License, Version 2.0 <LICENSE-APACHE
5 // or http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
6 // <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your option.
7 // You may not use this file except in accordance with one or both of these
8 // licenses.
9
10 //! Test that monitor update failures don't get our channel state out of sync.
11 //! One of the biggest concern with the monitor update failure handling code is that messages
12 //! resent after monitor updating is restored are delivered out-of-order, resulting in
13 //! commitment_signed messages having "invalid signatures".
14 //! To test this we stand up a network of three nodes and read bytes from the fuzz input to denote
15 //! actions such as sending payments, handling events, or changing monitor update return values on
16 //! a per-node basis. This should allow it to find any cases where the ordering of actions results
17 //! in us getting out of sync with ourselves, and, assuming at least one of our recieve- or
18 //! send-side handling is correct, other peers. We consider it a failure if any action results in a
19 //! channel being force-closed.
20
21 use bitcoin::TxMerkleNode;
22 use bitcoin::blockdata::block::BlockHeader;
23 use bitcoin::blockdata::constants::genesis_block;
24 use bitcoin::blockdata::transaction::{Transaction, TxOut};
25 use bitcoin::blockdata::script::{Builder, Script};
26 use bitcoin::blockdata::opcodes;
27 use bitcoin::blockdata::locktime::PackedLockTime;
28 use bitcoin::network::constants::Network;
29
30 use bitcoin::hashes::Hash as TraitImport;
31 use bitcoin::hashes::sha256::Hash as Sha256;
32 use bitcoin::hash_types::{BlockHash, WPubkeyHash};
33
34 use lightning::chain;
35 use lightning::chain::{BestBlock, ChannelMonitorUpdateStatus, chainmonitor, channelmonitor, Confirm, Watch};
36 use lightning::chain::channelmonitor::{ChannelMonitor, MonitorEvent};
37 use lightning::chain::transaction::OutPoint;
38 use lightning::chain::chaininterface::{BroadcasterInterface, ConfirmationTarget, FeeEstimator};
39 use lightning::chain::keysinterface::{KeyMaterial, InMemorySigner, Recipient, EntropySource, NodeSigner, SignerProvider};
40 use lightning::ln::{PaymentHash, PaymentPreimage, PaymentSecret};
41 use lightning::ln::channelmanager::{self, ChainParameters, ChannelDetails, ChannelManager, PaymentSendFailure, ChannelManagerReadArgs, PaymentId};
42 use lightning::ln::channel::FEE_SPIKE_BUFFER_FEE_INCREASE_MULTIPLE;
43 use lightning::ln::msgs::{self, CommitmentUpdate, ChannelMessageHandler, DecodeError, UpdateAddHTLC, Init};
44 use lightning::ln::script::ShutdownScript;
45 use lightning::util::enforcing_trait_impls::{EnforcingSigner, EnforcementState};
46 use lightning::util::errors::APIError;
47 use lightning::util::events;
48 use lightning::util::logger::Logger;
49 use lightning::util::config::UserConfig;
50 use lightning::util::events::MessageSendEventsProvider;
51 use lightning::util::ser::{Readable, ReadableArgs, Writeable, Writer};
52 use lightning::routing::router::{InFlightHtlcs, Route, RouteHop, RouteParameters, Router};
53
54 use crate::utils::test_logger::{self, Output};
55 use crate::utils::test_persister::TestPersister;
56
57 use bitcoin::secp256k1::{PublicKey, SecretKey, Scalar};
58 use bitcoin::secp256k1::ecdh::SharedSecret;
59 use bitcoin::secp256k1::ecdsa::RecoverableSignature;
60 use bitcoin::secp256k1::Secp256k1;
61
62 use std::mem;
63 use std::cmp::{self, Ordering};
64 use std::collections::{HashSet, hash_map, HashMap};
65 use std::sync::{Arc,Mutex};
66 use std::sync::atomic;
67 use std::io::Cursor;
68 use bitcoin::bech32::u5;
69
70 const MAX_FEE: u32 = 10_000;
71 struct FuzzEstimator {
72         ret_val: atomic::AtomicU32,
73 }
74 impl FeeEstimator for FuzzEstimator {
75         fn get_est_sat_per_1000_weight(&self, conf_target: ConfirmationTarget) -> u32 {
76                 // We force-close channels if our counterparty sends us a feerate which is a small multiple
77                 // of our HighPriority fee estimate or smaller than our Background fee estimate. Thus, we
78                 // always return a HighPriority feerate here which is >= the maximum Normal feerate and a
79                 // Background feerate which is <= the minimum Normal feerate.
80                 match conf_target {
81                         ConfirmationTarget::HighPriority => MAX_FEE,
82                         ConfirmationTarget::Background => 253,
83                         ConfirmationTarget::Normal => cmp::min(self.ret_val.load(atomic::Ordering::Acquire), MAX_FEE),
84                 }
85         }
86 }
87
88 struct FuzzRouter {}
89
90 impl Router for FuzzRouter {
91         fn find_route(
92                 &self, _payer: &PublicKey, _params: &RouteParameters, _first_hops: Option<&[&ChannelDetails]>,
93                 _inflight_htlcs: &InFlightHtlcs
94         ) -> Result<Route, msgs::LightningError> {
95                 Err(msgs::LightningError {
96                         err: String::from("Not implemented"),
97                         action: msgs::ErrorAction::IgnoreError
98                 })
99         }
100         fn notify_payment_path_failed(&self, _path: &[&RouteHop], _short_channel_id: u64) {}
101         fn notify_payment_path_successful(&self, _path: &[&RouteHop]) {}
102         fn notify_payment_probe_successful(&self, _path: &[&RouteHop]) {}
103         fn notify_payment_probe_failed(&self, _path: &[&RouteHop], _short_channel_id: u64) {}
104 }
105
106 pub struct TestBroadcaster {}
107 impl BroadcasterInterface for TestBroadcaster {
108         fn broadcast_transaction(&self, _tx: &Transaction) { }
109 }
110
111 pub struct VecWriter(pub Vec<u8>);
112 impl Writer for VecWriter {
113         fn write_all(&mut self, buf: &[u8]) -> Result<(), ::std::io::Error> {
114                 self.0.extend_from_slice(buf);
115                 Ok(())
116         }
117 }
118
119 struct TestChainMonitor {
120         pub logger: Arc<dyn Logger>,
121         pub keys: Arc<KeyProvider>,
122         pub persister: Arc<TestPersister>,
123         pub chain_monitor: Arc<chainmonitor::ChainMonitor<EnforcingSigner, Arc<dyn chain::Filter>, Arc<TestBroadcaster>, Arc<FuzzEstimator>, Arc<dyn Logger>, Arc<TestPersister>>>,
124         // If we reload a node with an old copy of ChannelMonitors, the ChannelManager deserialization
125         // logic will automatically force-close our channels for us (as we don't have an up-to-date
126         // monitor implying we are not able to punish misbehaving counterparties). Because this test
127         // "fails" if we ever force-close a channel, we avoid doing so, always saving the latest
128         // fully-serialized monitor state here, as well as the corresponding update_id.
129         pub latest_monitors: Mutex<HashMap<OutPoint, (u64, Vec<u8>)>>,
130         pub should_update_manager: atomic::AtomicBool,
131 }
132 impl TestChainMonitor {
133         pub fn new(broadcaster: Arc<TestBroadcaster>, logger: Arc<dyn Logger>, feeest: Arc<FuzzEstimator>, persister: Arc<TestPersister>, keys: Arc<KeyProvider>) -> Self {
134                 Self {
135                         chain_monitor: Arc::new(chainmonitor::ChainMonitor::new(None, broadcaster, logger.clone(), feeest, Arc::clone(&persister))),
136                         logger,
137                         keys,
138                         persister,
139                         latest_monitors: Mutex::new(HashMap::new()),
140                         should_update_manager: atomic::AtomicBool::new(false),
141                 }
142         }
143 }
144 impl chain::Watch<EnforcingSigner> for TestChainMonitor {
145         fn watch_channel(&self, funding_txo: OutPoint, monitor: channelmonitor::ChannelMonitor<EnforcingSigner>) -> chain::ChannelMonitorUpdateStatus {
146                 let mut ser = VecWriter(Vec::new());
147                 monitor.write(&mut ser).unwrap();
148                 if let Some(_) = self.latest_monitors.lock().unwrap().insert(funding_txo, (monitor.get_latest_update_id(), ser.0)) {
149                         panic!("Already had monitor pre-watch_channel");
150                 }
151                 self.should_update_manager.store(true, atomic::Ordering::Relaxed);
152                 self.chain_monitor.watch_channel(funding_txo, monitor)
153         }
154
155         fn update_channel(&self, funding_txo: OutPoint, update: channelmonitor::ChannelMonitorUpdate) -> chain::ChannelMonitorUpdateStatus {
156                 let mut map_lock = self.latest_monitors.lock().unwrap();
157                 let mut map_entry = match map_lock.entry(funding_txo) {
158                         hash_map::Entry::Occupied(entry) => entry,
159                         hash_map::Entry::Vacant(_) => panic!("Didn't have monitor on update call"),
160                 };
161                 let deserialized_monitor = <(BlockHash, channelmonitor::ChannelMonitor<EnforcingSigner>)>::
162                         read(&mut Cursor::new(&map_entry.get().1), &*self.keys).unwrap().1;
163                 deserialized_monitor.update_monitor(&update, &&TestBroadcaster{}, &FuzzEstimator { ret_val: atomic::AtomicU32::new(253) }, &self.logger).unwrap();
164                 let mut ser = VecWriter(Vec::new());
165                 deserialized_monitor.write(&mut ser).unwrap();
166                 map_entry.insert((update.update_id, ser.0));
167                 self.should_update_manager.store(true, atomic::Ordering::Relaxed);
168                 self.chain_monitor.update_channel(funding_txo, update)
169         }
170
171         fn release_pending_monitor_events(&self) -> Vec<(OutPoint, Vec<MonitorEvent>, Option<PublicKey>)> {
172                 return self.chain_monitor.release_pending_monitor_events();
173         }
174 }
175
176 struct KeyProvider {
177         node_id: u8,
178         rand_bytes_id: atomic::AtomicU32,
179         enforcement_states: Mutex<HashMap<[u8;32], Arc<Mutex<EnforcementState>>>>,
180 }
181
182 impl EntropySource for KeyProvider {
183         fn get_secure_random_bytes(&self) -> [u8; 32] {
184                 let id = self.rand_bytes_id.fetch_add(1, atomic::Ordering::Relaxed);
185                 let mut res = [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 11, self.node_id];
186                 res[30-4..30].copy_from_slice(&id.to_le_bytes());
187                 res
188         }
189 }
190
191 impl NodeSigner for KeyProvider {
192         fn get_node_secret(&self, _recipient: Recipient) -> Result<SecretKey, ()> {
193                 Ok(SecretKey::from_slice(&[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, self.node_id]).unwrap())
194         }
195
196         fn get_node_id(&self, recipient: Recipient) -> Result<PublicKey, ()> {
197                 let secp_ctx = Secp256k1::signing_only();
198                 Ok(PublicKey::from_secret_key(&secp_ctx, &self.get_node_secret(recipient)?))
199         }
200
201         fn ecdh(&self, recipient: Recipient, other_key: &PublicKey, tweak: Option<&Scalar>) -> Result<SharedSecret, ()> {
202                 let mut node_secret = self.get_node_secret(recipient)?;
203                 if let Some(tweak) = tweak {
204                         node_secret = node_secret.mul_tweak(tweak).unwrap();
205                 }
206                 Ok(SharedSecret::new(other_key, &node_secret))
207         }
208
209         fn get_inbound_payment_key_material(&self) -> KeyMaterial {
210                 KeyMaterial([0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, self.node_id])
211         }
212
213         fn sign_invoice(&self, _hrp_bytes: &[u8], _invoice_data: &[u5], _recipient: Recipient) -> Result<RecoverableSignature, ()> {
214                 unreachable!()
215         }
216 }
217
218 impl SignerProvider for KeyProvider {
219         type Signer = EnforcingSigner;
220
221         fn generate_channel_keys_id(&self, _inbound: bool, _channel_value_satoshis: u64, _user_channel_id: u128) -> [u8; 32] {
222                 let id = self.rand_bytes_id.fetch_add(1, atomic::Ordering::Relaxed) as u8;
223                 [id; 32]
224         }
225
226         fn derive_channel_signer(&self, channel_value_satoshis: u64, channel_keys_id: [u8; 32]) -> Self::Signer {
227                 let secp_ctx = Secp256k1::signing_only();
228                 let id = channel_keys_id[0];
229                 let keys = InMemorySigner::new(
230                         &secp_ctx,
231                         self.get_node_secret(Recipient::Node).unwrap(),
232                         SecretKey::from_slice(&[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 4, self.node_id]).unwrap(),
233                         SecretKey::from_slice(&[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 5, self.node_id]).unwrap(),
234                         SecretKey::from_slice(&[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 6, self.node_id]).unwrap(),
235                         SecretKey::from_slice(&[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 7, self.node_id]).unwrap(),
236                         SecretKey::from_slice(&[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 8, self.node_id]).unwrap(),
237                         [id, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 9, self.node_id],
238                         channel_value_satoshis,
239                         channel_keys_id,
240                 );
241                 let revoked_commitment = self.make_enforcement_state_cell(keys.commitment_seed);
242                 EnforcingSigner::new_with_revoked(keys, revoked_commitment, false)
243         }
244
245         fn read_chan_signer(&self, buffer: &[u8]) -> Result<Self::Signer, DecodeError> {
246                 let mut reader = std::io::Cursor::new(buffer);
247
248                 let inner: InMemorySigner = ReadableArgs::read(&mut reader, self.get_node_secret(Recipient::Node).unwrap())?;
249                 let state = self.make_enforcement_state_cell(inner.commitment_seed);
250
251                 Ok(EnforcingSigner {
252                         inner,
253                         state,
254                         disable_revocation_policy_check: false,
255                 })
256         }
257
258         fn get_destination_script(&self) -> Script {
259                 let secp_ctx = Secp256k1::signing_only();
260                 let channel_monitor_claim_key = SecretKey::from_slice(&[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2, self.node_id]).unwrap();
261                 let our_channel_monitor_claim_key_hash = WPubkeyHash::hash(&PublicKey::from_secret_key(&secp_ctx, &channel_monitor_claim_key).serialize());
262                 Builder::new().push_opcode(opcodes::all::OP_PUSHBYTES_0).push_slice(&our_channel_monitor_claim_key_hash[..]).into_script()
263         }
264
265         fn get_shutdown_scriptpubkey(&self) -> ShutdownScript {
266                 let secp_ctx = Secp256k1::signing_only();
267                 let secret_key = SecretKey::from_slice(&[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 3, self.node_id]).unwrap();
268                 let pubkey_hash = WPubkeyHash::hash(&PublicKey::from_secret_key(&secp_ctx, &secret_key).serialize());
269                 ShutdownScript::new_p2wpkh(&pubkey_hash)
270         }
271 }
272
273 impl KeyProvider {
274         fn make_enforcement_state_cell(&self, commitment_seed: [u8; 32]) -> Arc<Mutex<EnforcementState>> {
275                 let mut revoked_commitments = self.enforcement_states.lock().unwrap();
276                 if !revoked_commitments.contains_key(&commitment_seed) {
277                         revoked_commitments.insert(commitment_seed, Arc::new(Mutex::new(EnforcementState::new())));
278                 }
279                 let cell = revoked_commitments.get(&commitment_seed).unwrap();
280                 Arc::clone(cell)
281         }
282 }
283
284 #[inline]
285 fn check_api_err(api_err: APIError) {
286         match api_err {
287                 APIError::APIMisuseError { .. } => panic!("We can't misuse the API"),
288                 APIError::FeeRateTooHigh { .. } => panic!("We can't send too much fee?"),
289                 APIError::InvalidRoute { .. } => panic!("Our routes should work"),
290                 APIError::ChannelUnavailable { err } => {
291                         // Test the error against a list of errors we can hit, and reject
292                         // all others. If you hit this panic, the list of acceptable errors
293                         // is probably just stale and you should add new messages here.
294                         match err.as_str() {
295                                 "Peer for first hop currently disconnected/pending monitor update!" => {},
296                                 _ if err.starts_with("Cannot push more than their max accepted HTLCs ") => {},
297                                 _ if err.starts_with("Cannot send value that would put us over the max HTLC value in flight our peer will accept ") => {},
298                                 _ if err.starts_with("Cannot send value that would put our balance under counterparty-announced channel reserve value") => {},
299                                 _ if err.starts_with("Cannot send value that would put counterparty balance under holder-announced channel reserve value") => {},
300                                 _ if err.starts_with("Cannot send value that would overdraw remaining funds.") => {},
301                                 _ if err.starts_with("Cannot send value that would not leave enough to pay for fees.") => {},
302                                 _ if err.starts_with("Cannot send value that would put our exposure to dust HTLCs at") => {},
303                                 _ => panic!("{}", err),
304                         }
305                 },
306                 APIError::MonitorUpdateInProgress => {
307                         // We can (obviously) temp-fail a monitor update
308                 },
309                 APIError::IncompatibleShutdownScript { .. } => panic!("Cannot send an incompatible shutdown script"),
310         }
311 }
312 #[inline]
313 fn check_payment_err(send_err: PaymentSendFailure) {
314         match send_err {
315                 PaymentSendFailure::ParameterError(api_err) => check_api_err(api_err),
316                 PaymentSendFailure::PathParameterError(per_path_results) => {
317                         for res in per_path_results { if let Err(api_err) = res { check_api_err(api_err); } }
318                 },
319                 PaymentSendFailure::AllFailedResendSafe(per_path_results) => {
320                         for api_err in per_path_results { check_api_err(api_err); }
321                 },
322                 PaymentSendFailure::PartialFailure { results, .. } => {
323                         for res in results { if let Err(api_err) = res { check_api_err(api_err); } }
324                 },
325                 PaymentSendFailure::DuplicatePayment => panic!(),
326         }
327 }
328
329 type ChanMan<'a> = ChannelManager<Arc<TestChainMonitor>, Arc<TestBroadcaster>, Arc<KeyProvider>, Arc<FuzzEstimator>, &'a FuzzRouter, Arc<dyn Logger>>;
330
331 #[inline]
332 fn get_payment_secret_hash(dest: &ChanMan, payment_id: &mut u8) -> Option<(PaymentSecret, PaymentHash)> {
333         let mut payment_hash;
334         for _ in 0..256 {
335                 payment_hash = PaymentHash(Sha256::hash(&[*payment_id; 1]).into_inner());
336                 if let Ok(payment_secret) = dest.create_inbound_payment_for_hash(payment_hash, None, 3600) {
337                         return Some((payment_secret, payment_hash));
338                 }
339                 *payment_id = payment_id.wrapping_add(1);
340         }
341         None
342 }
343
344 #[inline]
345 fn send_payment(source: &ChanMan, dest: &ChanMan, dest_chan_id: u64, amt: u64, payment_id: &mut u8, payment_idx: &mut u64) -> bool {
346         let (payment_secret, payment_hash) =
347                 if let Some((secret, hash)) = get_payment_secret_hash(dest, payment_id) { (secret, hash) } else { return true; };
348         let mut payment_id = [0; 32];
349         payment_id[0..8].copy_from_slice(&payment_idx.to_ne_bytes());
350         *payment_idx += 1;
351         if let Err(err) = source.send_payment(&Route {
352                 paths: vec![vec![RouteHop {
353                         pubkey: dest.get_our_node_id(),
354                         node_features: channelmanager::provided_node_features(),
355                         short_channel_id: dest_chan_id,
356                         channel_features: channelmanager::provided_channel_features(),
357                         fee_msat: amt,
358                         cltv_expiry_delta: 200,
359                 }]],
360                 payment_params: None,
361         }, payment_hash, &Some(payment_secret), PaymentId(payment_id)) {
362                 check_payment_err(err);
363                 false
364         } else { true }
365 }
366 #[inline]
367 fn send_hop_payment(source: &ChanMan, middle: &ChanMan, middle_chan_id: u64, dest: &ChanMan, dest_chan_id: u64, amt: u64, payment_id: &mut u8, payment_idx: &mut u64) -> bool {
368         let (payment_secret, payment_hash) =
369                 if let Some((secret, hash)) = get_payment_secret_hash(dest, payment_id) { (secret, hash) } else { return true; };
370         let mut payment_id = [0; 32];
371         payment_id[0..8].copy_from_slice(&payment_idx.to_ne_bytes());
372         *payment_idx += 1;
373         if let Err(err) = source.send_payment(&Route {
374                 paths: vec![vec![RouteHop {
375                         pubkey: middle.get_our_node_id(),
376                         node_features: channelmanager::provided_node_features(),
377                         short_channel_id: middle_chan_id,
378                         channel_features: channelmanager::provided_channel_features(),
379                         fee_msat: 50000,
380                         cltv_expiry_delta: 100,
381                 },RouteHop {
382                         pubkey: dest.get_our_node_id(),
383                         node_features: channelmanager::provided_node_features(),
384                         short_channel_id: dest_chan_id,
385                         channel_features: channelmanager::provided_channel_features(),
386                         fee_msat: amt,
387                         cltv_expiry_delta: 200,
388                 }]],
389                 payment_params: None,
390         }, payment_hash, &Some(payment_secret), PaymentId(payment_id)) {
391                 check_payment_err(err);
392                 false
393         } else { true }
394 }
395
396 #[inline]
397 pub fn do_test<Out: Output>(data: &[u8], underlying_out: Out) {
398         let out = SearchingOutput::new(underlying_out);
399         let broadcast = Arc::new(TestBroadcaster{});
400         let router = FuzzRouter {};
401
402         macro_rules! make_node {
403                 ($node_id: expr, $fee_estimator: expr) => { {
404                         let logger: Arc<dyn Logger> = Arc::new(test_logger::TestLogger::new($node_id.to_string(), out.clone()));
405                         let keys_manager = Arc::new(KeyProvider { node_id: $node_id, rand_bytes_id: atomic::AtomicU32::new(0), enforcement_states: Mutex::new(HashMap::new()) });
406                         let monitor = Arc::new(TestChainMonitor::new(broadcast.clone(), logger.clone(), $fee_estimator.clone(),
407                                 Arc::new(TestPersister {
408                                         update_ret: Mutex::new(ChannelMonitorUpdateStatus::Completed)
409                                 }), Arc::clone(&keys_manager)));
410
411                         let mut config = UserConfig::default();
412                         config.channel_config.forwarding_fee_proportional_millionths = 0;
413                         config.channel_handshake_config.announced_channel = true;
414                         let network = Network::Bitcoin;
415                         let params = ChainParameters {
416                                 network,
417                                 best_block: BestBlock::from_genesis(network),
418                         };
419                         (ChannelManager::new($fee_estimator.clone(), monitor.clone(), broadcast.clone(), &router, Arc::clone(&logger), keys_manager.clone(), config, params),
420                         monitor, keys_manager)
421                 } }
422         }
423
424         macro_rules! reload_node {
425                 ($ser: expr, $node_id: expr, $old_monitors: expr, $keys_manager: expr, $fee_estimator: expr) => { {
426                     let keys_manager = Arc::clone(& $keys_manager);
427                         let logger: Arc<dyn Logger> = Arc::new(test_logger::TestLogger::new($node_id.to_string(), out.clone()));
428                         let chain_monitor = Arc::new(TestChainMonitor::new(broadcast.clone(), logger.clone(), $fee_estimator.clone(),
429                                 Arc::new(TestPersister {
430                                         update_ret: Mutex::new(ChannelMonitorUpdateStatus::Completed)
431                                 }), Arc::clone(& $keys_manager)));
432
433                         let mut config = UserConfig::default();
434                         config.channel_config.forwarding_fee_proportional_millionths = 0;
435                         config.channel_handshake_config.announced_channel = true;
436
437                         let mut monitors = HashMap::new();
438                         let mut old_monitors = $old_monitors.latest_monitors.lock().unwrap();
439                         for (outpoint, (update_id, monitor_ser)) in old_monitors.drain() {
440                                 monitors.insert(outpoint, <(BlockHash, ChannelMonitor<EnforcingSigner>)>::read(&mut Cursor::new(&monitor_ser), &*$keys_manager).expect("Failed to read monitor").1);
441                                 chain_monitor.latest_monitors.lock().unwrap().insert(outpoint, (update_id, monitor_ser));
442                         }
443                         let mut monitor_refs = HashMap::new();
444                         for (outpoint, monitor) in monitors.iter_mut() {
445                                 monitor_refs.insert(*outpoint, monitor);
446                         }
447
448                         let read_args = ChannelManagerReadArgs {
449                                 keys_manager,
450                                 fee_estimator: $fee_estimator.clone(),
451                                 chain_monitor: chain_monitor.clone(),
452                                 tx_broadcaster: broadcast.clone(),
453                                 router: &router,
454                                 logger,
455                                 default_config: config,
456                                 channel_monitors: monitor_refs,
457                         };
458
459                         let res = (<(BlockHash, ChanMan)>::read(&mut Cursor::new(&$ser.0), read_args).expect("Failed to read manager").1, chain_monitor.clone());
460                         for (funding_txo, mon) in monitors.drain() {
461                                 assert_eq!(chain_monitor.chain_monitor.watch_channel(funding_txo, mon),
462                                         ChannelMonitorUpdateStatus::Completed);
463                         }
464                         res
465                 } }
466         }
467
468         let mut channel_txn = Vec::new();
469         macro_rules! make_channel {
470                 ($source: expr, $dest: expr, $chan_id: expr) => { {
471                         $source.peer_connected(&$dest.get_our_node_id(), &Init { features: channelmanager::provided_init_features(), remote_network_address: None }).unwrap();
472                         $dest.peer_connected(&$source.get_our_node_id(), &Init { features: channelmanager::provided_init_features(), remote_network_address: None }).unwrap();
473
474                         $source.create_channel($dest.get_our_node_id(), 100_000, 42, 0, None).unwrap();
475                         let open_channel = {
476                                 let events = $source.get_and_clear_pending_msg_events();
477                                 assert_eq!(events.len(), 1);
478                                 if let events::MessageSendEvent::SendOpenChannel { ref msg, .. } = events[0] {
479                                         msg.clone()
480                                 } else { panic!("Wrong event type"); }
481                         };
482
483                         $dest.handle_open_channel(&$source.get_our_node_id(), channelmanager::provided_init_features(), &open_channel);
484                         let accept_channel = {
485                                 let events = $dest.get_and_clear_pending_msg_events();
486                                 assert_eq!(events.len(), 1);
487                                 if let events::MessageSendEvent::SendAcceptChannel { ref msg, .. } = events[0] {
488                                         msg.clone()
489                                 } else { panic!("Wrong event type"); }
490                         };
491
492                         $source.handle_accept_channel(&$dest.get_our_node_id(), channelmanager::provided_init_features(), &accept_channel);
493                         let funding_output;
494                         {
495                                 let events = $source.get_and_clear_pending_events();
496                                 assert_eq!(events.len(), 1);
497                                 if let events::Event::FundingGenerationReady { ref temporary_channel_id, ref channel_value_satoshis, ref output_script, .. } = events[0] {
498                                         let tx = Transaction { version: $chan_id, lock_time: PackedLockTime::ZERO, input: Vec::new(), output: vec![TxOut {
499                                                 value: *channel_value_satoshis, script_pubkey: output_script.clone(),
500                                         }]};
501                                         funding_output = OutPoint { txid: tx.txid(), index: 0 };
502                                         $source.funding_transaction_generated(&temporary_channel_id, &$dest.get_our_node_id(), tx.clone()).unwrap();
503                                         channel_txn.push(tx);
504                                 } else { panic!("Wrong event type"); }
505                         }
506
507                         let funding_created = {
508                                 let events = $source.get_and_clear_pending_msg_events();
509                                 assert_eq!(events.len(), 1);
510                                 if let events::MessageSendEvent::SendFundingCreated { ref msg, .. } = events[0] {
511                                         msg.clone()
512                                 } else { panic!("Wrong event type"); }
513                         };
514                         $dest.handle_funding_created(&$source.get_our_node_id(), &funding_created);
515
516                         let funding_signed = {
517                                 let events = $dest.get_and_clear_pending_msg_events();
518                                 assert_eq!(events.len(), 1);
519                                 if let events::MessageSendEvent::SendFundingSigned { ref msg, .. } = events[0] {
520                                         msg.clone()
521                                 } else { panic!("Wrong event type"); }
522                         };
523                         $source.handle_funding_signed(&$dest.get_our_node_id(), &funding_signed);
524
525                         funding_output
526                 } }
527         }
528
529         macro_rules! confirm_txn {
530                 ($node: expr) => { {
531                         let chain_hash = genesis_block(Network::Bitcoin).block_hash();
532                         let mut header = BlockHeader { version: 0x20000000, prev_blockhash: chain_hash, merkle_root: TxMerkleNode::all_zeros(), time: 42, bits: 42, nonce: 42 };
533                         let txdata: Vec<_> = channel_txn.iter().enumerate().map(|(i, tx)| (i + 1, tx)).collect();
534                         $node.transactions_confirmed(&header, &txdata, 1);
535                         for _ in 2..100 {
536                                 header = BlockHeader { version: 0x20000000, prev_blockhash: header.block_hash(), merkle_root: TxMerkleNode::all_zeros(), time: 42, bits: 42, nonce: 42 };
537                         }
538                         $node.best_block_updated(&header, 99);
539                 } }
540         }
541
542         macro_rules! lock_fundings {
543                 ($nodes: expr) => { {
544                         let mut node_events = Vec::new();
545                         for node in $nodes.iter() {
546                                 node_events.push(node.get_and_clear_pending_msg_events());
547                         }
548                         for (idx, node_event) in node_events.iter().enumerate() {
549                                 for event in node_event {
550                                         if let events::MessageSendEvent::SendChannelReady { ref node_id, ref msg } = event {
551                                                 for node in $nodes.iter() {
552                                                         if node.get_our_node_id() == *node_id {
553                                                                 node.handle_channel_ready(&$nodes[idx].get_our_node_id(), msg);
554                                                         }
555                                                 }
556                                         } else { panic!("Wrong event type"); }
557                                 }
558                         }
559
560                         for node in $nodes.iter() {
561                                 let events = node.get_and_clear_pending_msg_events();
562                                 for event in events {
563                                         if let events::MessageSendEvent::SendAnnouncementSignatures { .. } = event {
564                                         } else { panic!("Wrong event type"); }
565                                 }
566                         }
567                 } }
568         }
569
570         let fee_est_a = Arc::new(FuzzEstimator { ret_val: atomic::AtomicU32::new(253) });
571         let mut last_htlc_clear_fee_a =  253;
572         let fee_est_b = Arc::new(FuzzEstimator { ret_val: atomic::AtomicU32::new(253) });
573         let mut last_htlc_clear_fee_b =  253;
574         let fee_est_c = Arc::new(FuzzEstimator { ret_val: atomic::AtomicU32::new(253) });
575         let mut last_htlc_clear_fee_c =  253;
576
577         // 3 nodes is enough to hit all the possible cases, notably unknown-source-unknown-dest
578         // forwarding.
579         let (node_a, mut monitor_a, keys_manager_a) = make_node!(0, fee_est_a);
580         let (node_b, mut monitor_b, keys_manager_b) = make_node!(1, fee_est_b);
581         let (node_c, mut monitor_c, keys_manager_c) = make_node!(2, fee_est_c);
582
583         let mut nodes = [node_a, node_b, node_c];
584
585         let chan_1_funding = make_channel!(nodes[0], nodes[1], 0);
586         let chan_2_funding = make_channel!(nodes[1], nodes[2], 1);
587
588         for node in nodes.iter() {
589                 confirm_txn!(node);
590         }
591
592         lock_fundings!(nodes);
593
594         let chan_a = nodes[0].list_usable_channels()[0].short_channel_id.unwrap();
595         let chan_b = nodes[2].list_usable_channels()[0].short_channel_id.unwrap();
596
597         let mut payment_id: u8 = 0;
598         let mut payment_idx: u64 = 0;
599
600         let mut chan_a_disconnected = false;
601         let mut chan_b_disconnected = false;
602         let mut ab_events = Vec::new();
603         let mut ba_events = Vec::new();
604         let mut bc_events = Vec::new();
605         let mut cb_events = Vec::new();
606
607         let mut node_a_ser = VecWriter(Vec::new());
608         nodes[0].write(&mut node_a_ser).unwrap();
609         let mut node_b_ser = VecWriter(Vec::new());
610         nodes[1].write(&mut node_b_ser).unwrap();
611         let mut node_c_ser = VecWriter(Vec::new());
612         nodes[2].write(&mut node_c_ser).unwrap();
613
614         macro_rules! test_return {
615                 () => { {
616                         assert_eq!(nodes[0].list_channels().len(), 1);
617                         assert_eq!(nodes[1].list_channels().len(), 2);
618                         assert_eq!(nodes[2].list_channels().len(), 1);
619                         return;
620                 } }
621         }
622
623         let mut read_pos = 0;
624         macro_rules! get_slice {
625                 ($len: expr) => {
626                         {
627                                 let slice_len = $len as usize;
628                                 if data.len() < read_pos + slice_len {
629                                         test_return!();
630                                 }
631                                 read_pos += slice_len;
632                                 &data[read_pos - slice_len..read_pos]
633                         }
634                 }
635         }
636
637         loop {
638                 // Push any events from Node B onto ba_events and bc_events
639                 macro_rules! push_excess_b_events {
640                         ($excess_events: expr, $expect_drop_node: expr) => { {
641                                 let a_id = nodes[0].get_our_node_id();
642                                 let expect_drop_node: Option<usize> = $expect_drop_node;
643                                 let expect_drop_id = if let Some(id) = expect_drop_node { Some(nodes[id].get_our_node_id()) } else { None };
644                                 for event in $excess_events {
645                                         let push_a = match event {
646                                                 events::MessageSendEvent::UpdateHTLCs { ref node_id, .. } => {
647                                                         if Some(*node_id) == expect_drop_id { panic!("peer_disconnected should drop msgs bound for the disconnected peer"); }
648                                                         *node_id == a_id
649                                                 },
650                                                 events::MessageSendEvent::SendRevokeAndACK { ref node_id, .. } => {
651                                                         if Some(*node_id) == expect_drop_id { panic!("peer_disconnected should drop msgs bound for the disconnected peer"); }
652                                                         *node_id == a_id
653                                                 },
654                                                 events::MessageSendEvent::SendChannelReestablish { ref node_id, .. } => {
655                                                         if Some(*node_id) == expect_drop_id { panic!("peer_disconnected should drop msgs bound for the disconnected peer"); }
656                                                         *node_id == a_id
657                                                 },
658                                                 events::MessageSendEvent::SendChannelReady { .. } => continue,
659                                                 events::MessageSendEvent::SendAnnouncementSignatures { .. } => continue,
660                                                 events::MessageSendEvent::SendChannelUpdate { ref node_id, ref msg } => {
661                                                         assert_eq!(msg.contents.flags & 2, 0); // The disable bit must never be set!
662                                                         if Some(*node_id) == expect_drop_id { panic!("peer_disconnected should drop msgs bound for the disconnected peer"); }
663                                                         *node_id == a_id
664                                                 },
665                                                 _ => panic!("Unhandled message event {:?}", event),
666                                         };
667                                         if push_a { ba_events.push(event); } else { bc_events.push(event); }
668                                 }
669                         } }
670                 }
671
672                 // While delivering messages, we select across three possible message selection processes
673                 // to ensure we get as much coverage as possible. See the individual enum variants for more
674                 // details.
675                 #[derive(PartialEq)]
676                 enum ProcessMessages {
677                         /// Deliver all available messages, including fetching any new messages from
678                         /// `get_and_clear_pending_msg_events()` (which may have side effects).
679                         AllMessages,
680                         /// Call `get_and_clear_pending_msg_events()` first, and then deliver up to one
681                         /// message (which may already be queued).
682                         OneMessage,
683                         /// Deliver up to one already-queued message. This avoids any potential side-effects
684                         /// of `get_and_clear_pending_msg_events()` (eg freeing the HTLC holding cell), which
685                         /// provides potentially more coverage.
686                         OnePendingMessage,
687                 }
688
689                 macro_rules! process_msg_events {
690                         ($node: expr, $corrupt_forward: expr, $limit_events: expr) => { {
691                                 let mut events = if $node == 1 {
692                                         let mut new_events = Vec::new();
693                                         mem::swap(&mut new_events, &mut ba_events);
694                                         new_events.extend_from_slice(&bc_events[..]);
695                                         bc_events.clear();
696                                         new_events
697                                 } else if $node == 0 {
698                                         let mut new_events = Vec::new();
699                                         mem::swap(&mut new_events, &mut ab_events);
700                                         new_events
701                                 } else {
702                                         let mut new_events = Vec::new();
703                                         mem::swap(&mut new_events, &mut cb_events);
704                                         new_events
705                                 };
706                                 let mut new_events = Vec::new();
707                                 if $limit_events != ProcessMessages::OnePendingMessage {
708                                         new_events = nodes[$node].get_and_clear_pending_msg_events();
709                                 }
710                                 let mut had_events = false;
711                                 let mut events_iter = events.drain(..).chain(new_events.drain(..));
712                                 let mut extra_ev = None;
713                                 for event in &mut events_iter {
714                                         had_events = true;
715                                         match event {
716                                                 events::MessageSendEvent::UpdateHTLCs { node_id, updates: CommitmentUpdate { update_add_htlcs, update_fail_htlcs, update_fulfill_htlcs, update_fail_malformed_htlcs, update_fee, commitment_signed } } => {
717                                                         for (idx, dest) in nodes.iter().enumerate() {
718                                                                 if dest.get_our_node_id() == node_id {
719                                                                         for update_add in update_add_htlcs.iter() {
720                                                                                 out.locked_write(format!("Delivering update_add_htlc to node {}.\n", idx).as_bytes());
721                                                                                 if !$corrupt_forward {
722                                                                                         dest.handle_update_add_htlc(&nodes[$node].get_our_node_id(), update_add);
723                                                                                 } else {
724                                                                                         // Corrupt the update_add_htlc message so that its HMAC
725                                                                                         // check will fail and we generate a
726                                                                                         // update_fail_malformed_htlc instead of an
727                                                                                         // update_fail_htlc as we do when we reject a payment.
728                                                                                         let mut msg_ser = update_add.encode();
729                                                                                         msg_ser[1000] ^= 0xff;
730                                                                                         let new_msg = UpdateAddHTLC::read(&mut Cursor::new(&msg_ser)).unwrap();
731                                                                                         dest.handle_update_add_htlc(&nodes[$node].get_our_node_id(), &new_msg);
732                                                                                 }
733                                                                         }
734                                                                         for update_fulfill in update_fulfill_htlcs.iter() {
735                                                                                 out.locked_write(format!("Delivering update_fulfill_htlc to node {}.\n", idx).as_bytes());
736                                                                                 dest.handle_update_fulfill_htlc(&nodes[$node].get_our_node_id(), update_fulfill);
737                                                                         }
738                                                                         for update_fail in update_fail_htlcs.iter() {
739                                                                                 out.locked_write(format!("Delivering update_fail_htlc to node {}.\n", idx).as_bytes());
740                                                                                 dest.handle_update_fail_htlc(&nodes[$node].get_our_node_id(), update_fail);
741                                                                         }
742                                                                         for update_fail_malformed in update_fail_malformed_htlcs.iter() {
743                                                                                 out.locked_write(format!("Delivering update_fail_malformed_htlc to node {}.\n", idx).as_bytes());
744                                                                                 dest.handle_update_fail_malformed_htlc(&nodes[$node].get_our_node_id(), update_fail_malformed);
745                                                                         }
746                                                                         if let Some(msg) = update_fee {
747                                                                                 out.locked_write(format!("Delivering update_fee to node {}.\n", idx).as_bytes());
748                                                                                 dest.handle_update_fee(&nodes[$node].get_our_node_id(), &msg);
749                                                                         }
750                                                                         let processed_change = !update_add_htlcs.is_empty() || !update_fulfill_htlcs.is_empty() ||
751                                                                                 !update_fail_htlcs.is_empty() || !update_fail_malformed_htlcs.is_empty();
752                                                                         if $limit_events != ProcessMessages::AllMessages && processed_change {
753                                                                                 // If we only want to process some messages, don't deliver the CS until later.
754                                                                                 extra_ev = Some(events::MessageSendEvent::UpdateHTLCs { node_id, updates: CommitmentUpdate {
755                                                                                         update_add_htlcs: Vec::new(),
756                                                                                         update_fail_htlcs: Vec::new(),
757                                                                                         update_fulfill_htlcs: Vec::new(),
758                                                                                         update_fail_malformed_htlcs: Vec::new(),
759                                                                                         update_fee: None,
760                                                                                         commitment_signed
761                                                                                 } });
762                                                                                 break;
763                                                                         }
764                                                                         out.locked_write(format!("Delivering commitment_signed to node {}.\n", idx).as_bytes());
765                                                                         dest.handle_commitment_signed(&nodes[$node].get_our_node_id(), &commitment_signed);
766                                                                         break;
767                                                                 }
768                                                         }
769                                                 },
770                                                 events::MessageSendEvent::SendRevokeAndACK { ref node_id, ref msg } => {
771                                                         for (idx, dest) in nodes.iter().enumerate() {
772                                                                 if dest.get_our_node_id() == *node_id {
773                                                                         out.locked_write(format!("Delivering revoke_and_ack to node {}.\n", idx).as_bytes());
774                                                                         dest.handle_revoke_and_ack(&nodes[$node].get_our_node_id(), msg);
775                                                                 }
776                                                         }
777                                                 },
778                                                 events::MessageSendEvent::SendChannelReestablish { ref node_id, ref msg } => {
779                                                         for (idx, dest) in nodes.iter().enumerate() {
780                                                                 if dest.get_our_node_id() == *node_id {
781                                                                         out.locked_write(format!("Delivering channel_reestablish to node {}.\n", idx).as_bytes());
782                                                                         dest.handle_channel_reestablish(&nodes[$node].get_our_node_id(), msg);
783                                                                 }
784                                                         }
785                                                 },
786                                                 events::MessageSendEvent::SendChannelReady { .. } => {
787                                                         // Can be generated as a reestablish response
788                                                 },
789                                                 events::MessageSendEvent::SendAnnouncementSignatures { .. } => {
790                                                         // Can be generated as a reestablish response
791                                                 },
792                                                 events::MessageSendEvent::SendChannelUpdate { ref msg, .. } => {
793                                                         // When we reconnect we will resend a channel_update to make sure our
794                                                         // counterparty has the latest parameters for receiving payments
795                                                         // through us. We do, however, check that the message does not include
796                                                         // the "disabled" bit, as we should never ever have a channel which is
797                                                         // disabled when we send such an update (or it may indicate channel
798                                                         // force-close which we should detect as an error).
799                                                         assert_eq!(msg.contents.flags & 2, 0);
800                                                 },
801                                                 _ => if out.may_fail.load(atomic::Ordering::Acquire) {
802                                                         return;
803                                                 } else {
804                                                         panic!("Unhandled message event {:?}", event)
805                                                 },
806                                         }
807                                         if $limit_events != ProcessMessages::AllMessages {
808                                                 break;
809                                         }
810                                 }
811                                 if $node == 1 {
812                                         push_excess_b_events!(extra_ev.into_iter().chain(events_iter), None);
813                                 } else if $node == 0 {
814                                         if let Some(ev) = extra_ev { ab_events.push(ev); }
815                                         for event in events_iter { ab_events.push(event); }
816                                 } else {
817                                         if let Some(ev) = extra_ev { cb_events.push(ev); }
818                                         for event in events_iter { cb_events.push(event); }
819                                 }
820                                 had_events
821                         } }
822                 }
823
824                 macro_rules! drain_msg_events_on_disconnect {
825                         ($counterparty_id: expr) => { {
826                                 if $counterparty_id == 0 {
827                                         for event in nodes[0].get_and_clear_pending_msg_events() {
828                                                 match event {
829                                                         events::MessageSendEvent::UpdateHTLCs { .. } => {},
830                                                         events::MessageSendEvent::SendRevokeAndACK { .. } => {},
831                                                         events::MessageSendEvent::SendChannelReestablish { .. } => {},
832                                                         events::MessageSendEvent::SendChannelReady { .. } => {},
833                                                         events::MessageSendEvent::SendAnnouncementSignatures { .. } => {},
834                                                         events::MessageSendEvent::SendChannelUpdate { ref msg, .. } => {
835                                                                 assert_eq!(msg.contents.flags & 2, 0); // The disable bit must never be set!
836                                                         },
837                                                         _ => if out.may_fail.load(atomic::Ordering::Acquire) {
838                                                                 return;
839                                                         } else {
840                                                                 panic!("Unhandled message event")
841                                                         },
842                                                 }
843                                         }
844                                         push_excess_b_events!(nodes[1].get_and_clear_pending_msg_events().drain(..), Some(0));
845                                         ab_events.clear();
846                                         ba_events.clear();
847                                 } else {
848                                         for event in nodes[2].get_and_clear_pending_msg_events() {
849                                                 match event {
850                                                         events::MessageSendEvent::UpdateHTLCs { .. } => {},
851                                                         events::MessageSendEvent::SendRevokeAndACK { .. } => {},
852                                                         events::MessageSendEvent::SendChannelReestablish { .. } => {},
853                                                         events::MessageSendEvent::SendChannelReady { .. } => {},
854                                                         events::MessageSendEvent::SendAnnouncementSignatures { .. } => {},
855                                                         events::MessageSendEvent::SendChannelUpdate { ref msg, .. } => {
856                                                                 assert_eq!(msg.contents.flags & 2, 0); // The disable bit must never be set!
857                                                         },
858                                                         _ => if out.may_fail.load(atomic::Ordering::Acquire) {
859                                                                 return;
860                                                         } else {
861                                                                 panic!("Unhandled message event")
862                                                         },
863                                                 }
864                                         }
865                                         push_excess_b_events!(nodes[1].get_and_clear_pending_msg_events().drain(..), Some(2));
866                                         bc_events.clear();
867                                         cb_events.clear();
868                                 }
869                         } }
870                 }
871
872                 macro_rules! process_events {
873                         ($node: expr, $fail: expr) => { {
874                                 // In case we get 256 payments we may have a hash collision, resulting in the
875                                 // second claim/fail call not finding the duplicate-hash HTLC, so we have to
876                                 // deduplicate the calls here.
877                                 let mut claim_set = HashSet::new();
878                                 let mut events = nodes[$node].get_and_clear_pending_events();
879                                 // Sort events so that PendingHTLCsForwardable get processed last. This avoids a
880                                 // case where we first process a PendingHTLCsForwardable, then claim/fail on a
881                                 // PaymentClaimable, claiming/failing two HTLCs, but leaving a just-generated
882                                 // PaymentClaimable event for the second HTLC in our pending_events (and breaking
883                                 // our claim_set deduplication).
884                                 events.sort_by(|a, b| {
885                                         if let events::Event::PaymentClaimable { .. } = a {
886                                                 if let events::Event::PendingHTLCsForwardable { .. } = b {
887                                                         Ordering::Less
888                                                 } else { Ordering::Equal }
889                                         } else if let events::Event::PendingHTLCsForwardable { .. } = a {
890                                                 if let events::Event::PaymentClaimable { .. } = b {
891                                                         Ordering::Greater
892                                                 } else { Ordering::Equal }
893                                         } else { Ordering::Equal }
894                                 });
895                                 let had_events = !events.is_empty();
896                                 for event in events.drain(..) {
897                                         match event {
898                                                 events::Event::PaymentClaimable { payment_hash, .. } => {
899                                                         if claim_set.insert(payment_hash.0) {
900                                                                 if $fail {
901                                                                         nodes[$node].fail_htlc_backwards(&payment_hash);
902                                                                 } else {
903                                                                         nodes[$node].claim_funds(PaymentPreimage(payment_hash.0));
904                                                                 }
905                                                         }
906                                                 },
907                                                 events::Event::PaymentSent { .. } => {},
908                                                 events::Event::PaymentClaimed { .. } => {},
909                                                 events::Event::PaymentPathSuccessful { .. } => {},
910                                                 events::Event::PaymentPathFailed { .. } => {},
911                                                 events::Event::ProbeSuccessful { .. } | events::Event::ProbeFailed { .. } => {
912                                                         // Even though we don't explicitly send probes, because probes are
913                                                         // detected based on hashing the payment hash+preimage, its rather
914                                                         // trivial for the fuzzer to build payments that accidentally end up
915                                                         // looking like probes.
916                                                 },
917                                                 events::Event::PaymentForwarded { .. } if $node == 1 => {},
918                                                 events::Event::ChannelReady { .. } => {},
919                                                 events::Event::PendingHTLCsForwardable { .. } => {
920                                                         nodes[$node].process_pending_htlc_forwards();
921                                                 },
922                                                 events::Event::HTLCHandlingFailed { .. } => {},
923                                                 _ => if out.may_fail.load(atomic::Ordering::Acquire) {
924                                                         return;
925                                                 } else {
926                                                         panic!("Unhandled event")
927                                                 },
928                                         }
929                                 }
930                                 had_events
931                         } }
932                 }
933
934                 let v = get_slice!(1)[0];
935                 out.locked_write(format!("READ A BYTE! HANDLING INPUT {:x}...........\n", v).as_bytes());
936                 match v {
937                         // In general, we keep related message groups close together in binary form, allowing
938                         // bit-twiddling mutations to have similar effects. This is probably overkill, but no
939                         // harm in doing so.
940
941                         0x00 => *monitor_a.persister.update_ret.lock().unwrap() = ChannelMonitorUpdateStatus::InProgress,
942                         0x01 => *monitor_b.persister.update_ret.lock().unwrap() = ChannelMonitorUpdateStatus::InProgress,
943                         0x02 => *monitor_c.persister.update_ret.lock().unwrap() = ChannelMonitorUpdateStatus::InProgress,
944                         0x04 => *monitor_a.persister.update_ret.lock().unwrap() = ChannelMonitorUpdateStatus::Completed,
945                         0x05 => *monitor_b.persister.update_ret.lock().unwrap() = ChannelMonitorUpdateStatus::Completed,
946                         0x06 => *monitor_c.persister.update_ret.lock().unwrap() = ChannelMonitorUpdateStatus::Completed,
947
948                         0x08 => {
949                                 if let Some((id, _)) = monitor_a.latest_monitors.lock().unwrap().get(&chan_1_funding) {
950                                         monitor_a.chain_monitor.force_channel_monitor_updated(chan_1_funding, *id);
951                                         nodes[0].process_monitor_events();
952                                 }
953                         },
954                         0x09 => {
955                                 if let Some((id, _)) = monitor_b.latest_monitors.lock().unwrap().get(&chan_1_funding) {
956                                         monitor_b.chain_monitor.force_channel_monitor_updated(chan_1_funding, *id);
957                                         nodes[1].process_monitor_events();
958                                 }
959                         },
960                         0x0a => {
961                                 if let Some((id, _)) = monitor_b.latest_monitors.lock().unwrap().get(&chan_2_funding) {
962                                         monitor_b.chain_monitor.force_channel_monitor_updated(chan_2_funding, *id);
963                                         nodes[1].process_monitor_events();
964                                 }
965                         },
966                         0x0b => {
967                                 if let Some((id, _)) = monitor_c.latest_monitors.lock().unwrap().get(&chan_2_funding) {
968                                         monitor_c.chain_monitor.force_channel_monitor_updated(chan_2_funding, *id);
969                                         nodes[2].process_monitor_events();
970                                 }
971                         },
972
973                         0x0c => {
974                                 if !chan_a_disconnected {
975                                         nodes[0].peer_disconnected(&nodes[1].get_our_node_id(), false);
976                                         nodes[1].peer_disconnected(&nodes[0].get_our_node_id(), false);
977                                         chan_a_disconnected = true;
978                                         drain_msg_events_on_disconnect!(0);
979                                 }
980                         },
981                         0x0d => {
982                                 if !chan_b_disconnected {
983                                         nodes[1].peer_disconnected(&nodes[2].get_our_node_id(), false);
984                                         nodes[2].peer_disconnected(&nodes[1].get_our_node_id(), false);
985                                         chan_b_disconnected = true;
986                                         drain_msg_events_on_disconnect!(2);
987                                 }
988                         },
989                         0x0e => {
990                                 if chan_a_disconnected {
991                                         nodes[0].peer_connected(&nodes[1].get_our_node_id(), &Init { features: channelmanager::provided_init_features(), remote_network_address: None }).unwrap();
992                                         nodes[1].peer_connected(&nodes[0].get_our_node_id(), &Init { features: channelmanager::provided_init_features(), remote_network_address: None }).unwrap();
993                                         chan_a_disconnected = false;
994                                 }
995                         },
996                         0x0f => {
997                                 if chan_b_disconnected {
998                                         nodes[1].peer_connected(&nodes[2].get_our_node_id(), &Init { features: channelmanager::provided_init_features(), remote_network_address: None }).unwrap();
999                                         nodes[2].peer_connected(&nodes[1].get_our_node_id(), &Init { features: channelmanager::provided_init_features(), remote_network_address: None }).unwrap();
1000                                         chan_b_disconnected = false;
1001                                 }
1002                         },
1003
1004                         0x10 => { process_msg_events!(0, true, ProcessMessages::AllMessages); },
1005                         0x11 => { process_msg_events!(0, false, ProcessMessages::AllMessages); },
1006                         0x12 => { process_msg_events!(0, true, ProcessMessages::OneMessage); },
1007                         0x13 => { process_msg_events!(0, false, ProcessMessages::OneMessage); },
1008                         0x14 => { process_msg_events!(0, true, ProcessMessages::OnePendingMessage); },
1009                         0x15 => { process_msg_events!(0, false, ProcessMessages::OnePendingMessage); },
1010
1011                         0x16 => { process_events!(0, true); },
1012                         0x17 => { process_events!(0, false); },
1013
1014                         0x18 => { process_msg_events!(1, true, ProcessMessages::AllMessages); },
1015                         0x19 => { process_msg_events!(1, false, ProcessMessages::AllMessages); },
1016                         0x1a => { process_msg_events!(1, true, ProcessMessages::OneMessage); },
1017                         0x1b => { process_msg_events!(1, false, ProcessMessages::OneMessage); },
1018                         0x1c => { process_msg_events!(1, true, ProcessMessages::OnePendingMessage); },
1019                         0x1d => { process_msg_events!(1, false, ProcessMessages::OnePendingMessage); },
1020
1021                         0x1e => { process_events!(1, true); },
1022                         0x1f => { process_events!(1, false); },
1023
1024                         0x20 => { process_msg_events!(2, true, ProcessMessages::AllMessages); },
1025                         0x21 => { process_msg_events!(2, false, ProcessMessages::AllMessages); },
1026                         0x22 => { process_msg_events!(2, true, ProcessMessages::OneMessage); },
1027                         0x23 => { process_msg_events!(2, false, ProcessMessages::OneMessage); },
1028                         0x24 => { process_msg_events!(2, true, ProcessMessages::OnePendingMessage); },
1029                         0x25 => { process_msg_events!(2, false, ProcessMessages::OnePendingMessage); },
1030
1031                         0x26 => { process_events!(2, true); },
1032                         0x27 => { process_events!(2, false); },
1033
1034                         0x2c => {
1035                                 if !chan_a_disconnected {
1036                                         nodes[1].peer_disconnected(&nodes[0].get_our_node_id(), false);
1037                                         chan_a_disconnected = true;
1038                                         drain_msg_events_on_disconnect!(0);
1039                                 }
1040                                 if monitor_a.should_update_manager.load(atomic::Ordering::Relaxed) {
1041                                         node_a_ser.0.clear();
1042                                         nodes[0].write(&mut node_a_ser).unwrap();
1043                                 }
1044                                 let (new_node_a, new_monitor_a) = reload_node!(node_a_ser, 0, monitor_a, keys_manager_a, fee_est_a);
1045                                 nodes[0] = new_node_a;
1046                                 monitor_a = new_monitor_a;
1047                         },
1048                         0x2d => {
1049                                 if !chan_a_disconnected {
1050                                         nodes[0].peer_disconnected(&nodes[1].get_our_node_id(), false);
1051                                         chan_a_disconnected = true;
1052                                         nodes[0].get_and_clear_pending_msg_events();
1053                                         ab_events.clear();
1054                                         ba_events.clear();
1055                                 }
1056                                 if !chan_b_disconnected {
1057                                         nodes[2].peer_disconnected(&nodes[1].get_our_node_id(), false);
1058                                         chan_b_disconnected = true;
1059                                         nodes[2].get_and_clear_pending_msg_events();
1060                                         bc_events.clear();
1061                                         cb_events.clear();
1062                                 }
1063                                 let (new_node_b, new_monitor_b) = reload_node!(node_b_ser, 1, monitor_b, keys_manager_b, fee_est_b);
1064                                 nodes[1] = new_node_b;
1065                                 monitor_b = new_monitor_b;
1066                         },
1067                         0x2e => {
1068                                 if !chan_b_disconnected {
1069                                         nodes[1].peer_disconnected(&nodes[2].get_our_node_id(), false);
1070                                         chan_b_disconnected = true;
1071                                         drain_msg_events_on_disconnect!(2);
1072                                 }
1073                                 if monitor_c.should_update_manager.load(atomic::Ordering::Relaxed) {
1074                                         node_c_ser.0.clear();
1075                                         nodes[2].write(&mut node_c_ser).unwrap();
1076                                 }
1077                                 let (new_node_c, new_monitor_c) = reload_node!(node_c_ser, 2, monitor_c, keys_manager_c, fee_est_c);
1078                                 nodes[2] = new_node_c;
1079                                 monitor_c = new_monitor_c;
1080                         },
1081
1082                         // 1/10th the channel size:
1083                         0x30 => { send_payment(&nodes[0], &nodes[1], chan_a, 10_000_000, &mut payment_id, &mut payment_idx); },
1084                         0x31 => { send_payment(&nodes[1], &nodes[0], chan_a, 10_000_000, &mut payment_id, &mut payment_idx); },
1085                         0x32 => { send_payment(&nodes[1], &nodes[2], chan_b, 10_000_000, &mut payment_id, &mut payment_idx); },
1086                         0x33 => { send_payment(&nodes[2], &nodes[1], chan_b, 10_000_000, &mut payment_id, &mut payment_idx); },
1087                         0x34 => { send_hop_payment(&nodes[0], &nodes[1], chan_a, &nodes[2], chan_b, 10_000_000, &mut payment_id, &mut payment_idx); },
1088                         0x35 => { send_hop_payment(&nodes[2], &nodes[1], chan_b, &nodes[0], chan_a, 10_000_000, &mut payment_id, &mut payment_idx); },
1089
1090                         0x38 => { send_payment(&nodes[0], &nodes[1], chan_a, 1_000_000, &mut payment_id, &mut payment_idx); },
1091                         0x39 => { send_payment(&nodes[1], &nodes[0], chan_a, 1_000_000, &mut payment_id, &mut payment_idx); },
1092                         0x3a => { send_payment(&nodes[1], &nodes[2], chan_b, 1_000_000, &mut payment_id, &mut payment_idx); },
1093                         0x3b => { send_payment(&nodes[2], &nodes[1], chan_b, 1_000_000, &mut payment_id, &mut payment_idx); },
1094                         0x3c => { send_hop_payment(&nodes[0], &nodes[1], chan_a, &nodes[2], chan_b, 1_000_000, &mut payment_id, &mut payment_idx); },
1095                         0x3d => { send_hop_payment(&nodes[2], &nodes[1], chan_b, &nodes[0], chan_a, 1_000_000, &mut payment_id, &mut payment_idx); },
1096
1097                         0x40 => { send_payment(&nodes[0], &nodes[1], chan_a, 100_000, &mut payment_id, &mut payment_idx); },
1098                         0x41 => { send_payment(&nodes[1], &nodes[0], chan_a, 100_000, &mut payment_id, &mut payment_idx); },
1099                         0x42 => { send_payment(&nodes[1], &nodes[2], chan_b, 100_000, &mut payment_id, &mut payment_idx); },
1100                         0x43 => { send_payment(&nodes[2], &nodes[1], chan_b, 100_000, &mut payment_id, &mut payment_idx); },
1101                         0x44 => { send_hop_payment(&nodes[0], &nodes[1], chan_a, &nodes[2], chan_b, 100_000, &mut payment_id, &mut payment_idx); },
1102                         0x45 => { send_hop_payment(&nodes[2], &nodes[1], chan_b, &nodes[0], chan_a, 100_000, &mut payment_id, &mut payment_idx); },
1103
1104                         0x48 => { send_payment(&nodes[0], &nodes[1], chan_a, 10_000, &mut payment_id, &mut payment_idx); },
1105                         0x49 => { send_payment(&nodes[1], &nodes[0], chan_a, 10_000, &mut payment_id, &mut payment_idx); },
1106                         0x4a => { send_payment(&nodes[1], &nodes[2], chan_b, 10_000, &mut payment_id, &mut payment_idx); },
1107                         0x4b => { send_payment(&nodes[2], &nodes[1], chan_b, 10_000, &mut payment_id, &mut payment_idx); },
1108                         0x4c => { send_hop_payment(&nodes[0], &nodes[1], chan_a, &nodes[2], chan_b, 10_000, &mut payment_id, &mut payment_idx); },
1109                         0x4d => { send_hop_payment(&nodes[2], &nodes[1], chan_b, &nodes[0], chan_a, 10_000, &mut payment_id, &mut payment_idx); },
1110
1111                         0x50 => { send_payment(&nodes[0], &nodes[1], chan_a, 1_000, &mut payment_id, &mut payment_idx); },
1112                         0x51 => { send_payment(&nodes[1], &nodes[0], chan_a, 1_000, &mut payment_id, &mut payment_idx); },
1113                         0x52 => { send_payment(&nodes[1], &nodes[2], chan_b, 1_000, &mut payment_id, &mut payment_idx); },
1114                         0x53 => { send_payment(&nodes[2], &nodes[1], chan_b, 1_000, &mut payment_id, &mut payment_idx); },
1115                         0x54 => { send_hop_payment(&nodes[0], &nodes[1], chan_a, &nodes[2], chan_b, 1_000, &mut payment_id, &mut payment_idx); },
1116                         0x55 => { send_hop_payment(&nodes[2], &nodes[1], chan_b, &nodes[0], chan_a, 1_000, &mut payment_id, &mut payment_idx); },
1117
1118                         0x58 => { send_payment(&nodes[0], &nodes[1], chan_a, 100, &mut payment_id, &mut payment_idx); },
1119                         0x59 => { send_payment(&nodes[1], &nodes[0], chan_a, 100, &mut payment_id, &mut payment_idx); },
1120                         0x5a => { send_payment(&nodes[1], &nodes[2], chan_b, 100, &mut payment_id, &mut payment_idx); },
1121                         0x5b => { send_payment(&nodes[2], &nodes[1], chan_b, 100, &mut payment_id, &mut payment_idx); },
1122                         0x5c => { send_hop_payment(&nodes[0], &nodes[1], chan_a, &nodes[2], chan_b, 100, &mut payment_id, &mut payment_idx); },
1123                         0x5d => { send_hop_payment(&nodes[2], &nodes[1], chan_b, &nodes[0], chan_a, 100, &mut payment_id, &mut payment_idx); },
1124
1125                         0x60 => { send_payment(&nodes[0], &nodes[1], chan_a, 10, &mut payment_id, &mut payment_idx); },
1126                         0x61 => { send_payment(&nodes[1], &nodes[0], chan_a, 10, &mut payment_id, &mut payment_idx); },
1127                         0x62 => { send_payment(&nodes[1], &nodes[2], chan_b, 10, &mut payment_id, &mut payment_idx); },
1128                         0x63 => { send_payment(&nodes[2], &nodes[1], chan_b, 10, &mut payment_id, &mut payment_idx); },
1129                         0x64 => { send_hop_payment(&nodes[0], &nodes[1], chan_a, &nodes[2], chan_b, 10, &mut payment_id, &mut payment_idx); },
1130                         0x65 => { send_hop_payment(&nodes[2], &nodes[1], chan_b, &nodes[0], chan_a, 10, &mut payment_id, &mut payment_idx); },
1131
1132                         0x68 => { send_payment(&nodes[0], &nodes[1], chan_a, 1, &mut payment_id, &mut payment_idx); },
1133                         0x69 => { send_payment(&nodes[1], &nodes[0], chan_a, 1, &mut payment_id, &mut payment_idx); },
1134                         0x6a => { send_payment(&nodes[1], &nodes[2], chan_b, 1, &mut payment_id, &mut payment_idx); },
1135                         0x6b => { send_payment(&nodes[2], &nodes[1], chan_b, 1, &mut payment_id, &mut payment_idx); },
1136                         0x6c => { send_hop_payment(&nodes[0], &nodes[1], chan_a, &nodes[2], chan_b, 1, &mut payment_id, &mut payment_idx); },
1137                         0x6d => { send_hop_payment(&nodes[2], &nodes[1], chan_b, &nodes[0], chan_a, 1, &mut payment_id, &mut payment_idx); },
1138
1139                         0x80 => {
1140                                 let max_feerate = last_htlc_clear_fee_a * FEE_SPIKE_BUFFER_FEE_INCREASE_MULTIPLE as u32;
1141                                 if fee_est_a.ret_val.fetch_add(250, atomic::Ordering::AcqRel) + 250 > max_feerate {
1142                                         fee_est_a.ret_val.store(max_feerate, atomic::Ordering::Release);
1143                                 }
1144                                 nodes[0].maybe_update_chan_fees();
1145                         },
1146                         0x81 => { fee_est_a.ret_val.store(253, atomic::Ordering::Release); nodes[0].maybe_update_chan_fees(); },
1147
1148                         0x84 => {
1149                                 let max_feerate = last_htlc_clear_fee_b * FEE_SPIKE_BUFFER_FEE_INCREASE_MULTIPLE as u32;
1150                                 if fee_est_b.ret_val.fetch_add(250, atomic::Ordering::AcqRel) + 250 > max_feerate {
1151                                         fee_est_b.ret_val.store(max_feerate, atomic::Ordering::Release);
1152                                 }
1153                                 nodes[1].maybe_update_chan_fees();
1154                         },
1155                         0x85 => { fee_est_b.ret_val.store(253, atomic::Ordering::Release); nodes[1].maybe_update_chan_fees(); },
1156
1157                         0x88 => {
1158                                 let max_feerate = last_htlc_clear_fee_c * FEE_SPIKE_BUFFER_FEE_INCREASE_MULTIPLE as u32;
1159                                 if fee_est_c.ret_val.fetch_add(250, atomic::Ordering::AcqRel) + 250 > max_feerate {
1160                                         fee_est_c.ret_val.store(max_feerate, atomic::Ordering::Release);
1161                                 }
1162                                 nodes[2].maybe_update_chan_fees();
1163                         },
1164                         0x89 => { fee_est_c.ret_val.store(253, atomic::Ordering::Release); nodes[2].maybe_update_chan_fees(); },
1165
1166                         0xff => {
1167                                 // Test that no channel is in a stuck state where neither party can send funds even
1168                                 // after we resolve all pending events.
1169                                 // First make sure there are no pending monitor updates, resetting the error state
1170                                 // and calling force_channel_monitor_updated for each monitor.
1171                                 *monitor_a.persister.update_ret.lock().unwrap() = ChannelMonitorUpdateStatus::Completed;
1172                                 *monitor_b.persister.update_ret.lock().unwrap() = ChannelMonitorUpdateStatus::Completed;
1173                                 *monitor_c.persister.update_ret.lock().unwrap() = ChannelMonitorUpdateStatus::Completed;
1174
1175                                 if let Some((id, _)) = monitor_a.latest_monitors.lock().unwrap().get(&chan_1_funding) {
1176                                         monitor_a.chain_monitor.force_channel_monitor_updated(chan_1_funding, *id);
1177                                         nodes[0].process_monitor_events();
1178                                 }
1179                                 if let Some((id, _)) = monitor_b.latest_monitors.lock().unwrap().get(&chan_1_funding) {
1180                                         monitor_b.chain_monitor.force_channel_monitor_updated(chan_1_funding, *id);
1181                                         nodes[1].process_monitor_events();
1182                                 }
1183                                 if let Some((id, _)) = monitor_b.latest_monitors.lock().unwrap().get(&chan_2_funding) {
1184                                         monitor_b.chain_monitor.force_channel_monitor_updated(chan_2_funding, *id);
1185                                         nodes[1].process_monitor_events();
1186                                 }
1187                                 if let Some((id, _)) = monitor_c.latest_monitors.lock().unwrap().get(&chan_2_funding) {
1188                                         monitor_c.chain_monitor.force_channel_monitor_updated(chan_2_funding, *id);
1189                                         nodes[2].process_monitor_events();
1190                                 }
1191
1192                                 // Next, make sure peers are all connected to each other
1193                                 if chan_a_disconnected {
1194                                         nodes[0].peer_connected(&nodes[1].get_our_node_id(), &Init { features: channelmanager::provided_init_features(), remote_network_address: None }).unwrap();
1195                                         nodes[1].peer_connected(&nodes[0].get_our_node_id(), &Init { features: channelmanager::provided_init_features(), remote_network_address: None }).unwrap();
1196                                         chan_a_disconnected = false;
1197                                 }
1198                                 if chan_b_disconnected {
1199                                         nodes[1].peer_connected(&nodes[2].get_our_node_id(), &Init { features: channelmanager::provided_init_features(), remote_network_address: None }).unwrap();
1200                                         nodes[2].peer_connected(&nodes[1].get_our_node_id(), &Init { features: channelmanager::provided_init_features(), remote_network_address: None }).unwrap();
1201                                         chan_b_disconnected = false;
1202                                 }
1203
1204                                 for i in 0..std::usize::MAX {
1205                                         if i == 100 { panic!("It may take may iterations to settle the state, but it should not take forever"); }
1206                                         // Then, make sure any current forwards make their way to their destination
1207                                         if process_msg_events!(0, false, ProcessMessages::AllMessages) { continue; }
1208                                         if process_msg_events!(1, false, ProcessMessages::AllMessages) { continue; }
1209                                         if process_msg_events!(2, false, ProcessMessages::AllMessages) { continue; }
1210                                         // ...making sure any pending PendingHTLCsForwardable events are handled and
1211                                         // payments claimed.
1212                                         if process_events!(0, false) { continue; }
1213                                         if process_events!(1, false) { continue; }
1214                                         if process_events!(2, false) { continue; }
1215                                         break;
1216                                 }
1217
1218                                 // Finally, make sure that at least one end of each channel can make a substantial payment
1219                                 assert!(
1220                                         send_payment(&nodes[0], &nodes[1], chan_a, 10_000_000, &mut payment_id, &mut payment_idx) ||
1221                                         send_payment(&nodes[1], &nodes[0], chan_a, 10_000_000, &mut payment_id, &mut payment_idx));
1222                                 assert!(
1223                                         send_payment(&nodes[1], &nodes[2], chan_b, 10_000_000, &mut payment_id, &mut payment_idx) ||
1224                                         send_payment(&nodes[2], &nodes[1], chan_b, 10_000_000, &mut payment_id, &mut payment_idx));
1225
1226                                 last_htlc_clear_fee_a = fee_est_a.ret_val.load(atomic::Ordering::Acquire);
1227                                 last_htlc_clear_fee_b = fee_est_b.ret_val.load(atomic::Ordering::Acquire);
1228                                 last_htlc_clear_fee_c = fee_est_c.ret_val.load(atomic::Ordering::Acquire);
1229                         },
1230                         _ => test_return!(),
1231                 }
1232
1233                 node_a_ser.0.clear();
1234                 nodes[0].write(&mut node_a_ser).unwrap();
1235                 monitor_a.should_update_manager.store(false, atomic::Ordering::Relaxed);
1236                 node_b_ser.0.clear();
1237                 nodes[1].write(&mut node_b_ser).unwrap();
1238                 monitor_b.should_update_manager.store(false, atomic::Ordering::Relaxed);
1239                 node_c_ser.0.clear();
1240                 nodes[2].write(&mut node_c_ser).unwrap();
1241                 monitor_c.should_update_manager.store(false, atomic::Ordering::Relaxed);
1242         }
1243 }
1244
1245 /// We actually have different behavior based on if a certain log string has been seen, so we have
1246 /// to do a bit more tracking.
1247 #[derive(Clone)]
1248 struct SearchingOutput<O: Output> {
1249         output: O,
1250         may_fail: Arc<atomic::AtomicBool>,
1251 }
1252 impl<O: Output> Output for SearchingOutput<O> {
1253         fn locked_write(&self, data: &[u8]) {
1254                 // We hit a design limitation of LN state machine (see CONCURRENT_INBOUND_HTLC_FEE_BUFFER)
1255                 if std::str::from_utf8(data).unwrap().contains("Outbound update_fee HTLC buffer overflow - counterparty should force-close this channel") {
1256                         self.may_fail.store(true, atomic::Ordering::Release);
1257                 }
1258                 self.output.locked_write(data)
1259         }
1260 }
1261 impl<O: Output> SearchingOutput<O> {
1262         pub fn new(output: O) -> Self {
1263                 Self { output, may_fail: Arc::new(atomic::AtomicBool::new(false)) }
1264         }
1265 }
1266
1267 pub fn chanmon_consistency_test<Out: Output>(data: &[u8], out: Out) {
1268         do_test(data, out);
1269 }
1270
1271 #[no_mangle]
1272 pub extern "C" fn chanmon_consistency_run(data: *const u8, datalen: usize) {
1273         do_test(unsafe { std::slice::from_raw_parts(data, datalen) }, test_logger::DevNull{});
1274 }