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 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 only updates test data. It corrects the capitalization (checksum) of Ethereum addresses used in automated tests and refreshes the expected screen-shot hashes those tests compare against. There are no changes to the actual Trezo…
This commit changes the Trezor Python library so that, when setting up a Trezor Model One (the original Trezor 1 device), it now performs an entropy check by default if the device runs firmware 1.13.1 or newer. Previously, the library only…
Enables a previously disabled security/validation feature (entropy check) for a specific device modelAdds version-gated behavior to avoid errors on older firmwareDefensive hardening of wallet setup randomness verification
This commit is a routine cleanup that removes the old SCons build system files from the Trezor Core firmware repository. It deletes Makefiles, SConscripts, and related Python helper scripts, but does not change any firmware source code, cr…
This commit updates Trezor's embedded MicroPython interpreter from an older version to 1.28.0. It pulls in several upstream MicroPython bug fixes, including stricter buffer-size checks for converting integers to bytes, a new stack-safety A…
Synchronizes upstream MicroPython fixes that include buffer-size and stack-safety hardeningPrevents Ctrl+C interruption of frozen boot code, reducing denial-of-service/control-flow risk during bootFixes sys.stdout.buffer.write() return value, which could affect code relying on correct I/O semantics
This commit is a simple code cleanup that replaces a common MicroPython string-creation pattern with a new helper function. It does not change what the code does, only how it is written. There is no security issue visible in the change.
This is a large but straightforward internal cleanup: the project switched from using the MicroPython `ubinascii` module's `hexlify`/`unhexlify` functions to the standard Python `bytes.hex()` and `bytes.fromhex()` methods. The change remov…
This commit is a routine code cleanup in the Trezor firmware's embedded MicroPython modules. It replaces an older, two-step object allocation pattern with a newer helper that allocates memory and sets the object type in one step. The chang…
No security-relevant logic changes observedNo input validation changesNo memory safety bug fixes (e.g., no overflow, use-after-free, or uninitialized memory fixes)
This commit is a routine maintenance refactor. MicroPython removed its built-in 'utimeq' module, so Trezor copied the same code into its own firmware tree and renamed it to 'timeq'. There is no security bug being fixed and no new vulnerabi…
This is a code cleanup change in Trezor's firmware that swaps one MicroPython internal helper for another. It replaces calls that create byte or string objects from a vstr buffer with newer, purpose-built helpers. The commit message says t…
Refactor only: helper function renames with equivalent semanticsNew str helper adds UTF-8 validation; bytes helper does not validateNo input validation, length, or error-handling changes observed
This commit is a routine internal cleanup that switches how Trezor's firmware defines built-in MicroPython object types. It replaces older, manually-written type structures with a newer macro provided by the upstream MicroPython project. T…
This commit is a straightforward internal code refactor. It adds a small C wrapper file so that Rust code can call two specific internal MicroPython functions that are declared as static/inline in headers. There is no user-facing change, n…
This commit is a routine build-system update for the Trezor firmware. It changes where the build process looks for the MicroPython cross-compiler binary (mpy-cross) and synchronizes a configuration header with newer MicroPython code. There…
This commit updates the Trezor firmware build system to use a newer MicroPython mechanism for tracking special memory pointers called 'root pointers.' It removes hard-coded lists of these pointers from configuration files and instead colle…
GC root pointer handling changedBuild system now auto-collects root pointers instead of hard-coding themRemoves manual MICROPY_PORT_ROOT_POINTERS lists
Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.
Security candidatechore: enable pyright for emu.py and scripts in test/by Martin Milata · 16a7a0b8 · Jul 9, 2026 · 9 filesMessage 72 · AdequateInformational 15Details
Commit message · Martin Milata
chore: enable pyright for emu.py and scripts in test/
[no changelog]
72/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
cryptography-sensitive pathboot or update path
AI analysis · Informational 15/100
This commit is a routine developer tooling change. It turns on stricter Python type checking (Pyright) for the emulator script and some test helper scripts, and makes small code adjustments so those files pass the new checks. There is no change to the security-critical Trezor firmware itself, wallet operations, or cryptographic handling.
Security candidatebuild(core): enable and start Tropic emulator by defaultby Martin Milata · afb0fdd4 · Jul 9, 2026 · 16 filesMessage 62 · AdequateInformational 18Details
Commit message · Martin Milata
build(core): enable and start Tropic emulator by default
[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 pathauthentication path
AI analysis · Informational 18/100
This commit changes how the Trezor firmware build and test tooling handles the Tropic secure-element emulator. Previously, Tropic support was disabled by default in Unix emulator builds and had to be explicitly enabled; now it is enabled by default and the test harness starts the Tropic model automatically. It also refactors port allocation so each emulator worker reserves a wider block of ports. The changes are confined to build scripts, CI workflows, and test/development tooling. There is no direct fix for a runtime vulnerability in shipped firmware, but enabling a hardware-security emulator by default could theoretically expose test-only code paths or secrets if the emulator is misused outside of testing.
feat(core/python): create testing module in trezorlib
72/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
cryptography-sensitive pathsigning or wallet pathboot or update pathauthentication path
AI analysis · Informational 15/100
This commit is a routine code reorganization: it moves existing test-only helper code from the project's internal test directory into the public `trezorlib` Python package as a new `testing` module. The moved code includes utilities for handling emulated devices during tests, translation blob helpers, and Bitcoin compact-size encoding. There is no change to the firmware that runs on Trezor devices, no change to production cryptography, and no security fix or vulnerability introduced.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathboot or update path
AI analysis · Informational 15/100
This commit adds support for a new Trezor hardware model, the T3T2. It is a product enablement change that introduces board definitions, memory layouts, bootloader hashes, vendor headers, and Python tooling entries for the new device. There is no indication of a security vulnerability or a fix for one.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathboot or update path
AI analysis · Informational 15/100
This commit adds support for a new Trezor development board called D003 (based on an STM32U5 Nucleo evaluation kit). It is purely an enablement change for internal hardware bring-up: new configuration files, memory layout, bootloader hash, vendor headers, and build-system entries. There is no indication it fixes a security bug or introduces a vulnerability in shipping products.
✓ 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 adds support for a new microcontroller chip (the STM32U5A5) to the Trezor firmware build system. It is a hardware-enablement change: it wires up the right compiler flags, linker scripts, memory layouts, and trustzone settings so the existing code can compile and run on the new chip. There is no user-facing behavior change, no bug fix, and no security patch in this diff.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
constant-time or timing behaviorcryptography-sensitive pathboot or update path
AI analysis · Moderate 63/100
This commit hardens how Trezor compares secret values (passwords, PINs, cryptographic tags, and checksums) so that an attacker cannot learn information by measuring how long the comparison takes. It also adds a fault-injection check: if a glitch skips the comparison loop, the device is supposed to detect that and halt. The change is defensive and reduces the risk of side-channel and fault attacks, but it does not by itself fix a known, directly exploitable bug.
Security candidatetest(core/caesar): visit menu only on last pageby Roman Zeyde · 7fa4a20f · Jul 7, 2026 · 2 filesMessage 67 · AdequateInformational 17Details
Commit message · Roman Zeyde
test(core/caesar): visit menu only on last page
[no changelog]
67/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 17/100
This commit is a small test-only UI adjustment for the Trezor hardware wallet's Caesar layout. It changes when a menu icon is reported as visible in automated UI snapshots, limiting it to the last page of a multi-page flow. The code change only affects test fixtures and snapshot hashes, not user-facing security behavior. There is no indication this fixes a security vulnerability.
Security candidatechore(core/caesar): shorten `confirm_summary()` text if possibleby Roman Zeyde · bee67ca2 · Jul 7, 2026 · 2 filesMessage 77 · AdequateInformational 15Details
Commit message · Roman Zeyde
chore(core/caesar): shorten `confirm_summary()` text if possible
No need to add a newline if `amount` or `amount_label` are empty.
[no changelog]
77/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
cryptography-sensitive pathboot or update path
AI analysis · Informational 15/100
This is a minor user-interface polish change for the Trezor hardware wallet's Caesar layout. It removes an unnecessary blank line from transaction summary screens when either the amount label or the amount value is empty. There is no security relevance: it does not change what the device signs, how it signs, or any cryptographic behavior. The large diff is mostly updated test screenshot hashes reflecting the slightly different screen layout.
fix(core): scope allocation of clear signing definitions
Otherwise, they will not be garbage-collected.
[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
signing boundarysigning or wallet path
AI analysis · Low 27/100
This commit changes how Trezor's Ethereum 'clear signing' helper data is loaded in memory. Previously, all definitions were stored in a permanent module-level list that MicroPython's garbage collector could not reclaim. The patch wraps them in a generator function so they are created on demand and can be freed afterward. The stated goal is to avoid memory pressure, not to fix a security bug. There is no direct evidence of an exploitable vulnerability in the diff itself.
Security candidatefeat(core): switch from Python THP implementation to Rust-based oneby Martin Milata · 00ec9a7f · Jul 6, 2026 · 43 filesMessage 62 · AdequateLow 36Details
Commit message · Martin Milata
feat(core): switch from Python THP implementation to Rust-based one
[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
cryptography-sensitive pathboot or update pathparser or protocol path
AI analysis · Low 36/100
This commit replaces the Python implementation of Trezor's THP (Trezor Host Protocol) with a Rust-based one. It is a large refactoring that moves channel state management, encryption, packet handling, and handshake logic from Python into a Rust module exposed as `trezorthp`. The change also adjusts how credentials are validated, how sessions are cached, and how the event loop handles reads, writes, and retransmissions. There is no explicit security bug in the diff, but the scope of the rewrite and the removal of several safety checks (for example around buffer allocation and unexpected-message handling) create a non-trivial risk of introducing memory-management, concurrency, or protocol-edge-case bugs. The vendor does not describe this as a security fix.
Security candidaterefactor(core/rust): use ZeroizeOnDropby Martin Milata · fdb58ef8 · Jul 6, 2026 · 7 filesMessage 57 · ThinInformational 18Details
Commit message · Martin Milata
refactor(core/rust): use ZeroizeOnDrop
[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 path
AI analysis · Informational 18/100
This commit is a code cleanup in the Trezor firmware's Rust cryptography code. It replaces hand-written memory-clearing code with a standard library feature called ZeroizeOnDrop. The goal is the same: erase sensitive cryptographic data from memory when objects are destroyed. There is no evidence this commit fixes an active security bug; it is a defensive hardening and refactoring change.
Security candidatefix(core/ui): avoid signed overflow when calculating text widthby Martin Milata · 7c429403 · Jul 6, 2026 · 1 fileMessage 67 · AdequateLow 42Details
Commit message · Martin Milata
fix(core/ui): avoid signed overflow when calculating text width
[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 safety
AI analysis · Low 42/100
This commit fixes a bug in how the Trezor hardware wallet calculates the width of on-screen text. Previously, very long text could cause an internal counter to overflow from a large positive number to a negative number, which might make text appear incorrectly sized or positioned. The fix uses saturating arithmetic so the counter stops at the maximum safe value instead of wrapping around.
refactor(rust/trezor-thp): keep track if channel finished pairing/credentials
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
credential or privilege state
AI analysis · Informational 12/100
This commit is a straightforward internal code refactor in the Trezor hardware wallet's Rust transport-handshake protocol (THP) library. It replaces a simple pairing-state field with a richer 'phase' enum that tracks whether the channel is still in the pairing/credential setup stage or has moved to encrypted application messaging. The change adds helper methods so the application can explicitly mark pairing as finished. There is no direct security fix here; it is a structural improvement that makes state tracking clearer and less error-prone for callers.
✓ 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 adds a new Rust implementation of the Trezor Host Protocol (THP), which is the secure communication layer between a Trezor hardware wallet and a host computer/phone. It introduces encryption, channel management, pairing, and credential verification. Because this is a large new cryptographic and network-facing subsystem, any bugs here could affect device security, but the commit itself is a feature addition rather than a documented fix for a known vulnerability.
refactor(core/rust): trezor-crypto backend glue for noise-protocol
[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
cryptography-sensitive path
AI analysis · Low 35/100
This commit refactors the cryptographic glue code that connects Trezor's embedded firmware to a Rust-based Noise protocol implementation used for secure device communication. The most notable security-relevant change is the addition of a constant-time comparison function for checking AES-GCM authentication tags, which helps prevent timing-based attacks that could leak information about the tag. The commit also splits AES-GCM into separate encrypt and decrypt types, adds a non-pinned SHA-256 wrapper, and introduces wrappers for Curve25519 and AES-GCM to satisfy the Noise protocol backend interface. There is no explicit vendor statement that this fixes a security vulnerability, and no independent researcher is credited.
✓ 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 fixes a spelling mistake in the legal license text at the top of six source files. The typo 'HMAC_SHALL' is corrected to 'SHALL'. It does not change any program logic, security behavior, or compiled code.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100
This commit only adds new automated tests for an existing cryptographic handshake protocol called noise_xxpsk3. It does not change the actual security code, fix a bug, or introduce a vulnerability. The tests verify that the protocol works correctly, that tampered messages are rejected, and that keys are wiped after use.
✓ 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 15/100
This commit is a simple rename of the cryptographic module from 'noise' to 'noise_kk1' across file names, function names, constants, and build scripts. No security behavior changes are visible in the diff. It is a code-maintenance refactor, not a security fix.
Security candidatefix(crypto): minor fixes in `noise_kk1`by M1nd3r · 1b60dbbf · Jul 3, 2026 · 1 fileMessage 80 · StrongLow 31Details
Commit message · M1nd3r
fix(crypto): minor fixes in `noise_kk1`
- Mark `split` as static in `noise_kk1`. - Add missing memzero to `noise_kk1`. - Return false on encryption failure in `noise_kk1`. - Prevent using memcpy with NULL in noise. - Using `memcpy` to copy data from NULL buffer results in potentially undefined behavior. This raises a runtime error when using UB sanitizer.
[no changelog]
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
defensive validationcryptography-sensitive path
AI analysis · Low 31/100
This commit fixes several minor issues in a cryptographic module used for secure device communication. The changes include preventing undefined behavior when copying from NULL pointers, zeroing out sensitive key material after use, and properly returning errors when encryption fails. These are defensive hardening fixes rather than a clear, exploitable vulnerability.
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 7/100
This commit adds a brand-new cryptographic module implementing the Noise_XXpsk3_25519_AESGCM_SHA256 protocol for secure encrypted communication. It is a feature addition with no changelog entry and no accompanying security advisory. The code itself is a clean implementation of a well-known cryptographic handshake, but because it is new code handling private keys, pre-shared keys, and AES-GCM encryption, it is security-relevant by nature. There is no direct evidence in the commit or supplied references that this fixes a known vulnerability or introduces a bug.
where some other completely unrelated package provides a "version.h" file that happens to come first on the include path.
"version.h" is too generic a name to generally rely on C's imperfect header resolution; a redirect via a symlink seems to be the most correct solution
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
boot or update path
AI analysis · Informational 17/100
This commit fixes a build problem in Trezor's firmware. The code used a very common filename, version.h, which could accidentally pull in the wrong file from another software package during compilation. The fix creates a symlink with a more specific name, firmware_version.h, and uses that instead. There is no direct evidence this is a security vulnerability; it appears to be a build reliability fix.
build(core): avoid unused variable warning in rng_fill_buffer_strong
When neither USE_OPTIGA nor USE_TROPIC, rng_fill_buffer_strong is just rng_fill_buffer with extra steps. Make that explicit in code; otherwise the `dst` variable is never read from and we get a warning under clang.
97/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
entropy or randomnessseed or entropy path
AI analysis · Informational 15/100
This is a minor build cleanup. A developer added a compile-time branch so that when neither the Optiga nor Tropic security chips are used, the 'strong' random-number wrapper simply calls the normal random-number function and returns. The only reason for the change is to silence a compiler warning about an unused variable. There is no functional security change.
Security candidatefeat(ethereum): allow approve and transfer to send ethby PrisionMike · e785f6c6 · Jul 2, 2026 · 8 filesMessage 62 · AdequateModerate 64Details
Commit message · PrisionMike
feat(ethereum): allow approve and transfer to send eth
[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
cryptography-sensitive pathsigning or wallet path
AI analysis · Moderate 64/100
This commit changes how Trezor devices display Ethereum transactions when someone sends ERC-20 token approval or transfer calls that also include native ETH. Previously, the device might not clearly show the attached ETH, so a user could unknowingly approve sending tokens plus real ETH. The patch makes the device surface that native ETH amount on the confirmation screen. It is a defensive fix that improves transparency rather than introducing a vulnerability.
feat(clear_signing, ethereum) add new provider names.
[no changelog]
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing boundarysigning or wallet path
AI analysis · Informational 15/100
This commit is a routine feature update for Trezor's Ethereum 'clear signing' feature. It adds human-readable names for more DeFi and staking providers (like Lido, Morpho, Kiln) and makes the address lookup aware of which blockchain network (chain ID) an address belongs to. There is no security vulnerability or bug fix here.