TZ
← All projectsTrezor

Trezor firmware

Firmware monorepo for Trezor One, Model T, and Safe devices.

BitcoinHardware walletsNormal
Repository coverage

2703 commits in the local evidence base

Every captured commit receives deterministic security triage and a separate communication-quality score. Security candidates and broader second-pass signals receive full-patch Ollama analysis.

932security candidates315second-pass queue2675AI analyses
320commits · 30 days
564commits · 60 days
1361commits · 180 days
2624commits · 365 days
Backfill bands
Aug 5 → Feb 61298 seen115 candidatesComplete
Feb 6 → Jun 6775 seen58 candidatesComplete
Jun 6 → Jul 6217 seen13 candidatesComplete
Jul 6 → Aug 5360 seen54 candidatesComplete
Commit communication

Does the history explain itself?

Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.

66/100 average clarity
490Strong · 80–100
1339Adequate · 60–79
869Thin · 40–59
5Opaque · 0–39
Read the scoring rubric →
Developer activity

Who is changing the project?

Public Git author strings; identities are not independently verified.

DeveloperCommitsCandidatesAnalyzedHigh riskMessage avg.
tychovrahe292104292461
Roman Zeyde587185580272
PrisionMike945994273
Andrew Kozlik782976267
obrusvit21376213164
Jakub Janků22919182
Martin Pastyřík24823173
cepetr19982196059
Ioan Bizău23076230059
M1nd3r21173208071
Lukas Bielesch846784067
Martin Milata18146176063
Analysis record

Published AI watches

Last scanned 1 hour ago

Low 37 AI analysisMessage 100 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

fix(core/fido): don't fail if `_KEEPALIVE_STATUS_PROCESSING` is blocked

This update fixes a bug in the Trezor hardware wallet's FIDO2/WebAuthn support. Previously, when the device was busy and tried to send a 'still processing' keepalive message to the computer, it would crash the whole FIDO operation if the U…

Denial-of-service hardening: prevents a transient USB buffer-full condition from aborting a FIDO authentication/registration workflowException boundary added around low-level USB HID writeChangelog explicitly frames the change as a fix for FIDO interaction failure
e7e15170by Roman Zeyde+12−62 files
No security note in commit
Informational 18 AI analysisMessage 85 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(core): drop `USBIF.write_blocking()` and simplify FIDO2 keep-alive sending

This commit is a small internal cleanup in the Trezor hardware wallet's FIDO2 (WebAuthn) code. It removes an unused 'blocking write' USB function and replaces a general synchronous send routine with a simpler one that only sends one-byte '…

Removal of unused blocking USB write path reduces attack surfaceNo change to FIDO2 command parsing, signature generation, or credential storage logicNo bounds-checking or input-validation changes observed
f269eccdby Roman Zeyde+8−513 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

chore: remove libtropic's pin to develop branch

This commit removes a single line from the project's Git submodule configuration. It stops pinning the libtropic submodule to the 'develop' branch, meaning the project will now use the submodule's default branch (likely 'main' or 'master')…

8290eeefby M1nd3r+0−11 file
No security note in commit
Low 37 AI analysisMessage 70 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

chore(core/embed): clear temporary buffer across Optiga/Tropic calls

This is a hardening change for the random number generator inside Trezor hardware wallets. It makes sure that temporary memory holding secret random values from the Optiga and Tropic security chips is wiped immediately after use, rather th…

Sensitive intermediate buffer not cleared promptlyPotential stale-buffer reuse across RNG source boundariesDefense-in-depth memory sanitization for secure-element entropy
01ebda93by Roman Zeyde+2−21 file
No security note in commit
Informational 15 AI analysisMessage 72 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

test(core/ethereum): EIP-7702 tuple nonce must be non-zero

This commit only changes test data and expected screen fingerprints for Trezor's Ethereum EIP-7702 signing tests. It replaces one test case that used a nonce of 0 with the same scenario using a nonce of 67, because the real-world protocol …

Test-only change with no production code modificationEIP-7702 tuple nonce constraint reflected in test dataExpected UI test hashes regenerated for renamed test vector
76cd2ed4by Roman Zeyde+28−282 files
No security note in commit
Informational 15 AI analysisMessage 72 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

test(core): drop an unused parameter of `make_payment_request()`

This commit only cleans up test code by removing an unused 'session' argument from a helper function used in automated tests. It does not change the actual Trezor firmware or any code that runs on the device, so it has no security impact f…

6dd19fdcby Roman Zeyde+0−188 files
No security note in commit
Informational 15 AI analysisMessage 67 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

