Skip to content

Commit 9f640b5

Browse files
Remove unnecessary channel counterparty checks
1 parent 85caab1 commit 9f640b5

File tree

1 file changed

+7
-12
lines changed

1 file changed

+7
-12
lines changed

lightning/src/ln/channelmanager.rs

Lines changed: 7 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -6037,15 +6037,13 @@ where
60376037
let mut peer_state_lock = peer_state_mutex.lock().unwrap();
60386038
let peer_state = &mut *peer_state_lock;
60396039
peer_state.channel_by_id.retain(|_, chan| {
6040-
if chan.get_counterparty_node_id() == *counterparty_node_id {
6041-
chan.remove_uncommitted_htlcs_and_mark_paused(&self.logger);
6042-
if chan.is_shutdown() {
6043-
update_maps_on_chan_removal!(self, chan);
6044-
self.issue_channel_close_events(chan, ClosureReason::DisconnectedPeer);
6045-
return false;
6046-
} else {
6047-
no_channels_remain = false;
6048-
}
6040+
chan.remove_uncommitted_htlcs_and_mark_paused(&self.logger);
6041+
if chan.is_shutdown() {
6042+
update_maps_on_chan_removal!(self, chan);
6043+
self.issue_channel_close_events(chan, ClosureReason::DisconnectedPeer);
6044+
return false;
6045+
} else {
6046+
no_channels_remain = false;
60496047
}
60506048
true
60516049
});
@@ -6171,9 +6169,6 @@ where
61716169
let mut peer_state_lock = peer_state_mutex.lock().unwrap();
61726170
let peer_state = &mut *peer_state_lock;
61736171
if let Some(chan) = peer_state.channel_by_id.get_mut(&msg.channel_id) {
6174-
if chan.get_counterparty_node_id() != *counterparty_node_id {
6175-
return;
6176-
}
61776172
if let Ok(msg) = chan.maybe_handle_error_without_close(self.genesis_hash) {
61786173
channel_state.pending_msg_events.push(events::MessageSendEvent::SendOpenChannel {
61796174
node_id: *counterparty_node_id,

0 commit comments

Comments
 (0)