Skip to content

Commit 930d6ad

Browse files
authored
Merge pull request #266 from keepkey/pub-710
710 merge
2 parents 3ddcc5d + e49d455 commit 930d6ad

File tree

3 files changed

+3
-4
lines changed

3 files changed

+3
-4
lines changed

lib/firmware/ethereum.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -93,10 +93,10 @@ uint8_t ethereum_extractThorchainData(const EthereumSignTx *msg,
9393
// offset = deposit function hash + address + address + uint256
9494
uint16_t offset = 4 + (5 * 32);
9595
int16_t len = msg->data_length - offset;
96-
if (msg->has_data_length && len > 0) {
96+
if (msg->has_data_length && len > 0 && len < 256) {
9797
memcpy(buffer, msg->data_initial_chunk.bytes + offset, len);
9898
// String length must be < 255 characters
99-
return len < 256 ? (uint8_t)len : 0;
99+
return (uint8_t)len;
100100
}
101101
return 0;
102102
}

lib/firmware/ethereum_contracts/zxliquidtx.c

-1
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,6 @@ bool zx_confirmZxLiquidTx(uint32_t data_total, const EthereumSignTx *msg) {
168168
confirm(ButtonRequestType_ButtonRequest_ConfirmOutput, arStr,
169169
"Minimum %s", constr1);
170170

171-
snprintf(constr1, 32, "%lld", deadline);
172171
confirm(ButtonRequestType_ButtonRequest_ConfirmOutput, arStr,
173172
"Deadline %s", ctime((const time_t *)&deadline));
174173

0 commit comments

Comments
 (0)