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 queue2165AI 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.
tychovrahe292104217361
Roman Zeyde572177447271
PrisionMike945991273
Andrew Kozlik712452265
obrusvit21376153164
Jakub Janků22919182
Martin Pastyřík23821173
cepetr19682173059
Ioan Bizău23076203059
M1nd3r20671180071
Lukas Bielesch846771067
Martin Milata17446154063
Analysis record

Published AI watches

Last scanned 32 minutes 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.

Lower-prioritychore(nordic): add full_name to t3w1 board definitionby tychovrahe · a8104ec4 · May 6, 2026 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · tychovrahe

chore(nordic): add full_name to t3w1 board definition

[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 is a trivial board definition update for a Nordic chip configuration file. It adds a duplicate 'full_name' field that matches the existing 'name' field. There is no security relevance.

Lower-prioritychore(nordic): configuration cleanupby tychovrahe · b96e38c9 · May 6, 2026 · 11 filesMessage 47 · ThinInformational 11Details
Commit message · tychovrahe

chore(nordic): configuration cleanup

[no changelog]

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

This commit tidies up build configuration files for Trezor's Nordic Bluetooth/radio test firmware. It removes some redundant or outdated settings, adds a couple of clarifying options, and disables over-the-air firmware update slots for test builds. There is no direct evidence in the commit that this fixes a security vulnerability; it reads as a routine cleanup of build settings.

AI review queuedrefactor(nordic): make build_sign_flash.sh script take constants from DTS and config filesby tychovrahe · fa2dc6b7 · May 6, 2026 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · tychovrahe

refactor(nordic): make build_sign_flash.sh script take constants from DTS and config files

[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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit refactors a build script for Nordic hardware so that several hard-coded numbers (memory addresses, slot size, device model tag) are instead read from the project's generated device tree and configuration files. There is no change to runtime code, no new user-facing behavior, and no indication of a security fix or vulnerability.

AI review queuedrefactor(nordic): migrate away from pm_static towards DTS fixed-partitionsby tychovrahe · 7e3eb453 · May 6, 2026 · 8 filesMessage 62 · AdequateInformational 17Details
Commit message · tychovrahe

refactor(nordic): migrate away from pm_static towards DTS fixed-partitions

[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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 17/100

This is a build-system refactor for Trezor's Nordic Bluetooth chip firmware. It removes manually maintained flash partition files and switches to Zephyr's standard device-tree partition definitions. The change also fixes a build-time signing script bug where a duplicate image header could be inserted, which would cause the device to crash immediately on boot. There is no direct evidence this is a security vulnerability patch, but it prevents a real boot failure in the manufacturing/development workflow.

AI review queuedrefactor(nordic): dts cleanup and DK gpio driver removalby tychovrahe · a9470c60 · May 6, 2026 · 8 filesMessage 62 · AdequateInformational 15Details
Commit message · tychovrahe

refactor(nordic): dts cleanup and DK gpio driver removal

[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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit is a routine cleanup of the Nordic board configuration and Bluetooth Low Energy firmware for a Trezor hardware wallet. It removes the generic Nordic 'DK buttons and LEDs' driver and replaces it with direct GPIO (general-purpose input/output) definitions tied to the actual Trezor board pins. It also fixes a typo in the board name and renames some device-tree labels for clarity. There is no security-relevant change visible in the diff.

Security candidatefix(core/build): fix headertool keys selectionby tychovrahe · b2de55aa · May 6, 2026 · 2 filesMessage 57 · ThinLow 40Details
Commit message · tychovrahe

fix(core/build): fix headertool keys selection

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

This commit fixes a logic error in the build scripts that decide whether to use development or production signing keys when preparing Trezor firmware images. Before the fix, the condition was accidentally inverted, so production builds may have used development keys and development builds may have used production keys. Using the wrong keys could make firmware signatures invalid or cause devices to reject genuine firmware, but the commit itself does not show an exploitable runtime vulnerability in shipped code.

Lower-prioritychore(ci): GH action update [no changelog]by Michal Kazda · 0ff0c529 · May 6, 2026 · 4 filesMessage 72 · AdequateInformational 15Details
Commit message · Michal Kazda

chore(ci): GH action update
[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
documentation-only discount
AI analysis · Informational 15/100

This commit updates four GitHub Actions workflow files to use a newer version of an official GitHub action that creates authentication tokens for automated bots. It also renames a configuration setting from 'app-id' to 'client-id' to match the new action version's requirements. There is no change to the Trezor firmware code that runs on user devices, and no security vulnerability is present in the diff.

Lower-priorityrefactor(rust/trezor-thp): separate send/receive buffers for handshakeby Martin Milata · 5e18c9d6 · May 5, 2026 · 4 filesMessage 62 · AdequateInformational 12Details
Commit message · Martin Milata

refactor(rust/trezor-thp): separate send/receive buffers for handshake

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 code cleanup in the Trezor firmware's secure USB/Bluetooth transport layer (Trezor THP). It replaces a single shared buffer used during the cryptographic handshake with two separate buffers: one for messages being sent and one for messages being received. The change makes the code easier to reason about and removes some awkward buffer resizing and zeroing patterns, but it does not appear to fix any active security bug. There is no mention of a vulnerability, CVE, or security issue in the commit itself.

Lower-priorityfeat(rust/trezor-thp): ACK piggybackingby Martin Milata · cbc91a60 · May 5, 2026 · 10 filesMessage 57 · ThinLow 29Details
Commit message · Martin Milata

feat(rust/trezor-thp): ACK piggybacking

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

This commit adds a new optional transport-layer feature called 'ACK piggybacking' to the Trezor hardware wallet's Rust THP (Trezor Host Protocol) implementation. It lets a device or host include an acknowledgment (ACK) inside the next data packet instead of sending a separate tiny ACK packet, reducing USB/network traffic. The change is feature-gated by protocol version 2.1 and falls back to the older separate-ACK behavior if either side does not advertise support. The diff also includes related test and example-code cleanups, such as stricter error handling and a new API to set the peer's protocol version before the handshake begins.

Lower-priorityrefactor(rust/trezor-thp): enable full-duplex communicationby Martin Milata · 35ec92ae · May 5, 2026 · 4 filesMessage 62 · AdequateLow 27Details
Commit message · Martin Milata

refactor(rust/trezor-thp): enable full-duplex communication

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

This commit refactors the internal state machine of Trezor's THP (Trezor Host Protocol) Rust code so that sending and receiving are tracked independently, enabling full-duplex communication in the future. It does not appear to fix a known security bug; it is a structural code change. There is no vendor statement or external report linking it to a vulnerability.

Lower-priorityrefactor(rust/trezor-thp): expose length constantsby Martin Milata · 955ec11b · May 5, 2026 · 5 filesMessage 62 · AdequateInformational 12Details
Commit message · Martin Milata

refactor(rust/trezor-thp): expose length constants

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 code cleanup: it moves several fixed-size buffer length constants out of individual Rust source files and into a shared module so they can be reused consistently. The actual buffer sizes used during the cryptographic handshake are recalculated and, in some cases, slightly changed (for example, the device-properties limit drops from 128 to 64 bytes and a new allocation-response size is introduced). There is no direct evidence in the commit that this fixes a security vulnerability; it reads as a refactoring to make the code easier to maintain.

Lower-priorityfix(core): fix OOB read in NFC card emulationby tychovrahe · 74618edb · May 5, 2026 · 1 fileMessage 57 · ThinModerate 68Details
Commit message · tychovrahe

fix(core): fix OOB read in NFC card emulation

[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 · Moderate 68/100

This commit fixes several out-of-bounds (OOB) memory reads in the NFC card emulation code used in Trezor hardware wallets. The code previously trusted the length and content of NFC commands received from a nearby reader without checking whether the received data was actually long enough. A malicious or malformed NFC reader could send a truncated or specially crafted command, causing the device to read memory past the end of the command buffer. The patch adds length checks before accessing command bytes and before looping over variable-length block lists.

Lower-priorityfix(core): fix boot_image_check__verified validationby tychovrahe · 59d8d78e · May 5, 2026 · 2 filesMessage 62 · AdequateHigh 73Details
Commit message · tychovrahe

fix(core): fix boot_image_check__verified validation

[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 · High 73/100

This commit adds a missing safety check in the Trezor hardware wallet's core firmware. Two functions that validate boot images now verify that the firmware actually has permission to read the image's memory before processing it. Without this check, a malformed or attacker-controlled image pointer could potentially be passed in, leading to unauthorized memory reads or a crash during the boot verification process. The fix is straightforward and defensive.

Security candidatefix(core/bootloader): fix codec v1 overflow issuesby tychovrahe · eebd2e5e · May 5, 2026 · 1 fileMessage 67 · AdequateModerate 63Details
Commit message · tychovrahe

fix(core/bootloader): fix codec v1 overflow issues

[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
update trustmemory safetyboot or update path
AI analysis · Moderate 63/100

This patch fixes several integer-handling bugs in the Trezor bootloader's USB message decoder. The changes prevent small or maliciously crafted message sizes from causing arithmetic overflows or underflows when the device calculates how many USB packets to read. Such flaws could potentially let an attacker confuse the bootloader into reading memory out of bounds or behaving unpredictably during a firmware update.

Security candidatefix(core): fix OOB read in read_vendor_headerby tychovrahe · 3b64e4e8 · May 5, 2026 · 10 filesMessage 57 · ThinHigh 74Details
Commit message · tychovrahe

fix(core): fix OOB read in read_vendor_header

[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 · High 74/100

This commit fixes an out-of-bounds memory read bug in the Trezor bootloader's firmware header parser. The read_vendor_header function previously trusted size values inside a firmware file without first checking whether those sizes fit within the actual data buffer. A malformed firmware update or a corrupted on-device firmware image could have caused the bootloader to read memory beyond the allowed region. The patch adds size checks so the parser rejects too-small or oversized headers before using their contents.

Security candidatebuild(core): use BOOTLOADER_DEVEL flag for keys selectionby tychovrahe · a7b34c4a · May 5, 2026 · 30 filesMessage 62 · AdequateLow 30Details
Commit message · tychovrahe

build(core): use BOOTLOADER_DEVEL flag for keys selection

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

This commit renames and restructures how Trezor firmware selects cryptographic signing keys for bootloaders. Previously, non-production builds automatically used weaker development/QA keys. Now, a dedicated BOOTLOADER_DEVEL flag controls that, while a separate FORCE_BOOTLOADER_UPGRADE flag controls whether the firmware should forcibly update the bootloader. The change also prevents combining PRODUCTION=1 with BOOTLOADER_DEVEL=1. This is primarily a build-hardening and internal workflow cleanup; it does not by itself fix a remotely exploitable bug in shipped devices.

AI review queuedfix(core): prevent NFC driver crash on repeated deinitializationby tychovrahe · c856ba57 · May 5, 2026 · 1 fileMessage 62 · AdequateLow 32Details
Commit message · tychovrahe

fix(core): prevent NFC driver crash on repeated deinitialization

[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
second-pass: broader security terminology
AI analysis · Low 32/100

This commit fixes a bug in the Trezor hardware wallet's NFC driver where calling the NFC shutdown function more than once could crash the device. The fix adds a simple check to skip re-deinitializing the SPI hardware if it has already been shut down. It is a defensive hardening fix rather than a clear exploitable vulnerability.

Lower-prioritychore(core): improve logging when USE_N4W1 is unsetby Roman Zeyde · 8ec0077b · May 5, 2026 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

chore(core): improve logging when USE_N4W1 is unset

[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 is a minor code cleanup that changes when a debug-only warning message is printed. It does not fix or introduce any security issue. The change simply avoids logging a misleading warning when no backup method is selected in a non-production debug build.

Lower-prioritychore(core): update UI fixturesby Roman Zeyde · 4627ebeb · May 5, 2026 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · Roman Zeyde

chore(core): update UI 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 the expected visual-output fingerprints (called 'fixtures') used by the device's automated UI tests. It renames test cases to include a '[Display]' marker and adjusts the corresponding hashes, but does not change any firmware code that runs on the Trezor device itself. There is no security-relevant change visible in this commit.

Security candidatefeat(core): add N4W1 backup/recovery flowsby Roman Zeyde · 7325bb60 · May 5, 2026 · 18 filesMessage 90 · StrongLow 31Details
Commit message · Roman Zeyde

feat(core): add N4W1 backup/recovery flows

Can be manually tested by invoking:
```
$ core/emu.py -ea -c trezorctl device setup -b shamir # will run multi-share backup
$ core/tools/n1w1-emu.py run 127.0.0.1:21325 /tmp/tagN # simulate tag connection and I/O
```

Enabled N4W1-based backup/recovery device tests for SLIP-39 single group scenarios.
Other device & click tests will be added in subsequent PRs.

[no changelog]

90/100 · StrongMessage clarity
✓ 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
cryptography-sensitive path
AI analysis · Low 31/100

This commit adds a new backup and recovery method called N4W1 to Trezor firmware. It is a feature addition that lets users back up and restore their wallet seed using a near-field wireless tag instead of typing words on the device screen. The change is mostly about adding new code paths and tests; it does not appear to fix a known security bug, nor does the vendor describe it as a security patch.

Lower-prioritychore: update uv.lockby Roman Zeyde · d5a4fdff · May 5, 2026 · 1 fileMessage 40 · ThinInformational 15Details
Commit message · Roman Zeyde

chore: update uv.lock

[no changelog]

40/100 · ThinMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 15/100

This is a routine lock-file update for a Python dependency manager. It records that an internal Trezor package named 'tvl' now depends on two additional libraries ('cffi' and 'libcst'). There is no indication of a security fix, vulnerability, or malicious change in the commit itself.

AI review queuedrefactor(ethereum): extract functionby Ioan Bizău · f5aa9d87 · May 5, 2026 · 1 fileMessage 47 · ThinInformational 15Details
Commit message · Ioan Bizău

refactor(ethereum): extract function

[no changelog]

47/100 · ThinMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit is a simple code cleanup in the Ethereum clear-signing module. It pulls out a repeated snippet of code that reads variable-length data into a new helper function, then replaces two copies of that snippet with calls to the helper. No behavior changes are visible in the diff.

AI review queuedchore(ethereum): check array data lengthby Ioan Bizău · bbb17455 · May 5, 2026 · 1 fileMessage 57 · ThinLow 49Details
Commit message · Ioan Bizău

chore(ethereum): check array data 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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 49/100

This commit adds a length check to a function that parses arrays of 256-bit unsigned integers in Ethereum transaction data. Previously, if the raw data was not a multiple of 32 bytes, the function would silently ignore leftover bytes at the end. The fix now raises an error instead. This could matter for 'clear signing' displays on a Trezor hardware wallet, where malformed data might otherwise be shown or processed in a misleading way. The commit message does not describe this as a security fix, and no exploit is demonstrated.

AI review queuedchore(ethereum): precomputed approve/transfer sigby Ioan Bizău · 424bc94e · May 5, 2026 · 1 fileMessage 57 · ThinInformational 19Details
Commit message · Ioan Bizău

chore(ethereum): precomputed approve/transfer sig

[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 pathsecond-pass: security-sensitive path
AI analysis · Informational 19/100

This commit replaces runtime computation of two Ethereum function signatures with hard-coded byte values, then adds debug-only assertions to verify those values still match the original computation. It is a minor code cleanup with no apparent security impact. The change does not alter what the device signs, displays, or accepts.

AI review queuedchore(ethereum): check data lengthby Ioan Bizău · e64bfd98 · May 5, 2026 · 1 fileMessage 57 · ThinLow 47Details
Commit message · Ioan Bizău

chore(ethereum): check data 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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 47/100

This commit adds a safety check to a function that verifies unused data is zeroed. Before the change, if a caller passed an invalid 'used_bytes' value (for example, a negative number or one larger than 32), the function could read memory outside the intended 32-byte window or behave unexpectedly. The patch now rejects such invalid values immediately. The commit message calls it a routine cleanup ('chore') and does not claim it fixes a security vulnerability.