TZ
← All projectsTrezor

Trezor firmware

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

BitcoinHardware walletsNormal
Repository coverage

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

916security candidates311second-pass queue1385AI analyses
336commits · 30 days
588commits · 60 days
1363commits · 180 days
2648commits · 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
473Strong · 80–100
1321Adequate · 60–79
862Thin · 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.
PrisionMike945982273
Andrew Kozlik712437265
Roman Zeyde572177260171
tychovrahe292104133161
Jakub Janků22917182
Martin Pastyřík23811173
cepetr19682105059
Ioan Bizău23076140059
obrusvit21376111064
M1nd3r20671106071
Lukas Bielesch846771067
Martin Milata1744684063
Analysis record

Published AI watches

Last scanned 1 hour ago

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

docs: add xtask documentation

This commit only adds documentation. It updates the table of contents and adds a new markdown page plus a diagram explaining the project's 'xtask' build helper. No code, build scripts, or firmware logic was changed, so it cannot introduce …

4879dababy cepetr+296−03 files
No security note in commit
Informational 17 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

feat(xtask): introduce project-toml build-options

This commit is a build-system refactoring for Trezor firmware. It moves the mapping of command-line build options to Rust/cargo features out of hard-coded Rust logic and into per-project TOML files. It also adds a small change so debug bui…

Build-option mapping now lives in project.toml files, increasing the attack surface for supply-chain/build-configuration tamperingMakefile change automatically enables debug-link for PYOPT=0 debug buildsValidation added to reject storage_insecure_testing_mode in production builds
d44de007by cepetr+681−40817 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

ci(core): fix nightly ARM emulator filename

This commit fixes a filename-renaming step in the project's automated nightly build pipeline. It ensures ARM64 emulator files get the correct name expected by another testing tool. There is no change to the actual Trezor firmware code, dev…

ab95f08fby Martin Milata+3−01 file
No security note in commit
Informational 12 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

chore(vendor): avoid micropython submodule noise in git status

This is a routine housekeeping change to reduce clutter in Git's status output for a third-party component (the MicroPython submodule). There is no indication it changes any code that runs on the Trezor device or affects security.

06cab32bby Martin Milata+1−11 file
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 candidatefix(core/eckhart): unify bootscreen and lockscreen homebar textby Roman Zeyde · 3a9dc82f · Dec 19, 2025 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

fix(core/eckhart): unify bootscreen and lockscreen homebar text

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 commit changes the text shown on the bottom home bar of a Trezor hardware wallet screen. The boot screen now says 'Unlock' instead of 'Tap to start', matching the lock screen wording. It is a cosmetic/user-interface consistency fix with no security relevance.

Security candidatebuild(core): exclude `embed/rtl/printf.c` from non-prodtest buildsby Roman Zeyde · 6a637bd4 · Dec 18, 2025 · 8 filesMessage 100 · StrongInformational 15Details
Commit message · Roman Zeyde

build(core): exclude `embed/rtl/printf.c` from non-prodtest builds

Otherwise, nightly CI fails with:
https://github.com/trezor/trezor-firmware/actions/runs/20320385493/job/58431441939

We currently use `printf()` only in prodtest / non-production builds[^1].

[no changelog]

[^1]: https://github.com/trezor/trezor-firmware/pull/6208#discussion_r2610006444

100/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✓ Links an issue, advisory, or supporting reference
Why it was queued
boot or update path
AI analysis · Informational 15/100

This commit changes build scripts so that a file providing printf-style debugging output is only included in non-production builds. It is a build-system cleanup to fix a nightly CI failure, not a security fix. There is no indication it addresses a vulnerability or changes runtime behavior of shipped devices.

Security candidatechore: update secmon for T3W1by Ioan Bizău · 6647b72c · Dec 16, 2025 · 2 filesMessage 80 · StrongInformational 3Details
Commit message · Ioan Bizău

chore: update secmon for T3W1

* both production secmon (signed) and dev secmon (`BOOTLOADER_DEVEL=1`)
* built from `secmon/v1.0.5`

