From: Matt Corallo Date: Tue, 21 Jan 2020 03:05:29 +0000 (-0500) Subject: Track the full list of outpoints a chanmon wants monitoring for. X-Git-Tag: v0.0.12~131^2~2 X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=commitdiff_plain;h=73dce207dd0ea6c3ac5;p=rust-lightning Track the full list of outpoints a chanmon wants monitoring for. Upon deserialization/reload we need to be able to register each outpoint which spends the commitment txo which a channelmonitor believes to be on chain. While our other internal tracking is likely sufficient to regenerate these, its much easier to simply track all outpouts we've ever generated, so we do that here. --- diff --git a/lightning/src/ln/channelmonitor.rs b/lightning/src/ln/channelmonitor.rs index a1577712f..71ce91fd2 100644 --- a/lightning/src/ln/channelmonitor.rs +++ b/lightning/src/ln/channelmonitor.rs @@ -117,9 +117,16 @@ pub struct HTLCUpdate { pub trait ManyChannelMonitor: Send + Sync { /// Adds or updates a monitor for the given `funding_txo`. /// - /// Implementor must also ensure that the funding_txo outpoint is registered with any relevant - /// ChainWatchInterfaces such that the provided monitor receives block_connected callbacks with - /// any spends of it. + /// Implementer must also ensure that the funding_txo txid *and* outpoint are registered with + /// any relevant ChainWatchInterfaces such that the provided monitor receives block_connected + /// callbacks with the funding transaction, or any spends of it. + /// + /// Further, the implementer must also ensure that each output returned in + /// monitor.get_outputs_to_watch() is registered to ensure that the provided monitor learns about + /// any spends of any of the outputs. + /// + /// Any spends of outputs which should have been registered which aren't passed to + /// ChannelMonitors via block_connected may result in funds loss. fn add_update_monitor(&self, funding_txo: OutPoint, monitor: ChannelMonitor) -> Result<(), ChannelMonitorUpdateErr>; /// Used by ChannelManager to get list of HTLC resolved onchain and which needed to be updated @@ -259,6 +266,11 @@ impl Simpl self.chain_monitor.watch_all_txn(); } } + for (txid, outputs) in monitor.get_outputs_to_watch().iter() { + for (idx, script) in outputs.iter().enumerate() { + self.chain_monitor.install_watch_outpoint((*txid, idx as u32), script); + } + } monitors.insert(key, monitor); Ok(()) } @@ -666,6 +678,12 @@ pub struct ChannelMonitor { // actions when we receive a block with given height. Actions depend on OnchainEvent type. onchain_events_waiting_threshold_conf: HashMap>, + // If we get serialized out and re-read, we need to make sure that the chain monitoring + // interface knows about the TXOs that we want to be notified of spends of. We could probably + // be smart and derive them from the above storage fields, but its much simpler and more + // Obviously Correct (tm) if we just keep track of them explicitly. + outputs_to_watch: HashMap>, + // We simply modify last_block_hash in Channel's block_connected so that serialization is // consistent but hopefully the users' copy handles block_connected in a consistent way. // (we do *not*, however, update them in insert_combine to ensure any local user copies keep @@ -736,7 +754,8 @@ impl PartialEq for ChannelMonitor { self.to_remote_rescue != other.to_remote_rescue || self.pending_claim_requests != other.pending_claim_requests || self.claimable_outpoints != other.claimable_outpoints || - self.onchain_events_waiting_threshold_conf != other.onchain_events_waiting_threshold_conf + self.onchain_events_waiting_threshold_conf != other.onchain_events_waiting_threshold_conf || + self.outputs_to_watch != other.outputs_to_watch { false } else { @@ -966,6 +985,15 @@ impl ChannelMonitor { } } + (self.outputs_to_watch.len() as u64).write(writer)?; + for (txid, output_scripts) in self.outputs_to_watch.iter() { + txid.write(writer)?; + (output_scripts.len() as u64).write(writer)?; + for script in output_scripts.iter() { + script.write(writer)?; + } + } + Ok(()) } @@ -1036,6 +1064,7 @@ impl ChannelMonitor { claimable_outpoints: HashMap::new(), onchain_events_waiting_threshold_conf: HashMap::new(), + outputs_to_watch: HashMap::new(), last_block_hash: Default::default(), secp_ctx: Secp256k1::new(), @@ -1370,6 +1399,12 @@ impl ChannelMonitor { } } + /// Gets a list of txids, with their output scripts (in the order they appear in the + /// transaction), which we must learn about spends of via block_connected(). + pub fn get_outputs_to_watch(&self) -> &HashMap> { + &self.outputs_to_watch + } + /// Gets the sets of all outpoints which this ChannelMonitor expects to hear about spends of. /// Generally useful when deserializing as during normal operation the return values of /// block_connected are sufficient to ensure all relevant outpoints are being monitored (note @@ -2589,6 +2624,9 @@ impl ChannelMonitor { } } self.last_block_hash = block_hash.clone(); + for &(ref txid, ref output_scripts) in watch_outputs.iter() { + self.outputs_to_watch.insert(txid.clone(), output_scripts.iter().map(|o| o.script_pubkey.clone()).collect()); + } (watch_outputs, spendable_outputs, htlc_updated) } @@ -3241,6 +3279,20 @@ impl> ReadableArgs() + mem::size_of::>()))); + for _ in 0..outputs_to_watch_len { + let txid = Readable::read(reader)?; + let outputs_len: u64 = Readable::read(reader)?; + let mut outputs = Vec::with_capacity(cmp::min(outputs_len as usize, MAX_ALLOC_SIZE / mem::size_of::