TZ
← All projectsTrezor

Trezor firmware

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

BitcoinHardware walletsNormal
Repository coverage

2673 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.

923security candidates311second-pass queue2662AI analyses
323commits · 30 days
552commits · 60 days
1337commits · 180 days
2644commits · 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
482Strong · 80–100
1323Adequate · 60–79
863Thin · 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 Zeyde579182576272
PrisionMike945994273
Andrew Kozlik712470265
obrusvit21376213164
Jakub Janků22919182
Martin Pastyřík24823173
cepetr19682196059
Ioan Bizău23076230059
M1nd3r20772207071
Lukas Bielesch846784067
Martin Milata17446174063
Analysis record

Published AI watches

Last scanned 11 minutes ago

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 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
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 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
Informational 15 AI analysisMessage 47 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): introduce ResolvedBuildArgs

This commit is a straightforward internal code refactor in Trezor's build tooling. It introduces a new ResolvedBuildArgs structure that centralizes default values for command-line build options before they are used. There is no change to u…

c24d1be8by cepetr+99−295 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): make build args tristate

This commit is a straightforward internal refactor of the Trezor firmware build tool's command-line argument handling. It changes many on/off flags from plain booleans to optional booleans so they can explicitly be set to true, false, or l…

271e249aby cepetr+57−561 file
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): introduce build options

This commit is a straightforward internal code reorganization in Trezor's build tooling. It moves build-option definitions from one Rust source file to a new module and adds the ability to load those options from a configuration file. Ther…

75085e2aby cepetr+275−2007 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

feat(xtask): introduce build presets

This commit is a developer tooling change for the Trezor firmware build system. It introduces 'build presets'—named configuration bundles stored in TOML files—so developers can select common build settings with a single command-line flag i…

eefe8f7fby cepetr+397−419 files
No security note in commit
Informational 15 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

feat(xtask): upgrade manifest.yaml for tf-tools 0.3.0

This commit is a routine update to a build-system manifest file used by Trezor's internal tooling. It reorganizes and re-labels feature flags (for example, moving options between 'Features', 'Debugging', and 'Signing' groups and simplifyin…

57b28b89by cepetr+86−1351 file
No security note in commit
Informational 15 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

feat(core): add tf-tools 0.3.1+ configuration file

This commit adds a single configuration file for a Trezor developer tool (a VS Code extension). It only points the tool to existing folders in the repository and contains no executable code, no secrets, and no changes to firmware behavior.…

e554f618by cepetr+8−01 file
No security note in commit
Informational 15 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): rename ResolvedBuild to ResolvedBuildFeatures

This commit is a simple internal code cleanup: it renames a Rust data structure called ResolvedBuild to ResolvedBuildFeatures and updates the places that use it. There are no functional changes, no security fixes, and no changes to how the…

aaa2780bby cepetr+5−52 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

fix(core): fix up clippy warnings in storage.rs

This is a tiny code cleanup commit that replaces two calls to `ptr::null()` with Rust's `unwrap_or_default()` to silence automated Clippy lint warnings. It does not change what value is produced (still a null pointer when no salt is provid…

62354461by cepetr+2−21 file
No security note in commit
Informational 15 AI analysisMessage 47 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): simplify dependencies

This commit is a straightforward internal code cleanup in Trezor's build tooling. It moves a data structure and two helper methods from one Rust source file to another and updates callers accordingly. There is no change to user-facing beha…

6268349fby cepetr+9−193 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.

Security candidatechore(core): remove Japan from regulatory screenby obrusvit · e7e3a4b8 · Oct 7, 2025 · 3 filesMessage 72 · AdequateInformational 15Details
Commit message · obrusvit

chore(core): remove Japan from regulatory screen

- temporarily remove the regulatory screen for Japan until the
certification process is completed