```
$ sha256sum core/embed/models/T3W1/secmon/secmon*.bin
230d709677831166fd3099f3633ea7670195a7cfbde64381865fb3b5866ff2f7 core/embed/models/T3W1/secmon/secmon.bin
c237cec22ff4560447a48b23cacd4e54e33060fbfeab0e2920c3459fe4e91701 core/embed/models/T3W1/secmon/secmon_DEV.bin
```

[no changelog]

(cherry picked from commit d3f5db4a486611dbe95f3c9c1faf4c9b3374b249)

80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
update trust
AI analysis · Informational 3/100

This commit replaces two binary firmware files for the Trezor T3W1 hardware wallet's security monitor (secmon) with newer versions built from secmon v1.0.5. The actual code changes are not visible because the files are binary blobs. There is no claim in the commit message that this fixes a security vulnerability, and no independent references were provided. On its own, this looks like a routine component update.

Security candidatefix(core): remove rsod screens from rtl error handlingby cepetr · 80c5444a · Dec 16, 2025 · 22 filesMessage 62 · AdequateInformational 20Details
Commit message · cepetr

fix(core): remove rsod screens from rtl error handling

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

This commit is a code cleanup in the Trezor firmware. It moves special error-screen functions (for wipe code, too many PIN attempts, install restricted, and device wiped) out of the runtime library (RTL) error-handling module and into a new dedicated 'rsod_special' module. It also removes the 'ensure' macro from a public header. The change is architectural refactoring; it does not appear to fix a security bug or introduce a new vulnerability. The commit message and diff give no indication this is a security patch.

Security candidatefeat(core): introduce centralized loggingby cepetr · f804a1fb · Dec 12, 2025 · 25 filesMessage 57 · ThinInformational 18Details
Commit message · cepetr

feat(core): introduce centralized logging

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

This commit adds a new centralized logging system to the Trezor firmware. It is a feature/refactoring change: debug output is routed through a new 'syslog' layer with log levels and filters, and the debug console is now gated behind a compile-time flag. There is no direct evidence in the commit that this fixes a security vulnerability; it appears to be a developer-facing infrastructure improvement.

Security candidatefeat(core): add trezorctl set-log-filter commandby cepetr · 0974bcbc · Dec 12, 2025 · 16 filesMessage 57 · ThinInformational 24Details
Commit message · cepetr

feat(core): add trezorctl set-log-filter command

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

This commit adds a new developer-only command, `trezorctl set-log-filter`, that lets someone with physical debug access to a Trezor device change which internal log messages are printed. It only works when the device was built with a special debug-console feature enabled, and it does not change how ordinary user operations work. There is no direct evidence in the commit that this creates a security vulnerability, but any new debug command slightly increases the attack surface available to someone who already has debug access.

Security candidaterefactor(core): do not use mini_snprintf in board/bootloaderby cepetr · b969d71c · Dec 12, 2025 · 21 filesMessage 62 · AdequateLow 34Details
Commit message · cepetr

refactor(core): do not use mini_snprintf in board/bootloader

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

This commit is a code cleanup that replaces the use of a small printf-style formatter (mini_snprintf) with simpler, purpose-built string helpers in the Trezor bootloader and low-level system code. It does not add or remove security features, and there is no direct evidence it fixes a specific vulnerability. The change may reduce the attack surface by removing format-string parsing from some paths, but it also introduces new helper functions whose correctness now matters for safety.

Security candidatefix(core): save some flash in board/bootloaderby cepetr · 54688c1c · Dec 12, 2025 · 1 fileMessage 57 · ThinInformational 11Details
Commit message · cepetr

fix(core): save some flash in board/bootloader

