fix(core): optimize message decode logic (#384)

Author: somebodyLi

core/src/apps/common/signverify.py

@@ -29,7 +29,16 @@ def message_digest(coin: CoinInfo, message: bytes) -> bytes:
 
 
 def is_non_printable(message: str) -> bool:
-    return any(ord(c) < 32 or ord(c) == 127 for c in message)
+    allowed_control_chars = {9, 10, 13}  # \t, \n, \r
+
+    has_disallowed_control = any(
+        (ord(c) < 32 and ord(c) not in allowed_control_chars) or ord(c) == 127
+        for c in message
+    )
+
+    has_printable_chars = any(32 <= ord(c) <= 126 for c in message)
+
+    return has_disallowed_control or not has_printable_chars
 
 
 def decode_message(message: bytes) -> str:

core/src/apps/ethereum/layout.py

@@ -153,7 +153,7 @@ async def show_invalid_delegate(ctx: Context) -> None:
 
 def require_confirm_eip7702(
     ctx: Context,
-    authorty_adrr: str,
+    authorty_addr: str,
     delegate_addr: str,
     authority_network: str,
     amount: int,
@@ -176,7 +176,7 @@ def require_confirm_eip7702(
     return confirm_total_eip7702(
         ctx,
         title,
-        authorty_adrr,
+        authorty_addr,
         delegate_addr if not is_revoke else None,
         format_ethereum_amount(amount, None, chain_id),
         str(nonce),

core/src/trezor/lvglui/scrs/template.py

@@ -1037,7 +1037,7 @@ class TransactionDetailsEIP7702(TransactionDetailsBase):
     def __init__(
         self,
         title: str,
-        authorty_adrr: str,
+        authorty_addr: str,
         delegate_addr: str | None,
         value: str,
         nonce: str,
@@ -1054,7 +1054,7 @@ def __init__(
             primary_color,
             icon_path,
             direction_section=(
-                (_(i18n_keys.FIELDS_ACCOUNT), authorty_adrr),
+                (_(i18n_keys.FIELDS_ACCOUNT), authorty_addr),
                 (_(i18n_keys.FIELDS_DELEGATE_TO), delegate_addr),
             ),
             more_section=(

core/src/trezor/ui/layouts/lvgl/altcoin.py

@@ -194,7 +194,7 @@ async def confirm_total_ethereum_eip1559(
 async def confirm_total_eip7702(
     ctx: wire.GenericContext,
     title: str,
-    authorty_adrr: str,
+    authorty_addr: str,
     delegate_addr: str | None,
     amount: str,
     nonce: str,
@@ -208,7 +208,7 @@ async def confirm_total_eip7702(
 
     screen = TransactionDetailsEIP7702(
         title,
-        authorty_adrr,
+        authorty_addr,
         delegate_addr,
         amount,
         nonce,