fix: coin selection with include_unsafe option should respect nCoinType

UdjinM6 · UdjinM6 · commit e5114da32536 · 2024-12-02T17:26:26.000+03:00
diff --git a/src/wallet/wallet.cpp b/src/wallet/wallet.cpp
@@ -3041,7 +3041,7 @@ bool CWallet::SelectCoins(const std::vector<COutput>& vAvailableCoins, const CAm
             if (coin_control.m_include_unsafe_inputs
                 && SelectCoinsMinConf(value_to_select,
                     CoinEligibilityFilter(0 /* conf_mine */, 0 /* conf_theirs */, max_ancestors-1, max_descendants-1, true /* include_partial_groups */),
-                    vCoins, setCoinsRet, nValueRet, coin_selection_params, bnb_used)) {
+                    vCoins, setCoinsRet, nValueRet, coin_selection_params, bnb_used, nCoinType)) {
                 return true;
             }
             // Try with unlimited ancestors/descendants. The transaction will still need to meet

Original file line number	Diff line number	Diff line change
`@@ -3041,7 +3041,7 @@ bool CWallet::SelectCoins(const std::vector<COutput>& vAvailableCoins, const CAm`
`3041`	`3041`	`if (coin_control.m_include_unsafe_inputs`
`3042`	`3042`	`&& SelectCoinsMinConf(value_to_select,`
`3043`	`3043`	`CoinEligibilityFilter(0 /* conf_mine /, 0 / conf_theirs /, max_ancestors-1, max_descendants-1, true / include_partial_groups */),`
`3044`		`- vCoins, setCoinsRet, nValueRet, coin_selection_params, bnb_used)) {`
	`3044`	`+ vCoins, setCoinsRet, nValueRet, coin_selection_params, bnb_used, nCoinType)) {`
`3045`	`3045`	`return true;`
`3046`	`3046`	`}`
`3047`	`3047`	`// Try with unlimited ancestors/descendants. The transaction will still need to meet`