From: Viktor Tigerström <11711198+ViktorTigerstrom@users.noreply.github.com>
Date: Sat, 10 Dec 2022 17:03:22 +0000 (+0100)
Subject: Remove unnecessary channel counterparty checks
X-Git-Tag: v0.0.114-beta~67^2~7
X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=commitdiff_plain;h=c42323c8577e58ccdf2e964bdacac321d91bf3f6;p=rust-lightning

Remove unnecessary channel counterparty checks
---

diff --git a/lightning/src/ln/channelmanager.rs b/lightning/src/ln/channelmanager.rs
index 8dc456846..ca9127023 100644
--- a/lightning/src/ln/channelmanager.rs
+++ b/lightning/src/ln/channelmanager.rs
@@ -6037,15 +6037,13 @@ where
 				let mut peer_state_lock = peer_state_mutex.lock().unwrap();
 				let peer_state = &mut *peer_state_lock;
 				peer_state.channel_by_id.retain(|_, chan| {
-					if chan.get_counterparty_node_id() == *counterparty_node_id {
-						chan.remove_uncommitted_htlcs_and_mark_paused(&self.logger);
-						if chan.is_shutdown() {
-							update_maps_on_chan_removal!(self, chan);
-							self.issue_channel_close_events(chan, ClosureReason::DisconnectedPeer);
-							return false;
-						} else {
-							no_channels_remain = false;
-						}
+					chan.remove_uncommitted_htlcs_and_mark_paused(&self.logger);
+					if chan.is_shutdown() {
+						update_maps_on_chan_removal!(self, chan);
+						self.issue_channel_close_events(chan, ClosureReason::DisconnectedPeer);
+						return false;
+					} else {
+						no_channels_remain = false;
 					}
 					true
 				});
@@ -6171,9 +6169,6 @@ where
 					let mut peer_state_lock = peer_state_mutex.lock().unwrap();
 					let peer_state = &mut *peer_state_lock;
 					if let Some(chan) = peer_state.channel_by_id.get_mut(&msg.channel_id) {
-						if chan.get_counterparty_node_id() != *counterparty_node_id {
-							return;
-						}
 						if let Ok(msg) = chan.maybe_handle_error_without_close(self.genesis_hash) {
 							channel_state.pending_msg_events.push(events::MessageSendEvent::SendOpenChannel {
 								node_id: *counterparty_node_id,