chore(core): remove forgotten `log.info()` in `interface_context.py`
What the developer wrote
Authored by Roman Zeyde
chore(core): remove forgotten `log.info()` in `interface_context.py`
[no changelog]
What changed, and why it matters
This commit removes a single leftover debug logging line that printed the contents of a channel allocation packet. It is a cleanup change with no security relevance: the log line was informational, not an error, and removing it does not fix a vulnerability or change any behavior an attacker could exploit.
No security action needed. Treat as routine cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff deletes one log.info(__name__, "got alloc: %s", utils.hexlify_if_bytes(packet)) call in core/src/trezor/wire/thp/interface_context.py. This was a forgotten debug/info log in the Trezor Host Protocol (THP) interface context. The change is purely cosmetic/cleanup: it does not alter control flow, input validation, memory handling, cryptography, or access control. No security bug is being fixed.
Changed components
core/src/trezor/wire/thp/interface_context.pyInspect captured patch +0 / −1
diff --git a/core/src/trezor/wire/thp/interface_context.py b/core/src/trezor/wire/thp/interface_context.py
index 963574700..f52a95f56 100644
--- a/core/src/trezor/wire/thp/interface_context.py
+++ b/core/src/trezor/wire/thp/interface_context.py
@@ -147,7 +147,6 @@ class ThpContext:
if ctrl_byte != CHANNEL_ALLOCATION_REQ:
raise ThpError("Unexpected ctrl_byte in a broadcast channel packet")
- log.info(__name__, "got alloc: %s", utils.hexlify_if_bytes(packet))
channel_cache = channel_manager.create_new_channel(self._iface)
channel = self._load_channel(channel_cache)
Why this scored 15/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.