docs(core): document rng_fill_buffer_strong_time()

This commit only adds a plain-English comment explaining an existing function that estimates how long a random-number generation call takes. No code behavior changed, and nothing was fixed or added that affects security.

c5687f76by Andrew Kozlik+11−01 file
No security note in commit
Low 44 AI analysisMessage 77 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

chore: guard against insecure PRNG in bare-metal build

This commit adds compile-time guards to prevent an intentionally insecure random-number generator from being accidentally included in firmware that runs directly on Trezor hardware. It does not fix an active bug, but it adds safety rails s…

Compile-time guard around insecure PRNGExplicit block of bare-metal / freestanding targetsExisting production-build guard already present
4f8f4586by Andrew Kozlik+11−01 file
No security note in commit
Informational 15 AI analysisMessage 85 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

build(core): propagate emulator feature from projects

This commit is a build-system cleanup for the Trezor firmware's Rust code. It changes how the 'emulator' feature flag is passed between different software components. The firmware project no longer supports the emulator feature (which make…

5642074cby matejcik+5−44 files
No security note in commit
Informational 15 AI analysisMessage 85 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

fix(build-docker): stop swallowing pub-bin copy errors

This change fixes a build script that was ignoring copy errors. Previously, the script would silently ignore any failure when copying public binary files, which could hide real problems. Now it only skips the copy when the file genuinely d…

Build script silently ignored copy failures before this fixFix removes unconditional `|| true` error suppressionKnown-benign missing files are now explicitly skipped with a file existence check
d829eda8by Andrew Kozlik+4−11 file
No security note in commit
Low 27 AI analysisMessage 93 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(core): digest EIP-1559/EIP-7702 magic without RLP encoding

This commit is a small internal cleanup in Trezor's Ethereum signing code. It changes how two special 'magic' bytes (the transaction type markers for newer Ethereum transaction formats) are fed into the cryptographic hash. Previously the c…

Refactor of cryptographic hashing path for Ethereum transaction signingRemoval of implicit assumption that RLP encodes small integers as single bytesChange affects EIP-1559 and EIP-7702 transaction type markers
2362bc6fby Roman Zeyde+6−32 files
No security note in commit
Informational 15 AI analysisMessage 98 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

test(core): test RLP self-encoding bytestrings

This commit only adds a new unit test to the Trezor firmware codebase. It verifies that very short Ethereum RLP-encoded byte strings and small positive integers encode to themselves, which is a known property of the RLP specification. No p…

41538733by Roman Zeyde+16−01 file
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

docs: fix links outside book root

This commit only fixes broken or awkward documentation links inside the project's user guide. It replaces relative links that point outside the documentation folder with internal placeholder files or direct GitHub URLs. There is no change …

0d8d4b1bby Martin Milata+14−810 files
No security note in commit
Informational 18 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

chore(core): remove sha1 from regular FW

This commit removes the SHA-1 hashing function from the regular Trezor firmware. SHA-1 is an old, weak hash algorithm that is no longer considered secure for sensitive uses. The change deletes the code that exposes SHA-1 to apps running on…

Removal of a deprecated cryptographic primitive (SHA-1) from the firmware API surfaceReduction of attack surface and prevention of future misuse of a collision-vulnerable hashNo direct vulnerability patch or memory-safety bug is present in the diff
16d15774by M1nd3r+0−2006 files
No security note in commit
Low 27 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(crypto): pass static public key to `noise_xxpsk3_*_init()`

This commit changes how a cryptographic library sets up secure connections. Previously, the code automatically calculated the public key from the private key. Now, the caller must provide the public key directly. This is a code-quality ref…

Cryptographic key handling changed: public key is now supplied rather than derivedPotential reduction of key-mismatch risk if caller provides correct public keyNew null-pointer check added for static_public_key
1b3128a4by Ondřej Vejpustek+20−92 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(crypto): change order of parameters

This commit simply reorders the arguments of an internal function called dh() and updates every place that calls it. The actual math and security behavior are unchanged; it is a code cleanup with no security effect.

33a119eeby Ondřej Vejpustek+15−151 file
No security note in commit
Informational 15 AI analysisMessage 47 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(crypto): prefix enums

This commit simply renames two groups of internal status labels (called enums) in the Trezor firmware's cryptographic code. The old names like WAITING_FOR_REQUEST1 were shared between two different parts of the code, so the developer gave …

f1c74046by Ondřej Vejpustek+26−222 files
No security note in commit
Low 37 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

