f only remove compl updates on first loop

Author: TheBlueMatt

lightning/src/ln/channelmanager.rs

@@ -6099,19 +6099,20 @@ where
 		self.pending_outbound_payments.clear_pending_payments()
 	}
 
-	fn handle_monitor_update_release(&self, counterparty_node_id: PublicKey, channel_funding_outpoint: OutPoint, completed_blocker: Option<RAAMonitorUpdateBlockingAction>) {
+	fn handle_monitor_update_release(&self, counterparty_node_id: PublicKey, channel_funding_outpoint: OutPoint, mut completed_blocker: Option<RAAMonitorUpdateBlockingAction>) {
 		let mut errors = Vec::new();
 		loop {
 			let per_peer_state = self.per_peer_state.read().unwrap();
 			if let Some(peer_state_mtx) = per_peer_state.get(&counterparty_node_id) {
 				let mut peer_state_lck = peer_state_mtx.lock().unwrap();
 				let peer_state = &mut *peer_state_lck;
 
-				if let Some(blocker) = &completed_blocker {
+				if let Some(blocker) = completed_blocker.take() {
+					// Only do this on the first iteration of the loop.
 					if let Some(blockers) = peer_state.actions_blocking_raa_monitor_updates
 						.get_mut(&channel_funding_outpoint.to_channel_id())
 					{
-						blockers.retain(|iter| iter != blocker);
+						blockers.retain(|iter| iter != &blocker);
 					}
 				}