What the developer wrote
Authored by ww3512687
fix connect wallet ui
What changed, and why it matters
This commit changes which wallet icons and names appear in the status bar depending on which product version is being built (Web3, Cypherpunk, or Bitcoin-only). It is a user-interface adjustment, not a security fix.
No security action needed; treat as a normal UI/product configuration change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch replaces a single compile-time guard #ifndef BTC_ONLY with three separate guards (#ifdef WEB3_VERSION, #ifdef CYPHERPUNK_VERSION, and #ifdef BTC_ONLY) in the wallet list array used by the status bar UI. It adds or reorganizes wallet entries for each build variant. No cryptographic, memory-safety, authentication, or input-validation code is modified.
Changed components
src/ui/gui_components/gui_status_bar.cInspect captured patch +14 / −2
diff --git a/src/ui/gui_components/gui_status_bar.c b/src/ui/gui_components/gui_status_bar.c
index 8c7fc22..2d1679c 100644
--- a/src/ui/gui_components/gui_status_bar.c
+++ b/src/ui/gui_components/gui_status_bar.c
@@ -132,7 +132,7 @@ const static CoinWalletInfo_t g_coinWalletBtn[] = {
};
const static WalletInfo_t g_walletBtn[] = {
-#ifndef BTC_ONLY
+#ifdef WEB3_VERSION
{WALLET_LIST_KEYSTONE, "Keystone Nexus", &walletKeystone},
{WALLET_LIST_METAMASK, "MetaMask", &walletMetamask},
{WALLET_LIST_OKX, "OKX Wallet", &walletOkx},
@@ -176,7 +176,19 @@ const static WalletInfo_t g_walletBtn[] = {
{WALLET_LIST_FEATHER, "Feather Wallet", &walletFeather},
{WALLET_LIST_CORE, "Core Wallet", &walletCore},
{WALLET_LIST_IOTA, "IOTA Wallet", &walletIota},
-#else
+#endif
+
+#ifdef CYPHERPUNK_VERSION
+ {WALLET_LIST_BLUE, "Blue Wallet", &walletBluewallet},
+ {WALLET_LIST_SPARROW, "Sparrow", &walletSparrow},
+ {WALLET_LIST_UNISAT, "UniSat", &walletUniSat},
+ {WALLET_LIST_ZEUS, "Zeus Wallet", &walletZeus},
+ // {WALLET_LIST_CAKE, "Cake Wallet", &walletCake},
+ {WALLET_LIST_FEATHER, "Feather Wallet", &walletFeather},
+ {WALLET_LIST_ZASHI, "Zashi", &walletZashi},
+#endif
+
+#ifdef BTC_ONLY
{WALLET_LIST_BLUE, "BlueWallet", &walletBluewallet},
{WALLET_LIST_SPECTER, "Specter", &walletSpecter},
{WALLET_LIST_SPARROW, "Sparrow", &walletSparrow},
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.