fix(crypto): prevent calling `memzero(NULL, ...)`

This commit fixes three places in the Trezor firmware's cryptographic code where a memory-clearing function could be called with a NULL pointer. In practice, passing NULL to memzero is harmless on Trezor's platform (it does nothing), but i…

NULL pointer passed to memory-zeroing helper in cryptographic codeUndefined behavior in C standard library contractDefensive hardening in Noise protocol implementation
0394e934by Ondřej Vejpustek+11−51 file
No security note in commit
Informational 18 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(crypto): return remote static key from `noise_xxpsk3_*_handle_*()`

This commit is a code cleanup (refactor) for the cryptographic handshake code used in Trezor devices. It changes how the other party's long-term public key is returned to the caller: instead of storing it inside an internal state structure…

Removal of long-term public key storage from internal handshake stateCaller-supplied output buffer for remote static public key reduces internal secret retentionError-path memzero of returned key material on failure
3ead1aa6by Ondřej Vejpustek+79−423 files
No security note in commit
Informational 15 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): rename ProjectProfile to ProjectConfig

This commit is a simple rename from 'ProjectProfile' to 'ProjectConfig' in two Rust source files. It only changes variable names, struct names, and error messages. There is no change to program logic, security behavior, or how data is hand…

581b37bcby cepetr+18−182 files
No security note in commit
Repository ledger

Explore captured commits

Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.

Lower-priorityfeat(core): firmware suspend by power buttonby obrusvit · dbda8d0d · Aug 21, 2025 · 3 filesMessage 72 · AdequateInformational 18Details
Commit message · obrusvit

feat(core): firmware suspend by power button

- power button event is handled before the event is passed to rust
layout

[no changelog]

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
AI analysis · Informational 18/100

This commit adds a new power-saving feature: pressing the physical power button on a Trezor device can now suspend the firmware. If the device has a PIN and is currently unlocked, the button press first locks the device (ending any active workflow) and then triggers a shutdown. Otherwise, the device is suspended and later woken up. The change is a feature addition, not a fix for a known security bug, and there is no evidence in the commit or supplied references that it addresses a vulnerability.

Lower-priorityfix(core): increase prodtest stack sizeby cepetr · a6f6dbb3 · Aug 21, 2025 · 1 fileMessage 57 · ThinLow 28Details
Commit message · cepetr

fix(core): increase prodtest stack size

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Low 28/100

This commit doubles the stack memory size for the Trezor hardware wallet's production-test firmware from 12 KB to 24 KB. A stack that is too small can cause crashes (the commit notes 'Overflow causes UsageFault'). There is no direct evidence in the commit that this is exploitable as a security vulnerability, and the change appears to be a stability fix for a special manufacturing/QA mode rather than the normal user-facing wallet firmware.

AI review queuedbuild(core): fix dependency versions in trezor-core-toolsby matejcik · b14e3432 · Aug 21, 2025 · 3 filesMessage 97 · StrongInformational 15Details
Commit message · matejcik

build(core): fix dependency versions in trezor-core-tools

otherwise it's not possible to `poetry lock` when one of trezor or
trezor-core-tools is touched, because their version specs clash

97/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
second-pass: broader security terminology
AI analysis · Informational 15/100

This commit is a routine build/dependency housekeeping change. It updates Python version constraints and the internal 'trezor' package version requirement so that the project's Poetry lock file can be regenerated without conflicts. There is no indication this fixes a security vulnerability or changes runtime behavior of the Trezor firmware or wallet software.

Security candidatefeat(core/bootloader): stronger haptic effect on power onby tychovrahe · f947c4c6 · Aug 21, 2025 · 3 filesMessage 62 · AdequateInformational 15Details
Commit message · tychovrahe

feat(core/bootloader): stronger haptic effect on power on

[no changelog]

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
update trustboot or update path
AI analysis · Informational 15/100

This commit changes the vibration pattern that the Trezor hardware wallet produces when it powers on. It adds a new dedicated 'power on' haptic effect and uses it instead of reusing the existing button-press vibration. There is no security relevance in this change.

Lower-priorityfeat(core): reimplement THP retransmissionsby Roman Zeyde · 3ca95252 · Aug 21, 2025 · 2 filesMessage 57 · ThinLow 34Details
Commit message · Roman Zeyde

feat(core): reimplement THP retransmissions

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Low 34/100

