Skip to content

Commit 8e72922

Browse files
committed
Don't hold write lock in LSPS{1,2}ServiceHandler::peer_disconnected
.. as there's no need to do so.
1 parent 3e3e8e3 commit 8e72922

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

lightning-liquidity/src/lsps1/service.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ where
128128
}
129129

130130
pub(crate) fn peer_disconnected(&self, counterparty_node_id: PublicKey) {
131-
let outer_state_lock = self.per_peer_state.write().unwrap();
131+
let outer_state_lock = self.per_peer_state.read().unwrap();
132132
if let Some(inner_state_lock) = outer_state_lock.get(&counterparty_node_id) {
133133
let mut peer_state_lock = inner_state_lock.lock().unwrap();
134134
// We clean up the peer state, but leave removing the peer entry to the prune logic in

lightning-liquidity/src/lsps2/service.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1875,7 +1875,7 @@ where
18751875
}
18761876

18771877
pub(crate) fn peer_disconnected(&self, counterparty_node_id: PublicKey) {
1878-
let outer_state_lock = self.per_peer_state.write().unwrap();
1878+
let outer_state_lock = self.per_peer_state.read().unwrap();
18791879
if let Some(inner_state_lock) = outer_state_lock.get(&counterparty_node_id) {
18801880
let mut peer_state_lock = inner_state_lock.lock().unwrap();
18811881
// We clean up the peer state, but leave removing the peer entry to the prune logic in

0 commit comments

Comments
 (0)