Merge pull request #3 from blooo-io/switch-to-strnlen

fix: switch from strlen to strnlen
blooo-io · Oct 29, 2021 · 10b976a · 10b976a
2 parents 6e5cfaa + 8b2cff2
commit 10b976a
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 2 deletions.
diff --git a/src/handle_query_contract_ui.c b/src/handle_query_contract_ui.c
@@ -6,8 +6,8 @@ static void prepend_ticker(char *dest, size_t destsize, const char *ticker) {
     if (dest == NULL || ticker == NULL) {
         THROW(0x6503);
     }
-    uint8_t ticker_len = strlen(ticker);
-    uint8_t dest_len = strlen(dest);
+    size_t ticker_len = strnlen(ticker, MAX_TICKER_LEN);
+    size_t dest_len = strnlen(dest, destsize - ticker_len);
 
     if (dest_len + ticker_len >= destsize) {
         THROW(0x6503);

diff --git a/tests/elfs/paraswap_nanos.elf b/tests/elfs/paraswap_nanos.elf
diff --git a/tests/elfs/paraswap_nanox.elf b/tests/elfs/paraswap_nanox.elf