Skip to content

Commit 902e584

Browse files
authored
fix(legacy): fix OK-22577 (#469)
1 parent 945a0f8 commit 902e584

File tree

5 files changed

+9
-15
lines changed

5 files changed

+9
-15
lines changed

legacy/firmware/fsm_msg_algorand.h

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,8 @@ void fsm_msgAlgorandGetAddress(AlgorandGetAddress *msg) {
3333
algorand_get_address_from_public_key(node->public_key + 1, resp->address);
3434

3535
if (msg->has_show_display && msg->show_display) {
36-
char desc[16] = {0};
37-
strcat(desc, "Algorand");
38-
strcat(desc, _("Address:"));
36+
char desc[20] = {0};
37+
snprintf(desc, 20, "%s %s", "Algorand", _("Address:"));
3938
if (!fsm_layoutAddress(resp->address, desc, false, 0, msg->address_n,
4039
msg->address_n_count, true, NULL, 0, 0, NULL)) {
4140
return;

legacy/firmware/fsm_msg_conflux.h

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,9 +57,8 @@ void fsm_msgConfluxGetAddress(const ConfluxGetAddress *msg) {
5757
return;
5858
}
5959
if (msg->has_show_display && msg->show_display) {
60-
char desc[16] = {0};
61-
strcat(desc, "Conflux");
62-
strcat(desc, _("Address:"));
60+
char desc[20] = {0};
61+
snprintf(desc, 20, "%s %s", "Conflux", _("Address:"));
6362
if (!fsm_layoutAddress(resp->address, desc, false, 0, msg->address_n,
6463
msg->address_n_count, false, NULL, 0, 0, NULL)) {
6564
return;

legacy/firmware/fsm_msg_ethereum_onekey.h

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -185,9 +185,7 @@ void fsm_msgEthereumGetAddressOneKey(const EthereumGetAddressOneKey *msg) {
185185
} else {
186186
ASSIGN_ETHEREUM_NAME(chain_name, 0); // unknown chain
187187
}
188-
strcat(desc, chain_name);
189-
strcat(desc, " ");
190-
strcat(desc, _("Address:"));
188+
snprintf(desc, 32, "%s %s", chain_name, _("Address:"));
191189
if (!fsm_layoutAddress(resp->address, desc, false, 0, msg->address_n,
192190
msg->address_n_count, true, NULL, 0, 0, NULL)) {
193191
return;

legacy/firmware/fsm_msg_filecoin.h

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,8 @@ void fsm_msgFilecoinGetAddress(const FilecoinGetAddress *msg) {
3939
}
4040
if (!get_filecoin_addr(pk, resp)) return;
4141
if (msg->has_show_display && msg->show_display) {
42-
char desc[16] = {0};
43-
strcat(desc, "Filecoin");
44-
strcat(desc, _("Address:"));
42+
char desc[20] = {0};
43+
snprintf(desc, 20, "%s %s", "Filecoin", _("Address:"));
4544
if (!fsm_layoutAddress(resp->address, desc, false, 0, msg->address_n,
4645
msg->address_n_count, true, NULL, 0, 0, NULL)) {
4746
return;

legacy/firmware/fsm_msg_starcoin.h

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,8 @@ void fsm_msgStarcoinGetAddress(const StarcoinGetAddress *msg) {
3535
starcoin_get_address_from_public_key(node->public_key + 1, resp->address + 2);
3636

3737
if (msg->has_show_display && msg->show_display) {
38-
char desc[16] = {0};
39-
strcat(desc, "Starcoin");
40-
strcat(desc, _("Address:"));
38+
char desc[20] = {0};
39+
snprintf(desc, 20, "%s %s", "Starcoin", _("Address:"));
4140
if (!fsm_layoutAddress(resp->address, desc, false, 0, msg->address_n,
4241
msg->address_n_count, true, NULL, 0, 0, NULL)) {
4342
return;

0 commit comments

Comments
 (0)