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.
Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.
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
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
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.
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 …
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
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
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…
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…
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…
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…
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…
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…
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.…
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…
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…
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…
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 …
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
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…
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.
Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.
Security candidatefix(core): get rid of CONST weirdnessby cepetr · 7b27f4be · May 8, 2026 · 3 filesMessage 57 · ThinInformational 12Details
Commit message · cepetr
fix(core): get rid of CONST weirdness
[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
cryptography-sensitive path
AI analysis · Informational 12/100
This commit is a cleanup of how the word 'const' is used in the Trezor firmware's ed25519 cryptographic code. Previously, the code defined a custom 'CONST' macro that meant 'const' on newer compilers and nothing on older GCC versions. The commit removes that macro and uses plain 'const' everywhere. It also adjusts a Rust build script comment and compiler flag. There is no indication in the commit that this fixes a security vulnerability or changes runtime behavior.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100
This commit is a straightforward internal code refactor. It renames a build feature flag from 'applet' to 'applets' and replaces the KERNEL preprocessor guard with a new USE_APPLETS guard in the relevant source files. There is no change to actual security logic, no bug fix, and no indication of a vulnerability being addressed.
Lower-priorityrefactor(core): use setjmp instead of native x64 implby cepetr · 5efbe1a2 · May 8, 2026 · 1 fileMessage 62 · AdequateInformational 12Details
Commit message · cepetr
refactor(core): use setjmp instead of native x64 impl
[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 changes a build configuration for Trezor's Unix emulator so that MicroPython uses the standard C setjmp/longjmp mechanism for non-local returns instead of a custom x86_64 assembly implementation. There is no direct security fix here; it is a portability/refactoring change. It does not affect the actual Trezor hardware firmware.
fix(core): add prebuilt std/core for thumbv8m target
[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 updates the Trezor firmware build environment to include prebuilt Rust standard library support for a newer ARM processor target (thumbv8m, used in the T3 device model). It is a build tooling change with no visible security relevance.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Low 31/100
This commit fixes how the Trezor firmware build generates a Rust binding for an internal identifier type called Qstr. Previously, the build always assumed the underlying C enum was a 32-bit integer, but the firmware actually compiles with smaller 16-bit enums. The mismatch could cause the generated Rust code to use the wrong size, leading to memory corruption or crashes when Rust and C exchange Qstr values. The patch makes the build choose the correct enum size for firmware versus host builds.
AI review queuedrefactor(ethereum): use DisplayFormat to parse yieldingby Ioan Bizău · 6daa3bd6 · May 7, 2026 · 2 filesMessage 62 · AdequateInformational 12Details
Commit message · Ioan Bizău
refactor(ethereum): use DisplayFormat to parse yielding
[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 12/100
This commit is a code cleanup in Trezor's Ethereum transaction handling. It replaces a hand-rolled method of reading Ethereum vault transaction data with a shared, more structured parsing system called DisplayFormat. There is no indication this fixes a security bug; it appears to be a refactoring change with no functional security relevance stated by the vendor.
Security candidatechore: migrate from SDL2 to SDL3by M1nd3r · 9fe8ae02 · May 7, 2026 · 19 filesMessage 57 · ThinInformational 20Details
Commit message · M1nd3r
chore: migrate from SDL2 to SDL3
[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 20/100
This commit updates the Trezor firmware build system and emulator code to use SDL3 instead of the older SDL2 library. It is a routine dependency migration affecting only emulator builds (software simulations of the hardware wallet), not the real device firmware. The changes rename functions and constants to match SDL3's API, adjust build scripts, and add a couple of small safety checks. There is no indication this fixes a known security vulnerability.
Lower-prioritychore: update fixtures after migration to SDL3by M1nd3r · 00d88185 · May 7, 2026 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · M1nd3r
chore: update fixtures after migration to SDL3
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 expected test result hashes (called 'fixtures') in a single test data file. The firmware code itself is not changed, and there is no indication of a security fix or vulnerability. It appears to be routine maintenance after switching the UI test framework to SDL3, which likely changed how screen snapshots are rendered and therefore changed their hashes.
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): replace printk with logby tychovrahe · b9c3a808 · May 6, 2026 · 4 filesMessage 57 · ThinInformational 15Details
Commit message · tychovrahe
chore(nordic): replace printk with log
[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 is a routine cleanup change in the Nordic Bluetooth Low Energy firmware for Trezor hardware wallets. It replaces old-style printk debug messages with Zephyr's standard logging macros (LOG_ERR, LOG_WRN) and removes an unused logging configuration option. There is no security-relevant change here.
✓ 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.
Lower-prioritychore(nordic): remove deprecated symbol CONFIG_NFCT_PINS_AS_GPIOSby tychovrahe · bbfdc15b · May 6, 2026 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · tychovrahe
chore(nordic): remove deprecated symbol CONFIG_NFCT_PINS_AS_GPIOS
[no changelog]
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 15/100
This commit removes a single deprecated configuration option, CONFIG_NFCT_PINS_AS_GPIOS, from the Nordic Bluetooth Low Energy firmware build file. The change is a routine cleanup with no visible security relevance. It does not alter runtime behavior in a way that introduces or fixes a vulnerability based on the information provided.
Lower-prioritychore(nordic): use SPI alias for trezor SPI communication selectionby tychovrahe · 8c3829e4 · May 6, 2026 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · tychovrahe
chore(nordic): use SPI alias for trezor SPI communication 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
AI analysis · Informational 15/100
This is a minor internal cleanup in the Nordic (nRF52) Bluetooth Low Energy firmware for a Trezor hardware wallet. It replaces a hard-coded reference to 'spi0' with a device-tree alias called 'spi-master' so the same code can more easily be reused across different board revisions. There is no security-relevant change visible in the diff.
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.
✓ 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.
✓ 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-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
✓ 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-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.
✓ 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.
✓ 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.
✓ 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.