[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
boot or update path
AI analysis · Informational 15/100

This commit removes Japan from a device menu screen that displays regulatory certification information. It is a routine product compliance change, not a security fix. There are no code changes that affect cryptography, authentication, or how the device protects funds or data.

Security candidatechore: remove unusued importby obrusvit · 5ca87c69 · Oct 7, 2025 · 1 fileMessage 47 · ThinInformational 15Details
Commit message · obrusvit

chore: remove unusued import

[no changelog]

47/100 · ThinMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
boot or update path
AI analysis · Informational 15/100

This is a trivial code cleanup: a single unused import was removed from a Rust source file. It has no functional or security effect on the Trezor firmware.

Security candidatefeat(core/eckhart): pin and wipe code device menu testsby Lukas Bielesch · 2cb2f9dc · Oct 7, 2025 · 12 filesMessage 72 · AdequateInformational 15Details
Commit message · Lukas Bielesch

feat(core/eckhart): pin and wipe code device menu tests

[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
Why it was queued
access controlsigning or wallet pathauthentication path
AI analysis · Informational 15/100

This commit adds and updates automated user-interface tests for the PIN and wipe-code menus on the Trezor Safe 7 (Eckhart) hardware wallet. It also renames some Spanish translation strings for consistency. There is no change to the firmware's security logic, cryptography, or device behavior—only test code and translations are modified.

Lower-priorityfix(core): reset settings on wallet wipeby tychovrahe · 133d138a · Oct 7, 2025 · 1 fileMessage 57 · ThinLow 33Details
Commit message · tychovrahe

fix(core): reset settings on wallet wipe

[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 33/100

This commit fixes a bug where wiping a Trezor wallet did not reset all device settings. Previously, settings like haptic feedback, RGB LED, Bluetooth, and experimental features could remain enabled or disabled based on the old wallet's preferences instead of being reloaded from storage after a wipe. The fix makes sure these settings are reloaded when the wallet is reset, so the device behaves consistently with a fresh state.

Security candidateFix: Center align shown pin for Eckhart (#5946)by Suyash Shandilya · 6d1ff1b3 · Oct 6, 2025 · 2 filesMessage 88 · StrongInformational 15Details
Commit message · Suyash Shandilya

Fix: Center align shown pin for Eckhart (#5946)

* fix(eckhart): shown pin should be center aligned

Added a new `space_out_str` function in strutils which adds spaces
between an input string for a clean look. Removed use of Chunks for
spacing visible PIN string.
[no changelog]

* chore(ui): update fixtures

88/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
Why it was queued
access controlboot or update pathauthentication path
AI analysis · Informational 15/100

This commit is a visual polish fix for the Trezor hardware wallet's newer 'Eckhart' screen layout. It changes how the entered PIN is displayed so the digits are centered and spaced out nicely. There is no security-relevant change here—only the on-screen appearance is adjusted, and the corresponding test snapshots are updated.

Lower-priorityrefactor(storage): rename `stretch_pin()` to `mcu_pin_stretch()`by Ondřej Vejpustek · 10ec8308 · Oct 6, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Ondřej Vejpustek

refactor(storage): rename `stretch_pin()` to `mcu_pin_stretch()`

[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 is a simple rename of an internal function from stretch_pin() to mcu_pin_stretch(). No behavior, logic, or security properties of the code changed. It is a routine code cleanup to make the function's purpose clearer.

Lower-priorityrefactor(core/embed): introduce `optiga_reset_counter()`by Ondřej Vejpustek · a84f8c2c · Oct 6, 2025 · 3 filesMessage 62 · AdequateInformational 12Details
Commit message · Ondřej Vejpustek

refactor(core/embed): introduce `optiga_reset_counter()`

[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 12/100

This commit is a straightforward code cleanup: it replaces several places that manually built an 8-byte zero-padded counter value with a single helper function called optiga_reset_counter(). The behavior is unchanged, and there is no indication of a security bug being fixed.

Lower-priorityrefactor(core/embed): remember optiga auto statesby Ondřej Vejpustek · eb21ec78 · Oct 6, 2025 · 3 filesMessage 57 · ThinLow 35Details
Commit message · Ondřej Vejpustek

refactor(core/embed): remember optiga auto states

[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 35/100

This change refactors how the Trezor hardware wallet tracks and clears temporary 'auto-state' sessions with the Optiga secure chip. Previously, the code manually cleared specific auto-states one by one. Now it keeps a list of all active auto-states and clears them all at once. The main security-relevant improvement is that the list is updated before sending the command to the chip, so if a communication error happens while receiving the response, the state is still recorded and can be cleaned up later. This reduces the risk of leaking or exhausting limited secure-chip session slots.

Lower-prioritychore: update fixtures [no changelog]by M1nd3r · 4341e094 · Oct 6, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · M1nd3r

chore: update fixtures
[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 15/100

This commit only updates SHA-256 hashes in a test fixtures file (tests/ui_tests/fixtures.json). These hashes are used by automated UI tests to verify that screen outputs match expected snapshots. No firmware code, cryptographic logic, or user-facing behavior was changed. There is no security relevance in the diff itself.

Lower-priorityfeat(storage,core/embed): support multiple stretched pins in optigaby Ondřej Vejpustek · ba5c9158 · Oct 6, 2025 · 5 filesMessage 62 · AdequateLow 35Details
Commit message · Ondřej Vejpustek

feat(storage,core/embed): support multiple stretched pins in optiga

[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 · Low 35/100

This commit rewrites how the Trezor hardware wallet stores and verifies PINs when the Optiga secure chip is present. It replaces a single 'stretched PIN' slot with a chain of up to 10 slots (currently configured to 1, so behavior is unchanged for users). The goal is to make future brute-force and side-channel attacks against the PIN harder by adding more authorization steps and by separating the HMAC counter reset key from the main unlock path. Because the new code is complex and touches the PIN verification flow, any bug here could affect device security, but the commit itself is a defensive hardening feature rather than a fix for a known vulnerability.

Security candidatebuild(legacy): do not use tropicby Ondřej Vejpustek · 7a19da9d · Oct 6, 2025 · 1 fileMessage 57 · ThinInformational 16Details
Commit message · Ondřej Vejpustek

build(legacy): do not use tropic

[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 path
AI analysis · Informational 16/100

This commit adds a build flag that disables a feature called 'tropic' in the legacy Trezor firmware build. On its own, the change is a straightforward build configuration tweak with no visible security bug fix. It may be a defensive hardening step or simply removing an unused feature, but the commit message gives no security context.

Security candidatefeat(storage,core/embed): use tropic for pin protectionby Ondřej Vejpustek · 67dcf308 · Oct 6, 2025 · 5 filesMessage 62 · AdequateLow 35Details
Commit message · Ondřej Vejpustek

feat(storage,core/embed): use tropic for pin protection

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
access control
AI analysis · Low 35/100

This commit adds support for using a second hardware security chip called Tropic to protect the device PIN, alongside the existing Optiga chip. It is a feature addition that strengthens PIN protection by adding another physical barrier against brute-force and chip-level attacks. There is no direct evidence in the commit that it fixes an existing security flaw; it appears to be a defensive hardening change.

Security candidatefeat(core/embed): refactor tropic secure channel initializationby Ondřej Vejpustek · 6cc6a877 · Oct 6, 2025 · 2 filesMessage 62 · AdequateLow 31Details
Commit message · Ondřej Vejpustek

feat(core/embed): refactor tropic secure channel initialization

[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
seed or entropy path
AI analysis · Low 31/100

This commit refactors how a Trezor hardware wallet establishes a secure communication channel with an optional Tropic cryptographic chip. It removes a 'PRODUCTION-only hotfix' that had been mixing Tropic-sourced randomness into the device's random number generator. It also restructures the pairing-key selection logic so the code tries privileged, then unprivileged, then factory keys in a clearer sequence. The change is described as a feature/refactor with no changelog, not as a security fix. There is no direct evidence in the diff of an exploitable vulnerability, but removing a hotfix and changing secure-channel initialization are security-adjacent changes that warrant review.

Security candidatefix(storage): use `rng_fill_buffer_strong()`by Ondřej Vejpustek · c3e3d85b · Oct 6, 2025 · 3 filesMessage 47 · ThinModerate 57Details
Commit message · Ondřej Vejpustek

fix(storage): use `rng_fill_buffer_strong()`

[no changelog]

47/100 · ThinMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
seed or entropy path
AI analysis · Moderate 57/100

This commit changes how the Trezor hardware wallet generates random numbers used to protect PINs, wipe codes, and encrypted storage. It replaces direct calls to a basic random function with a stronger random-number wrapper for sensitive operations, and adds a weaker wrapper for less critical uses. The change suggests the developers want to enforce a stricter random source for security-critical material, but the commit itself does not explain whether the old function was actually weak or exploitable.

Security candidatefeat(core/embed): make pin verification faster on T3W1by Ondřej Vejpustek · 915d14d3 · Oct 6, 2025 · 2 filesMessage 72 · AdequateLow 44Details
Commit message · Ondřej Vejpustek

feat(core/embed): make pin verification faster on T3W1

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
Why it was queued
access controldefensive validation
AI analysis · Low 44/100

This commit changes how many times a device 'stretches' (repeats and strengthens) the user's PIN before checking it. On T3W1 devices, when a build-time flag called STRETCHED_PIN_COUNT is greater than 1, the code now reduces both Optiga hardware stretching and software PBKDF2 stretching to a single iteration. The stated goal is to make PIN verification faster. Reducing stretching weakens the protection against brute-force attacks if an attacker obtains the stored secrets, but the change is conditional on a build flag and does not by itself create a direct remote exploit. It is a security-relevant hardening reduction rather than an obvious vulnerability.

Lower-priorityfix(core): fix syshandle_write_blocking behaviorby cepetr · f6357091 · Oct 6, 2025 · 1 fileMessage 57 · ThinLow 32Details
Commit message · cepetr

fix(core): fix syshandle_write_blocking behavior

[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 32/100

This commit fixes a bug in a low-level Trezor firmware function that writes data from the device. Previously, when a timeout was requested, the code only waited for the channel to become ready but then tried to write all data at once, ignoring that the hardware might only accept a portion. The fix loops, writing in chunks and waiting again until all data is sent or the timeout expires. This is a reliability/robustness fix that could prevent incomplete or failed writes in communication with the host.

Lower-priorityfix(prodtest): resolve unreliable USB VCP transmissionby cepetr · cd23a4db · Oct 6, 2025 · 2 filesMessage 62 · AdequateInformational 16Details
Commit message · cepetr

fix(prodtest): resolve unreliable USB VCP transmission

[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 16/100

This commit fixes a minor reliability issue in Trezor's production-test firmware (a factory diagnostic tool, not the main wallet firmware). The change adjusts how long the device waits when sending text over a USB virtual serial port, so it doesn't get stuck if no computer is listening. There is no direct evidence this is a security vulnerability or that it affects end-user devices.

Lower-priorityfix(core): fix usb vcp ring bufferby cepetr · edae5ae8 · Oct 6, 2025 · 5 filesMessage 57 · ThinLow 48Details
Commit message · cepetr

fix(core): fix usb vcp ring buffer

[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 48/100

This commit rewrites the USB virtual serial port (VCP) ring buffer code in Trezor's firmware. The old code tracked buffer space using simple read/write counters, which could become inconsistent when the USB data-out interrupt and the application read data at the same time. The new code uses a dedicated ring-buffer module that protects its state with interrupt locks and tracks how many bytes are actually used. It also stops requesting new USB packets when the buffer is nearly full, preventing overflow and lost data. The change is a hardening/bug-fix patch; the commit message does not call it a security fix, but race conditions in USB buffering can affect device reliability and, in the worst case, be abused to cause information leakage or denial of service.

Security candidatefeat(prodtest): ping command with an argumentby cepetr · 3df55083 · Oct 6, 2025 · 3 filesMessage 57 · ThinInformational 16Details
Commit message · cepetr

feat(prodtest): ping command with an argument

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
authentication path
AI analysis · Informational 16/100

This commit adds an optional text argument to the Trezor production-test firmware's 'ping' command. Previously, 'ping' always replied with just 'OK'. Now, if you type 'ping ABC', the device replies 'OK ABC'. It is a small feature change in a diagnostic/testing tool, not a security fix or vulnerability.

Security candidatefix(core): clear PIN keyboard in bootscreenby obrusvit · 4927c63d · Oct 6, 2025 · 2 filesMessage 57 · ThinLow 47Details
Commit message · obrusvit

fix(core): clear PIN keyboard in bootscreen

[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
access control
AI analysis · Low 47/100

This commit changes how the Trezor hardware wallet displays and clears its PIN-entry keyboard during the boot screen. The title says it 'clears PIN keyboard in bootscreen,' suggesting the previous code may have left PIN-keyboard graphics or state visible when the device went to sleep or resumed. The patch adds a notification channel so the boot screen can be told when a suspend is happening, and it re-creates the lockscreen after suspend instead of continuing straight to PIN entry. There is no direct evidence in the diff of a security vulnerability such as PIN leakage, but a leftover PIN keyboard on screen could theoretically expose partial PIN-entry state or confuse the user interface.

Security candidatefix(core): clear PIN keyboard in FW main loopby obrusvit · b8ed0284 · Oct 6, 2025 · 1 fileMessage 57 · ThinLow 49Details
Commit message · obrusvit

fix(core): clear PIN keyboard in FW main loop

[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
access control
AI analysis · Low 49/100

This patch fixes a UI state issue in Trezor hardware wallets. If the device auto-locks while the user is in the middle of typing their PIN, the on-screen PIN keyboard could remain visible even though the device is now locked. The fix detects this situation and clears/restarts the workflow so the keyboard doesn't linger. It is primarily a user-interface consistency bug, but leaving an active PIN entry screen on a locked device could confuse users or create a minor information-leak signal (that a PIN was being entered).

Lower-prioritychore: remove unused variableby obrusvit · 8bc0ae97 · Oct 6, 2025 · 1 fileMessage 47 · ThinInformational 15Details
Commit message · obrusvit

chore: remove unused variable

[no changelog]

47/100 · ThinMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 15/100

This commit simply deletes an unused internal variable named _SCREENSAVER_IS_ON. There is no functional code change, no bug fix, and no security relevance.

Security candidatechore(core/eckhart): update pagination style for action barby Lukas Bielesch · 7785df84 · Oct 6, 2025 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · Lukas Bielesch

chore(core/eckhart): update pagination style for action bar

- pagination buttons have equal size

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
boot or update path
AI analysis · Informational 15/100

This is a user-interface styling change for the Trezor hardware wallet's action bar. It adjusts how pagination buttons are sized and positioned on the new 'Eckhart' device layout, and updates the corresponding visual test snapshots. There is no indication this affects security, cryptography, or how transactions are approved.

Lower-priorityfix(core): add missing `if utils.USE_POWER_MANAGER`by Roman Zeyde · d066b3c1 · Oct 6, 2025 · 1 fileMessage 62 · AdequateLow 28Details
Commit message · Roman Zeyde

fix(core): add missing `if utils.USE_POWER_MANAGER`

[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 · Low 28/100

This commit fixes a small coding mistake in the Trezor hardware wallet's startup code. The device was always trying to reset a power-management timer, even on models that don't have that feature. The fix adds a check so the reset only happens on models that actually support it. This is a defensive bug fix; the diff alone does not show a clear security vulnerability, but it could prevent a crash or unexpected behavior during boot on some device models.

Security candidatefeat(core/eckhart): bluetooth switching in device menuby Lukas Bielesch · 182fe7e6 · Oct 6, 2025 · 22 filesMessage 62 · AdequateInformational 20Details
Commit message · Lukas Bielesch

feat(core/eckhart): bluetooth switching in device menu

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
cryptography-sensitive pathsigning or wallet pathboot or update path
AI analysis · Informational 20/100

This commit adds a user-facing Bluetooth on/off toggle in the device menu for the Trezor Safe 7 (T3W1). It stores the user's preference and only starts Bluetooth at boot if it was previously enabled. There is no direct evidence of a security vulnerability in the change, but it touches Bluetooth state management and storage, which are security-relevant areas.