[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
update trust
AI analysis · Informational 11/100

This is a one-line build configuration change in low-level firmware code. It swaps the condition used to decide whether to compile a special secure-mode memory check. The change appears aimed at reducing flash memory usage in board/bootloader builds, not at fixing a security vulnerability. There is no direct evidence in the commit of a security issue.

Security candidatefeat(core): add extended formatting capabilities to snprintfby cepetr · b1af5166 · Dec 12, 2025 · 15 filesMessage 62 · AdequateLow 39Details
Commit message · cepetr

feat(core): add extended formatting capabilities to snprintf

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

This commit replaces Trezor's small, home-grown printf-style string formatter with a more capable third-party library (eyalroz/printf). The change is described as adding 'extended formatting capabilities' and is marked with '[no changelog]'. The new library supports more format specifiers, including the %n specifier, which can write back into caller-provided memory. Trezor's configuration enables %n support. Because this is a low-level runtime component used across bootloader, firmware, kernel, secure monitor, and production-test builds, any bug in the new formatter could have broad consequences. However, the commit itself does not fix a known vulnerability, and there is no vendor statement that this change is security-relevant.

Security candidatefix(core): reduce unnecessary section alignmentby cepetr · 5dfbfaa0 · Dec 12, 2025 · 2 filesMessage 57 · ThinInformational 11Details
Commit message · cepetr

fix(core): reduce unnecessary section alignment

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

This commit changes two bootloader linker scripts for Trezor hardware wallets, reducing the alignment padding between code/data sections from 512 bytes down to 4 bytes. The stated purpose is to remove unnecessary padding and save flash space. There is no indication in the commit or supplied references that this fixes a security vulnerability.

Security candidaterefactor(core): use LOG macros instead of printfsby cepetr · a1fa1fbb · Dec 12, 2025 · 16 filesMessage 57 · ThinInformational 15Details
Commit message · cepetr

refactor(core): use LOG macros instead of printfs

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

This commit is a straightforward code cleanup: it replaces old-style printf debug messages with a new centralized logging system (LOG_ macros). It also removes some unused non-production debug hooks and switches header include guards to '#pragma once'. There is no change to security logic, cryptography, user-facing behavior, or access controls.

Security candidatefeat(core): implementation of nRF FW version readout via SMP commands for realiability increase e.g. for cases when the nRF FW application is broken from whatever reason and can't reply to Trezor's FW version check for possible nRF FW update. The alternative way of FW version readout uses the MCUbooby bleska · 00bc9339 · Dec 11, 2025 · 5 filesMessage 95 · StrongLow 35Details
Commit message · bleska

feat(core): implementation of nRF FW version readout via SMP commands for realiability increase e.g. for cases when the nRF FW application is broken from whatever reason and can't reply to Trezor's FW version check for possible nRF FW update.
The alternative way of FW version readout uses the MCUboot SMP serial recovery feature which had already been enabled and used for nRF FW update in the field mainly. The version is returned as a structure of 4 uint fields representing major, minor, revision and build_num respectively (string received from nRF IC is decoded).
The development releases of nRF FW are to be built with: "./scripts/build_sign_flash.sh -b t3w1_revA_nrf52832/nrf52832 -c -d -s -f" command. '-d' is essential, with '-p' the build script generates and signs the image in a way that MCUboot can't verify that (it uses debug keys see hash_signer.py but the MCUboot expects the production key which has embedded inside "image_validate.c" file).

[no changelog]

95/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
defensive validationboot or update path
AI analysis · Low 35/100

This commit adds a backup way for the Trezor device to check the version of the wireless nRF chip's firmware. If the normal app-to-app communication fails, the device can now ask the nRF chip's bootloader directly over a different channel. This is meant to make firmware update decisions more reliable, not to fix a known security bug. The change does not appear to introduce an obvious vulnerability, but it does add parsing of data received from the nRF chip and changes how the device decides whether to force an update.

Security candidatefix(core): avoid constructing an empty Pagerby Roman Zeyde · 9ab7f8fb · Dec 11, 2025 · 1 fileMessage 72 · AdequateInformational 21Details
Commit message · Roman Zeyde

fix(core): avoid constructing an empty Pager

`Pager::new()` will panic when `debug_assert`s are enabled.

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

This is a tiny one-line fix in the user-interface code of the Trezor hardware wallet. It changes a placeholder value from 0 to 1 so that a UI scrollbar helper object is never created with 'zero pages,' which would trigger an internal sanity check (a debug assertion) and crash the device in debug/test builds. In release builds the assertion is normally removed, so the practical security impact is low. There is no direct evidence this is exploitable by an attacker.

Security candidatefix(core): avoid integer underflow in Caesar loaderby Roman Zeyde · f7cbe439 · Dec 11, 2025 · 1 fileMessage 89 · StrongInformational 18Details
Commit message · Roman Zeyde

fix(core): avoid integer underflow in Caesar loader

Otherwise, it will panic when `debug_assert`s are enabled.

[no changelog]

89/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Explains rationale or failure mode
Why it was queued
cryptography-sensitive path
AI analysis · Informational 18/100

This commit fixes a minor arithmetic bug in the star-shaped loading animation on Trezor hardware wallets. When the previous code calculated the position just before the currently selected star, it could subtract 1 from 0, causing an integer underflow. In Rust, this only triggers a panic when special debug checks are enabled, so normal user devices would not crash. The fix avoids the underflow by adding the total star count before subtracting one, then wrapping with modulo. It is a UI-only change with no direct security impact on funds or device secrets.

Security candidatefix(legacy): make sure EVM symbol is visibleby Roman Zeyde · 460dc70c · Dec 11, 2025 · 3 filesMessage 72 · AdequateLow 39Details
Commit message · Roman Zeyde

fix(legacy): make sure EVM symbol is visible

Separate amount value and unit, so they are displayed on separate lines.

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

This commit fixes how the Trezor One (legacy) hardware wallet displays Ethereum and EVM token amounts on its small screen. Previously, the numeric value and the currency symbol (like 'ETH' or a token ticker) were combined into one long string. On the device's narrow display this could cause the symbol to be cut off or hidden, making it hard for the user to know which asset they are approving. The change splits the value and the unit onto separate lines so the symbol is always visible. This is a user-interface safety improvement rather than a cryptographic bug, but it can help prevent users from accidentally approving misleading transactions.

Security candidatefix(legacy): use symbol name instead of EVM network shortcutby Roman Zeyde · c0749eef · Dec 11, 2025 · 2 filesMessage 62 · AdequateInformational 16Details
Commit message · Roman Zeyde

fix(legacy): use symbol name instead of EVM network shortcut

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

This commit fixes a data-generation template for Trezor's older (legacy) firmware so that Ethereum network entries use the proper currency symbol and network name fields instead of incorrectly copying the network shortcut into the symbol field and leaving the name blank. It appears to be a UI/labeling correctness fix rather than a vulnerability, but the change could affect what users see when confirming Ethereum transactions on the device.

Security candidatefeat(rust/trezor-thp): transport layerby Martin Milata · 912bea98 · Dec 9, 2025 · 11 filesMessage 57 · ThinInformational 12Details
Commit message · Martin Milata

feat(rust/trezor-thp): transport layer

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

This commit adds a brand-new Rust implementation of the Trezor Host Protocol (THP), a low-level transport layer that handles packet headers, fragmentation/reassembly, checksums, sequence/ack bits, and basic message types such as ping/pong, acknowledgments, and handshakes. It is a feature commit with no changelog entry and no accompanying security disclosure. The code is purely a new library plus tests/example; it does not patch any reported vulnerability.

Security candidaterefactor(legacy): XPUB buffer lengthby Martin Milata · 2a010f8e · Dec 9, 2025 · 2 filesMessage 57 · ThinLow 27Details
Commit message · Martin Milata

refactor(legacy): XPUB buffer length

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

This commit slightly enlarges two internal memory buffers used to hold extended public keys (XPUBs) in the Trezor legacy firmware. The change gives each buffer one extra byte, likely to ensure a null terminator can always fit after a maximum-length serialized key. It is a defensive hardening change rather than a fix for a confirmed exploitable bug, and the commit message calls it a refactor with no changelog.

Security candidatefeat(legacy): send BIP-380 descriptor in GetPublicKey responseby Martin Milata · 529607e8 · Dec 9, 2025 · 7 filesMessage 62 · AdequateInformational 19Details
Commit message · Martin Milata

feat(legacy): send BIP-380 descriptor in GetPublicKey response

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

This commit adds a new convenience feature to the older Trezor One (legacy) firmware: when you ask the device for a public key, it now also returns a BIP-380 output descriptor string. This is a standardized way to describe how that public key should be used in Bitcoin wallets. The change is a feature addition, not a fix for a known security bug. It does not change how private keys are handled or how transactions are signed.

Security candidaterefactor(core/bootloader): extract PB helper macros into separate fileby tychovrahe · 71089e52 · Dec 9, 2025 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · tychovrahe

refactor(core/bootloader): extract PB helper macros into separate file

[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 change simply moves a set of helper macros from one file to a new shared header file. The macros themselves are unchanged, and no program behavior is modified. It is a routine code cleanup (refactoring) with no security relevance.

Security candidatefeat(core): send build version in featuresby tychovrahe · 68dfb262 · Dec 9, 2025 · 10 filesMessage 57 · ThinInformational 19Details
Commit message · tychovrahe

feat(core): send build version in features

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 19/100

This commit adds a new 'build version' number to the information a Trezor device reports about itself when asked. It is a feature addition that exposes more version detail to host software and users. There is no indication in the commit that it fixes a security bug or introduces a vulnerability.

Security candidaterefactor(core/bootloader): extract fw check from main.cby tychovrahe · 4a126817 · Dec 8, 2025 · 8 filesMessage 62 · AdequateInformational 15Details
Commit message · tychovrahe

refactor(core/bootloader): extract fw check from main.c

[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 is a straightforward code cleanup: it moves the existing firmware-validation logic out of the bootloader's main.c into a new file (fw_check.c/h) and updates the surrounding code to use the new structure. There is no change to what the bootloader actually checks or how strict it is. It is a pure refactor with no security-relevant behavior change visible in the diff.

Security candidatefix(legacy): Improve handling of value overflows.by Andrew Kozlik · ea542943 · Dec 5, 2025 · 8 filesMessage 57 · ThinModerate 59Details
Commit message · Andrew Kozlik

fix(legacy): Improve handling of value overflows.

[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
memory safetycryptography-sensitive pathsigning or wallet pathboot or update path
AI analysis · Moderate 59/100

This commit fixes several numeric overflow bugs in the older Trezor One (legacy) firmware. The most user-visible issue was a fee-rate display overflow: when a transaction used extremely large input amounts, multiplying the fee by 100 to show decimal places could wrap around to a tiny-looking fee, potentially tricking a user into approving a much higher fee than shown. The patch also prevents overflow when summing duplicate NEM mosaic quantities and stops BIP-32 child derivation once the maximum derivation depth is reached. A comment-only change notes that PINs can be up to 50 characters, but no code behavior was altered there.

Security candidatefix(legacy): Improve handling of busy deadline overflow.by Andrew Kozlik · adaa63f9 · Dec 5, 2025 · 4 filesMessage 67 · AdequateLow 37Details
Commit message · Andrew Kozlik

fix(legacy): Improve handling of busy deadline overflow.

[no changelog]

67/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Names security-relevant behavior explicitly! No meaningful explanatory body
Why it was queued
memory safetyboot or update path
AI analysis · Low 37/100

This commit fixes a subtle arithmetic bug in older Trezor hardware wallets related to how a 'busy' screen timeout is calculated. The old code added a delay value to the current time and compared it directly, which could misbehave when the internal millisecond counter wraps around to zero after a long period. The new code stores the start time and duration separately and checks whether the elapsed time is still within that duration, which handles wraparound correctly. This could affect whether the device reports itself as busy, shows the right screen, or clears the busy state at the expected time.

Security candidatefix(legacy): Avoid hypothetical division by zero.by Andrew Kozlik · a11de800 · Dec 5, 2025 · 1 fileMessage 57 · ThinInformational 16Details
Commit message · Andrew Kozlik

fix(legacy): Avoid hypothetical division by zero.

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

This commit fixes a theoretical 'division by zero' bug in the progress-bar code shown on the Trezor hardware wallet during Bitcoin transaction signing. The bug could only crash the device if internal counters used to calculate progress ever became zero, which the developers describe as 'hypothetical.' It does not affect cryptographic signing itself or expose funds.