What the developer wrote
Authored by ww3512687
fix ui bugs
What changed, and why it matters
This commit fixes a user-interface bug in the Keystone 3 firmware. It adds a missing entry for the 'Bull' wallet in the wallet tutorial list and corrects a typo in the icon array from 'walletXBull' to 'walletBull'. There is no security-relevant change visible in the diff.
No security action required; treat as a routine UI bug fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch updates two UI source files. In gui_wallet_tutorial_widgets.c it registers a new tutorial entry for WALLET_LIST_BULL using BULL_WALLET_TITLE and BULL_WALLET_LINK macros. In gui_connect_wallet_widgets.c it replaces an apparently incorrect icon reference ‘&walletXBull’ with ‘&walletBull’ in the BTC wallet coin array. No cryptographic, parsing, memory-management, or authorization logic is modified.
Changed components
src/ui/gui_widgets/gui_wallet_tutorial_widgets.csrc/ui/gui_widgets/multi/web3/gui_connect_wallet_widgets.cInspect captured patch +8 / −1
diff --git a/src/ui/gui_widgets/gui_wallet_tutorial_widgets.c b/src/ui/gui_widgets/gui_wallet_tutorial_widgets.c
index 1f31498..0ec9307 100644
--- a/src/ui/gui_widgets/gui_wallet_tutorial_widgets.c
+++ b/src/ui/gui_widgets/gui_wallet_tutorial_widgets.c
@@ -460,6 +460,13 @@ static void WalletTutorialsInit()
g_tutorials[WALLET_LIST_BABYLON].items[0].qrTitle = "Babylon";
g_tutorials[WALLET_LIST_BABYLON].items[0].qrUrl = "https://keyst.one/t/3rd/babylon";
+ g_tutorials[WALLET_LIST_BULL].len = 1;
+ g_tutorials[WALLET_LIST_BULL].desc = _("connect_wallet_desc");
+ g_tutorials[WALLET_LIST_BULL].items[0].walletName = BULL_WALLET_TITLE;
+ g_tutorials[WALLET_LIST_BULL].items[0].url = BULL_WALLET_LINK;
+ g_tutorials[WALLET_LIST_BULL].items[0].qrTitle = BULL_WALLET_TITLE;
+ g_tutorials[WALLET_LIST_BULL].items[0].qrUrl = BULL_WALLET_LINK;
+
g_tutorials[WALLET_LIST_BITCOIN_SAFE].len = 1;
g_tutorials[WALLET_LIST_BITCOIN_SAFE].desc = _("connect_wallet_desc");
g_tutorials[WALLET_LIST_BITCOIN_SAFE].items[0].walletName = "Bitcoin Safe";
diff --git a/src/ui/gui_widgets/multi/web3/gui_connect_wallet_widgets.c b/src/ui/gui_widgets/multi/web3/gui_connect_wallet_widgets.c
index 8b35944..2e4acc0 100644
--- a/src/ui/gui_widgets/multi/web3/gui_connect_wallet_widgets.c
+++ b/src/ui/gui_widgets/multi/web3/gui_connect_wallet_widgets.c
@@ -195,7 +195,7 @@ static const lv_img_dsc_t *g_ThorWalletCoinArray[3] = {
};
static const lv_img_dsc_t *g_btcWalletCoinArray[] = {
- &walletBluewallet, &walletSparrow, &walletZeus, &walletBabylon, &walletXBull,
+ &walletBluewallet, &walletSparrow, &walletZeus, &walletBabylon, &walletBull,
};
static const lv_img_dsc_t *g_adaCoinArray[1] = {
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.