What changed, and why it matters
This commit fixes a user-interface bug for Zcash in the Keystone 3 hardware wallet firmware. It adds a missing UI handler so the Zcash transaction overview screen can be displayed, and adds the Zcash icon to the status bar. There is no indication this changes security-sensitive code or fixes a vulnerability.
No security action needed; treat as a routine UI fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch registers ‘GuiZcashOverview’ in the custom container lookup table in gui_general_analyze.c and adds CHAIN_ZCASH with the ZEC coin icon to the status bar button array in gui_status_bar.c. These are purely UI wiring changes for Zcash support. No cryptographic, parsing, memory-management, or authorization logic is modified.
Changed components
src/ui/gui_analyze/multi/web3/gui_general_analyze.csrc/ui/gui_components/gui_status_bar.cInspect captured patch +3 / −0
diff --git a/src/ui/gui_analyze/multi/web3/gui_general_analyze.c b/src/ui/gui_analyze/multi/web3/gui_general_analyze.c
index c2156ba..c7f3bbc 100644
--- a/src/ui/gui_analyze/multi/web3/gui_general_analyze.c
+++ b/src/ui/gui_analyze/multi/web3/gui_general_analyze.c
@@ -111,6 +111,8 @@ GetCustomContainerFunc GetOtherChainCustomFunc(char *funcName)
return GuiIotaTxRawData;
} else if (!strcmp(funcName, "GuiCustomPathNotice")) {
return GuiCustomPathNotice;
+ } else if (!strcmp(funcName, "GuiZcashOverview")) {
+ return GuiZcashOverview;
}
return NULL;
diff --git a/src/ui/gui_components/gui_status_bar.c b/src/ui/gui_components/gui_status_bar.c
index 370c51f..7a47ca8 100644
--- a/src/ui/gui_components/gui_status_bar.c
+++ b/src/ui/gui_components/gui_status_bar.c
@@ -124,6 +124,7 @@ const static CoinWalletInfo_t g_coinWalletBtn[] = {
{CHAIN_QCK, "", &coinQck},
{CHAIN_TGD, "", &coinTgd},
{CHAIN_DOT, "", &coinDot},
+ {CHAIN_ZCASH, "", &coinZec},
#endif
#ifdef CYPHERPUNK_VERSION
Why this scored 12/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.