Skip to content

Commit c393dd1

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 fb2f456 commit c393dd1

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
@@ -127,7 +127,7 @@ where
127127
}
128128

129129
pub(crate) fn peer_disconnected(&self, counterparty_node_id: PublicKey) {
130-
let outer_state_lock = self.per_peer_state.write().unwrap();
130+
let outer_state_lock = self.per_peer_state.read().unwrap();
131131
if let Some(inner_state_lock) = outer_state_lock.get(&counterparty_node_id) {
132132
let mut peer_state_lock = inner_state_lock.lock().unwrap();
133133
// 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)