This commit finishes a previously stubbed-out feature in Trezor's core firmware: automatic retransmission of lost USB packets during a secure protocol called THP. Before this change, the code had a TODO comment where retransmissions should have been. After this change, the device will resend a packet up to 50 times with an increasing timeout if it does not receive an acknowledgment. This is a normal reliability improvement, but because it touches low-level packet handling and timeout logic, it could introduce subtle bugs such as the device hanging, retrying forever in edge cases, or misinterpreting a delayed packet as a fresh one.

Lower-prioritytest(core): wait for device availability to avoid UI tests flakinessby Roman Zeyde · 12c045d4 · Aug 21, 2025 · 1 fileMessage 84 · StrongInformational 15Details
Commit message · Roman Zeyde

test(core): wait for device availability to avoid UI tests flakiness

[no changelog]

84/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Explains rationale or failure mode✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 15/100

This is a test-only change that replaces a fixed 0.1-second sleep with an explicit device-readiness check before starting screen recordings in UI tests. It does not touch firmware, wallet logic, cryptography, or any production code path, so it has no direct security relevance for end users.

Lower-prioritytest(core): wait for debuglink task before restarting THP event loopby Roman Zeyde · 5bb49f1e · Aug 21, 2025 · 1 fileMessage 72 · AdequateInformational 17Details
Commit message · Roman Zeyde

test(core): wait for debuglink task before restarting THP event loop

[no changelog]

72/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 17/100

This is a test-only change for the Trezor hardware wallet firmware. It adds a small cleanup step in the debug/test code path that waits for a debug-link helper task to finish before restarting the event loop used by the new THP (Trezor Host Protocol) code. It does not change normal user-facing behavior and is not a security fix for a live vulnerability.

Security candidatefeat(core): reimplement THP event loop restartsby Roman Zeyde · c3200208 · Aug 21, 2025 · 21 filesMessage 57 · ThinLow 38Details
Commit message · Roman Zeyde

feat(core): reimplement THP event loop restarts

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
boot or update pathparser or protocol path
AI analysis · Low 38/100

This commit is a large internal refactor of the Trezor firmware's THP (Trezor Host Protocol) event loop and message handling. It removes an old global buffer-locking scheme and a separate retransmission task, replacing them with per-channel buffer objects and a simpler synchronous send-and-wait-for-ACK flow. The change also removes the old `loop.clear()` restart-on-error behavior in favor of explicitly joining workflows before restarting. The diff itself is a feature rewrite with no explicit security claims, but it touches sensitive code paths (encrypted channel setup, session scheduling, buffer management) and removes some safety checks, so it should be reviewed carefully for regressions.

Lower-priorityfix(core/prodtest): fix conditional usage of tropic testing keysby tychovrahe · 8e51245b · Aug 21, 2025 · 1 fileMessage 72 · AdequateLow 32Details
Commit message · tychovrahe

fix(core/prodtest): fix conditional usage of tropic testing keys

[no changelog]

72/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Low 32/100

This is a one-line fix in Trezor's factory production-test code for the Tropic secure chip. It changes a preprocessor condition from an apparently wrong/placeholder macro name 'ABAB' to the intended 'TROPIC_TESTING_KEYS'. The effect is to ensure that testing-only cryptographic keys are compiled in only when the proper configuration flag is set, rather than under an unrelated or accidental flag. This reduces the risk that production devices could be built with test keys instead of real factory keys.

Lower-priorityfix(prodtest): fix secrets-certdev-write arg count checkby cepetr · 755f6ab2 · Aug 21, 2025 · 2 filesMessage 62 · AdequateLow 37Details
Commit message · cepetr

fix(prodtest): fix secrets-certdev-write arg count check

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Low 37/100

This commit fixes a command in Trezor's production-test firmware that checks how many arguments a user passed. The old check rejected commands with more than zero arguments, but accidentally allowed commands with zero arguments through. The fix requires exactly one argument. The affected command is part of a low-level factory/testing tool, not the normal wallet firmware users run every day.

Lower-priorityfix(prodtest): fix wpc-info command outputby cepetr · ac426c6b · Aug 21, 2025 · 3 filesMessage 57 · ThinInformational 15Details
Commit message · cepetr

fix(prodtest): fix wpc-info command output

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 15/100

This commit fixes the formatting of a debug/production-test command output for a wireless charging chip. It changes how numbers are printed (decimal to hexadecimal) and moves the data values from a 'PROGRESS' line to the 'OK' line. There is no security issue here—it's a cosmetic/output-formatting fix in a factory testing tool.

Lower-priorityfix(prodtest): remove extra crlf in response in non-interactive modeby cepetr · 441283c1 · Aug 21, 2025 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · cepetr

