fix avax from error message display
What the developer wrote
Authored by ww3512687
fix avax from error message display
What changed, and why it matters
This commit fixes a simple UI bug in the Avalanche (AVAX) transaction review screen. The code was accidentally passing the recipient's address length where the sender's address length should go, which could cause the sender's address to be displayed incorrectly or truncated on the hardware wallet screen. It does not appear to affect transaction signing logic or funds security.
Treat as a low-risk UI fix. No urgent security response required. Verify the sender address now renders fully and correctly in the AVAX transaction review screen during QA.
Security signals we found
UI display parameter mismatch in transaction review screen
Potential sender address truncation or misrendering
No change to signing, parsing, or cryptographic code
Evidence from the diff
In GuiAvaxTxOverview(), the call to CreateTxOverviewFromTo() previously used txData->data->to->size as the length argument for the ‘from’ field, when it should have used txData->data->from->size. The patch corrects this argument. This is a display/UI parameter mismatch in a single function call; the underlying transaction data structure is unchanged.
Changed components
src/ui/gui_chain/multi/web3/gui_avax.cAvalanche (AVAX) transaction overview UIInspect captured patch +1 / −1
diff --git a/src/ui/gui_chain/multi/web3/gui_avax.c b/src/ui/gui_chain/multi/web3/gui_avax.c
index cc8d8d7..de2456a 100644
--- a/src/ui/gui_chain/multi/web3/gui_avax.c
+++ b/src/ui/gui_chain/multi/web3/gui_avax.c
@@ -174,7 +174,7 @@ void GuiAvaxTxOverview(lv_obj_t *parent, void *totalData)
GuiAlignToPrevObj(container, LV_ALIGN_OUT_BOTTOM_LEFT, 0, 16);
}
- container = CreateTxOverviewFromTo(parent, txData->data->from->data, txData->data->to->size, txData->data->to->data, txData->data->to->size);
+ container = CreateTxOverviewFromTo(parent, txData->data->from->data, txData->data->from->size, txData->data->to->data, txData->data->to->size);
GuiAlignToPrevObj(container, LV_ALIGN_OUT_BOTTOM_LEFT, 0, 16);
lv_obj_update_layout(parent);
}
Why this scored 18/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.