From 2279b332d4c1fd06c761e841ec15adfc2b5787dc Mon Sep 17 00:00:00 2001
From: Michael Mallan <michaelmallan@proton.me>
Date: Fri, 4 Apr 2025 12:24:29 +0100
Subject: [PATCH] filter for confirmed coins when checking recovery paths

The recovery transaction created by `createrecovery` only
includes confirmed coins and so we should only consider these
coins when checking for available recovery paths.

An unconfirmed coin could not be broadcast in a recovery
transaction as it would not yet satisfy the timelock constraint.
---
 liana-gui/src/app/state/recovery.rs | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/liana-gui/src/app/state/recovery.rs b/liana-gui/src/app/state/recovery.rs
index aded0454..3f50dedc 100644
--- a/liana-gui/src/app/state/recovery.rs
+++ b/liana-gui/src/app/state/recovery.rs
@@ -244,7 +244,8 @@ fn recovery_paths(wallet: &Wallet, coins: &[Coin], blockheight: i32) -> Vec<Reco
             let (number_of_coins, total_amount) = coins
                 .iter()
                 .filter(|coin| {
-                    coin.spend_info.is_none()
+                    coin.block_height.is_some() // only confirmed coins are included in a recovery transaction
+                        && coin.spend_info.is_none()
                         && remaining_sequence(coin, blockheight as u32, sequence) <= 1
                 })
                 .fold(