fix(prodtest): remove extra crlf in response in non-interactive mode

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 15/100

This commit fixes a minor formatting bug in the Trezor production-test command-line tool. Previously, every response line in non-interactive mode had an extra blank line (CRLF) appended. The change moves that blank line so it only appears in interactive mode, where it serves as a prompt separator. There is no security relevance in the commit or diff.

AI review queuedrefactor(python): inline `ProtocolV2Channel.prepare_channel_without_pairing()`by Roman Zeyde · 3406932f · Aug 21, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

refactor(python): inline `ProtocolV2Channel.prepare_channel_without_pairing()`

[no changelog]

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
parser or protocol pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit is a simple code cleanup: it removes a small helper method and places its contents directly into the function that called it. No behavior changes, no security fixes, and no new risks are visible.

Lower-prioritychore(core): remove forgotten `log.info()` in `interface_context.py`by Roman Zeyde · e451adc9 · Aug 21, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

chore(core): remove forgotten `log.info()` in `interface_context.py`

[no changelog]

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 15/100

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.

AI review queuedrefactor(python): don't return SEQ bit for irrelevant THP messagesby Roman Zeyde · a303f9c9 · Aug 21, 2025 · 2 filesMessage 62 · AdequateInformational 17Details
Commit message · Roman Zeyde

refactor(python): don't return SEQ bit for irrelevant THP messages

[no changelog]

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
parser or protocol pathsecond-pass: security-sensitive path
AI analysis · Informational 17/100

This is a small code cleanup in Trezor's Python library for a newer transport protocol (THP). It changes a helper function so it returns 'no sequence bit' for message types that don't carry one, and adds an assertion to catch any unexpected case. The change is labeled a refactor with no changelog and appears defensive rather than a fix for an active security bug.

AI review queuedfeat(python): ignore retransmitted THP payloadsby Roman Zeyde · 8b19697a · Aug 21, 2025 · 1 fileMessage 57 · ThinLow 35Details
Commit message · Roman Zeyde

feat(python): ignore retransmitted THP payloads

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
parser or protocol pathsecond-pass: security-sensitive path
AI analysis · Low 35/100

This commit changes the Trezor Python library's handling of messages sent over the new THP (Trezor Host Protocol) transport. Previously, if a message had a bad checksum, the code would read a new message but did not track whether the newly received message was a fresh one or an old retransmission. The new code loops until it receives a message that passes the checksum and also has the expected sequence/sync bit, ignoring stale or out-of-order retransmissions. This is a hardening change: it makes the host-side protocol implementation more robust against duplicate or replayed packets, which could otherwise confuse state machines or cause commands to be processed twice.

Lower-priorityfeat(core): join debuglink task before restarting Codec v1 event loopby Roman Zeyde · d9286ea4 · Aug 21, 2025 · 1 fileMessage 62 · AdequateInformational 22Details
Commit message · Roman Zeyde

feat(core): join debuglink task before restarting Codec v1 event loop

[no changelog]

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 22/100

This is a small cleanup change in the Trezor firmware's internal message-handling loop. It makes sure a special developer/debug task is fully shut down before the device resets its communication loop. The change only affects debug builds (not normal user devices) and appears designed to prevent a stale debug session from surviving across a loop restart. There is no direct evidence in the commit that this fixes an exploitable security vulnerability in production.

Lower-priorityrefactor(core): step `loop.race()` task without `loop.schedule()`by Roman Zeyde · 308f9e9f · Aug 21, 2025 · 1 fileMessage 62 · AdequateInformational 11Details
Commit message · Roman Zeyde

refactor(core): step `loop.race()` task without `loop.schedule()`

[no changelog]

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 11/100

This is a one-line internal code cleanup in Trezor's task scheduler. It changes how a background racing task is advanced, switching from a public scheduling function to a direct internal stepping call. There is no visible security problem in the diff itself, and no security context is provided by the vendor.

Lower-prioritytest(core): allow joining debuglink task before restarting event loopby Roman Zeyde · d9a5a3ce · Aug 21, 2025 · 2 filesMessage 72 · AdequateInformational 16Details
Commit message · Roman Zeyde

test(core): allow joining debuglink task before restarting event loop

[no changelog]

72/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 16/100

This commit changes internal test-only debug code in the Trezor firmware so that the debug communication session can be cleanly shut down between test scenarios. It adds a flag, a mailbox signal, and a timeout-based join to prevent the debug task from getting stuck when the test framework restarts the device's event loop. The production security relevance appears low because the code is wrapped in `if __debug__` and only used during automated UI tests.

