]> git.bitcoin.ninja Git - rust-lightning/commitdiff
Add key_storage selection in ChannelMonitor insert_combine
authorAntoine Riard <ariard@student.42.fr>
Tue, 6 Nov 2018 03:23:22 +0000 (03:23 +0000)
committerAntoine Riard <ariard@student.42.fr>
Tue, 20 Nov 2018 01:42:55 +0000 (20:42 -0500)
Based on commitment_number

src/ln/channelmonitor.rs

index 8b209c1e7b00e94ca8f69767ce808f5df4b93223..7f584aeba804e3b86ee3a12c18ee9aabb12ae61a 100644 (file)
@@ -552,6 +552,15 @@ impl ChannelMonitor {
                if our_min_secret > other_min_secret {
                        self.provide_secret(other_min_secret, other.get_secret(other_min_secret).unwrap())?;
                }
+               if let Some(ref local_tx) = self.current_local_signed_commitment_tx {
+                       if let Some(ref other_local_tx) = other.current_local_signed_commitment_tx {
+                               let our_commitment_number = 0xffffffffffff - ((((local_tx.tx.input[0].sequence as u64 & 0xffffff) << 3*8) | (local_tx.tx.lock_time as u64 & 0xffffff)) ^ self.commitment_transaction_number_obscure_factor);
+                               let other_commitment_number = 0xffffffffffff - ((((other_local_tx.tx.input[0].sequence as u64 & 0xffffff) << 3*8) | (other_local_tx.tx.lock_time as u64 & 0xffffff)) ^ other.commitment_transaction_number_obscure_factor);
+                               if our_commitment_number >= other_commitment_number {
+                                       self.key_storage = other.key_storage;
+                               }
+                       }
+               }
                // TODO: We should use current_remote_commitment_number and the commitment number out of
                // local transactions to decide how to merge
                if our_min_secret >= other_min_secret {
@@ -567,6 +576,7 @@ impl ChannelMonitor {
                        }
                        self.payment_preimages = other.payment_preimages;
                }
+
                self.current_remote_commitment_number = cmp::min(self.current_remote_commitment_number, other.current_remote_commitment_number);
                Ok(())
        }