Lower-priorityfix(core): set model variant in THP DevicePropertiesby Martin Milata · ebf0d7c2 · Aug 20, 2025 · 2 filesMessage 62 · AdequateInformational 23Details
Commit message · Martin Milata

fix(core): set model variant in THP DeviceProperties

[no changelog]

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 23/100

This commit finishes a previously incomplete feature in Trezor's new wireless pairing protocol (THP). Before the fix, the device told the connecting computer/phone its model variant as 'None' (effectively blank). After the fix, it correctly reports the device's color, whether it is a Bitcoin-only edition, and its packaging type. This is mostly a correctness/functional fix, but an incorrect or missing model variant could in theory confuse host software during pairing or cause it to trust the wrong device identity. There is no direct evidence in the commit of an exploitable vulnerability.

Security candidaterefactor(core): improve Stellar signing UXby Jun Luo · 6dc393af · Aug 20, 2025 · 24 filesMessage 57 · ThinLow 30Details
Commit message · Jun Luo

refactor(core): improve Stellar signing UX

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing boundarycryptography-sensitive pathsigning or wallet path
AI analysis · Low 30/100

This commit is a user-interface refactor for Stellar transaction signing on Trezor hardware wallets. It changes the wording and order of confirmation screens, adds warnings when the transaction source account does not belong to the device, and streamlines how operation outputs are shown. There is no direct evidence in the diff of a security vulnerability; it appears to be a UX improvement. However, any change to what users confirm before signing deserves a careful look to ensure nothing important was removed or weakened.

Security candidatefeat(core/eckhart): regulatory screenby Lukas Bielesch · 0e72c217 · Aug 19, 2025 · 22 filesMessage 57 · ThinInformational 15Details
Commit message · Lukas Bielesch

feat(core/eckhart): regulatory screen

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathboot or update path
AI analysis · Informational 15/100

This commit adds a new 'Regulatory certification' screen to the Trezor hardware wallet's user interface. It is purely a UI feature that displays certification marks (FCC, CE, RCM, etc.) and regional compliance information. There is no security-relevant code change, no handling of secrets, and no network or cryptographic logic.

Lower-priorityfix(core): unify power manager timings.by kopecdav · e06b3311 · Aug 19, 2025 · 3 filesMessage 57 · ThinInformational 17Details
Commit message · kopecdav

fix(core): unify power manager timings.

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 17/100

This commit simplifies the Trezor hardware wallet's power-monitoring code by using one timer interval (100 ms) everywhere instead of two different intervals (100 ms and 300 ms). It removes a duplicate timer setting and renames the constant. There is no clear security bug being fixed; it appears to be a cleanup or consistency improvement in how often the device checks its battery and power state.

Security candidatefeat(core/eckhart): add label keyboard layoutby Lukas Bielesch · e3fcde17 · Aug 19, 2025 · 33 filesMessage 72 · AdequateInformational 19Details
Commit message · Lukas Bielesch

feat(core/eckhart): add label keyboard layout

- utilize all passphrase input parameters
- add label changing device menu action
[no changelog]

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
access controlcryptography-sensitive pathsigning or wallet pathboot or update pathauthentication path
AI analysis · Informational 19/100

This commit adds a new on-device screen for changing the device's name (label) on the upcoming Trezor 'Eckhart' model. It also makes the passphrase input code more flexible by allowing the caller to specify the prompt text and maximum length, and it wires up a new 'Device name' menu item. There is no obvious security bug in the diff, but it is a large feature patch touching keyboard input, menu navigation, and device settings, so it carries normal implementation risks.

Lower-priorityfix(core): fine tune BLE behavior after unexpected eventsby tychovrahe · e5e2e060 · Aug 19, 2025 · 1 fileMessage 77 · AdequateInformational 24Details
Commit message · tychovrahe

fix(core): fine tune BLE behavior after unexpected events

i.e. where the is nRF reset, we want to restart the pairing if there was paring mode before

[no changelog]

77/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
AI analysis · Informational 24/100

This commit adjusts how a Trezor hardware wallet's Bluetooth subsystem behaves after unexpected resets or state changes in its wireless companion chip (nRF). It makes pairing restart automatically if it was interrupted, stops advertising when no bonded peers exist, and ends pairing cleanly when a connection is established. These are robustness fixes to Bluetooth state management rather than obvious security patches, though poor BLE state handling can in principle create attack windows.