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 candidatechore(core): update T3W1 dev bootloaderby tychovrahe · 79471a5b · Nov 3, 2025 · 2 filesMessage 57 · ThinInformational 3Details
Commit message · tychovrahe
chore(core): update T3W1 dev 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 trustboot or update path
AI analysis · Informational 3/100
This commit simply swaps in a newer pre-built bootloader binary for the T3W1 hardware model used in QA/testing, and updates the matching hash list so the firmware recognizes it. There is no source-code change, no description of a security fix, and no public reference linking it to a vulnerability.
Security candidatefix(core): remove "Go to bootloader" item from non-debug buildsby Roman Zeyde · 23d1fb32 · Oct 31, 2025 · 1 fileMessage 62 · AdequateLow 36Details
Commit message · Roman Zeyde
fix(core): remove "Go to bootloader" item from non-debug builds
[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 36/100
This commit removes a 'Go to bootloader' menu option from normal user builds of the Trezor hardware wallet, keeping it only in debug/developer builds. The change prevents an ordinary user from intentionally or accidentally rebooting the device into bootloader mode through the on-device menu. While entering bootloader mode is a normal part of firmware updates, exposing it in production builds could let a local attacker with brief physical access force the device into a state where they could attempt to install modified firmware or interfere with an update. The patch is narrow: it only hides the menu item behind a debug-only compile flag, so the underlying bootloader entry mechanism still exists for legitimate update workflows.
✓ 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 pathboot or update path
AI analysis · Informational 15/100
This commit simply shortens the on-screen label for a device menu item from 'Regulatory certification' to 'Regulatory' (or translated equivalents) and updates the internal translation key from regulatory_certification__title to regulatory__title. It is a cosmetic UI text change with no security relevance.
Security candidatefeat(core): show an explicit warning when `ButtonRequest` ACK is delayedby Roman Zeyde · 661681f5 · Oct 31, 2025 · 11 filesMessage 100 · StrongInformational 19Details
Commit message · Roman Zeyde
feat(core): show an explicit warning when `ButtonRequest` ACK is delayed
This should help the user to avoid the "blank screen" issue[1] (if the layout is closed before `ButtonRequest` loop is done, the user will see a blank screen - until a new message is received).
This PR doesn't allow the user to abort the layout task, since it may leave the THP channel in a "non-writeable" state (if the flow is aborted before `ButtonRequest` THP-level ACK is received, ABP will prevent further writes into the channel[2].
✓ 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
signing or wallet pathboot or update path
AI analysis · Informational 19/100
This commit adds a user-facing warning message on Trezor devices when the connected computer or phone is slow to respond during a button-confirmation step. It is a usability improvement to prevent users from seeing a confusing blank screen; it does not fix a vulnerability and does not introduce a new attack path.
Security candidateRevert "chore(core): rename `ble__waiting_for_host` to `words__waiting_for_host`"by Roman Zeyde · c8073aa2 · Oct 31, 2025 · 13 filesMessage 65 · AdequateInformational 15Details
Commit message · Roman Zeyde
Revert "chore(core): rename `ble__waiting_for_host` to `words__waiting_for_host`"
This reverts commit f4424b657e4eca2f07824b1e8c40ce2f4e41c7c8.
65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
Why it was queued
signing or wallet pathboot or update path
AI analysis · Informational 15/100
This commit simply reverts a previous internal renaming of a user-interface text label. It changes the translation key for a Bluetooth 'Waiting for connection...' message from 'words__waiting_for_host' back to 'ble__waiting_for_host' and updates the generated translation files accordingly. There is no security relevance.
Security candidatefix(core): reduce +/- buttons' size to allow more text to fitby Roman Zeyde · 23f6518a · Oct 31, 2025 · 3 filesMessage 94 · StrongInformational 17Details
Commit message · Roman Zeyde
fix(core): reduce +/- buttons' size to allow more text to fit
Otherwise, "seconds" translation may overflow on T3W1.
94/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✓ Names security-relevant behavior explicitly
Why it was queued
memory safetyboot or update path
AI analysis · Informational 17/100
This is a visual layout tweak for the Trezor T3W1 hardware wallet. It slightly shrinks the plus/minus buttons on a settings screen so that translated text (like the word "seconds") does not get cut off. There is no indication of a security vulnerability or exploit.
Security candidatefix(core/bootloader): disable 'pair new device' menu item in case BLE is disabledby tychovrahe · a9c676f9 · Oct 30, 2025 · 1 fileMessage 62 · AdequateInformational 19Details
Commit message · tychovrahe
fix(core/bootloader): disable 'pair new device' menu item in case BLE is disabled
[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 19/100
This commit changes the bootloader menu on a Trezor hardware wallet so that the 'Pair new device' Bluetooth option is grayed out when Bluetooth Low Energy (BLE) is disabled. It is a UI consistency fix rather than a fix for an exploitable vulnerability. There is no evidence in the commit or supplied references that this resolves a security issue.
fix(core/bootloader): erase protected backup ram on wipe code entry
[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 · Moderate 57/100
This bootloader patch makes the device erase a special protected memory region (backup RAM) when the user enters a 'wipe code' to factory-reset the device. Without this cleanup, sensitive leftovers could remain in that protected memory after a wipe. The change is small and defensive, but the commit message does not explain what specific risk it fixes.
Security candidatefeat(core): respect BLE settings in bootloaderby tychovrahe · 20dfec51 · Oct 30, 2025 · 4 filesMessage 57 · ThinLow 35Details
Commit message · tychovrahe
feat(core): respect BLE settings in bootloader
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 trustboot or update path
AI analysis · Low 35/100
This commit changes the Trezor bootloader so that Bluetooth Low Energy (BLE) settings are remembered across reboots. Previously, the bootloader would turn BLE back on even if the user had turned it off. Now the bootloader reads a saved 'BLE enabled' flag from backup memory and only enables BLE if the user had left it on. The pairing/setup workflows still force BLE on because they need it. This is a privacy/usability fix rather than a critical security patch, but it prevents an unwanted BLE radio from being active when the user expects it to be off.
Security candidatefeat(core): notify host on power status change using internal notification systemby tychovrahe · 25dadcef · Oct 30, 2025 · 12 filesMessage 62 · AdequateInformational 19Details
Commit message · tychovrahe
feat(core): notify host on power status change using internal notification system
[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 19/100
This commit adds a way for the Trezor device to tell the connected computer or phone when its power source changes (for example, when USB is plugged in or a wireless charger is placed nearby). It also exposes two new status flags, usb_connected and wireless_connected, in the device's feature report. There is no direct security fix here; it is a feature that improves real-time status reporting.
- follow the design principle for Screen to construct them with appropriate components which should do the right thing internally
[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
boot or update path
AI analysis · Informational 13/100
This is a small internal code cleanup in the user-interface code for the Trezor hardware wallet's newer 'Eckhart' layout. It removes a special 'pagination hint' builder flag and instead lets callers pass an ordinary page-counter hint component. The visible behavior is meant to stay the same: a page counter appears only when content spans more than one page. There is no indication this change fixes a security bug or introduces a vulnerability.
- PageCounter and ActionBar::PaginateOnly are shown only if the content does not fit the page - offloading place to a free-standing function saves 3kB of flash space [no changelog]
70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
boot or update path
AI analysis · Informational 14/100
This is a routine user-interface code cleanup for the Trezor hardware wallet. It changes how on-screen navigation controls (page counters and action bars) are shown, hiding them when text fits on a single page. There is no indication this fixes a security bug or introduces a security risk.
fix(storage,core/embed): fix and improve estimates
[no changelog]
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
seed or entropy path
AI analysis · Informational 19/100
This commit is a code-quality and accuracy improvement for how the Trezor firmware estimates how long PIN operations will take. It replaces a single coarse heuristic with finer-grained per-command timing estimates for the Optiga and Tropic secure chips, adds a helper that models the chip's security-event-counter throttling, and moves the estimate logic closer to the functions it describes. There is no direct security vulnerability being fixed; the change is about making progress-bar timing more accurate, which indirectly helps user experience and may make side-channel timing analysis slightly more predictable. The commit message says '[no changelog]' and does not claim to fix a security issue.
Security candidatefeat(core): use ecc sram in bootloader/boardloaderby cepetr · 0cac085e · Oct 29, 2025 · 6 filesMessage 62 · AdequateInformational 11Details
Commit message · cepetr
feat(core): use ecc sram in bootloader/boardloader
[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 11/100
This commit changes how the Trezor hardware wallet's earliest startup code (boardloader and bootloader) uses internal memory on the T3B1 model. It shifts small data and uninitialized-memory sections from one RAM bank (AUX1_RAM) to another (MAIN_RAM), and slightly adjusts the size and start addresses of those RAM regions. The title says this is to 'use ECC SRAM'—memory with built-in error-correction, which is a reliability/hardening improvement, not a vulnerability fix. There is no direct evidence in the commit of a security bug being patched.
Security candidatefix(core): get rid of confidential global varsby cepetr · dfb72957 · Oct 29, 2025 · 37 filesMessage 57 · ThinModerate 63Details
Commit message · cepetr
fix(core): get rid of confidential global vars
[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 pathboot or update path
AI analysis · Moderate 63/100
This commit removes a special firmware memory section called ".confidential" that was meant to keep sensitive data (like cryptographic keys and seed-derived values) separate from ordinary memory. Previously, variables marked with a CONFIDENTIAL attribute were placed in this dedicated RAM region and copied from flash at startup. The patch deletes that whole mechanism across all Trezor models and instead marks sensitive local variables with a new LOCAL_CONFIDENTIAL macro. On the legacy build this still maps to the old behavior, but on the modern STM32 builds it effectively becomes a no-op, meaning those sensitive buffers now live in normal RAM/data sections. The change is presented as a cleanup, but it reduces a defense-in-depth isolation feature and could make it easier for a successful attacker to locate or extract secret material in memory.
Security candidatefeat(core): use LED effect for BLE pairingby Roman Zeyde · 5f0727a1 · Oct 28, 2025 · 12 filesMessage 57 · ThinInformational 15Details
Commit message · Roman Zeyde
feat(core): use LED effect for BLE pairing
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 user-interface feature: it makes the Trezor T3W1 device show a special LED lighting effect while Bluetooth Low Energy (BLE) pairing is in progress. It refactors how the firmware tracks LED state so that the renderer can request either a static color or a hardware-driven effect (pairing/charging). There is no security-relevant change visible in the diff.
feat(core): implementation of backlight brightness correction.
Having re-considered that, the gamma correction function shall stay in the backlight driver's level. Although, not the GAMMA EXPONENT. The exponent is to be passed as an argument to backlight_init() function either to be used/not_used further on. This exponent shall be defined for different displays separately in their respective header files.
[no changelog]
85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
authentication path
AI analysis · Informational 15/100
This commit is a routine hardware driver improvement. It makes the screen backlight brightness curve adjustable per display panel by passing a gamma exponent into the backlight driver. There is no security-relevant change: no memory corruption, no cryptographic code, no privilege changes, and no user data handling.
feat(core): implementation of backlight brightness correction.
Constants' re-calculation so that the physical brightness remains static between the old and new approach. It should minimize the effect of the new approach for user, tests at production line, etc.
[no changelog]
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✓ Mentions testing or verification
Why it was queued
boot or update path
AI analysis · Informational 18/100
This commit adjusts the screen brightness numbers used by the Trezor T3W1 hardware wallet so that the actual perceived brightness stays the same after a change in how brightness is calculated. It is a display-calibration tweak, not a security fix or vulnerability. The only user-visible effect should be that brightness levels look the same as before.
✓ 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 internal code cleanup for the T3W1 hardware wallet's on-device menu. It changes how menu choices are represented behind the scenes (from unique object tokens to plain integer codes) and reorganizes the Python handler code into smaller functions. There is no indication this fixes a security bug or introduces a new vulnerability.
Security candidate feat(core): save binary representation of mnemonic in storage for Cardano secret derivationby Martin Pastyřík · 45d21530 · Oct 27, 2025 · 11 filesMessage 62 · AdequateLow 33Details
Commit message · Martin Pastyřík
feat(core): save binary representation of mnemonic in storage for Cardano secret derivation
[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 33/100
This commit changes how Trezor stores wallet backup words (mnemonics) for Cardano support. It now keeps an extra binary copy of the mnemonic in device storage, separate from the encrypted text version. The change is described as a feature, not a security fix. It does not appear to introduce an obvious remote exploit, but it does add a new storage location holding sensitive seed material and includes a debug-only option that lets invalid mnemonics be loaded without raising an error.
Security candidatefeat(test): add upgrade test for app storage version 3by Martin Pastyřík · b065c9c1 · Oct 27, 2025 · 1 fileMessage 72 · AdequateInformational 12Details
Commit message · Martin Pastyřík
feat(test): add upgrade test for app storage version 3
[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
boot or update path
AI analysis · Informational 12/100
This commit only adds a new automated test to the Trezor firmware test suite. It checks that Cardano public keys and addresses stay the same before and after a firmware upgrade that migrates internal app storage from version 2 to the current version. There is no change to the actual firmware code, wallet logic, or security behavior.
Security candidatechore(test): update Cardano unit tests to use saved binary mnemonicby Martin Pastyřík · 59aeeb2e · Oct 27, 2025 · 4 filesMessage 72 · AdequateInformational 15Details
Commit message · Martin Pastyřík
chore(test): update Cardano unit tests to use saved binary mnemonic
[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
seed or entropy path
AI analysis · Informational 15/100
This commit only updates Cardano unit tests to convert text seed phrases into binary format before passing them to a cryptographic function. It does not change any production wallet code, user-facing behavior, or security-sensitive logic. There is no indication this fixes or introduces a vulnerability.
chore(core): add PROD payment request public signing key
[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 boundary
AI analysis · Informational 18/100
This commit simply fills in a previously empty placeholder with the real production public key used to verify signed payment requests. By itself, it is a routine configuration/key-installation change, not an exploitable vulnerability. The only security-relevant observation is that before this commit, payment-request signatures using this key could not have been validated, so the feature would not have worked in production.
Security candidatechore: remove `GetNonce` and `PaymentRequest` from experimental featuresby M1nd3r · 6b844d37 · Oct 27, 2025 · 24 filesMessage 62 · AdequateInformational 19Details
Commit message · M1nd3r
chore: remove `GetNonce` and `PaymentRequest` from experimental features
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
nonce handlingcryptography-sensitive pathsigning or wallet path
AI analysis · Informational 19/100
This commit removes the 'experimental' label from two Trezor firmware features: PaymentRequest (used in several cryptocurrency transaction flows) and GetNonce (a random-number request). It also updates generated code and tests accordingly. The change itself is a feature graduation, not a fix for a known security bug. There is no direct evidence in the commit that it patches an active vulnerability, but it does widen the audience for these features by removing the experimental opt-in requirement.
refactor(core): split reboot and reboot_to_bootloader functions
[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 internal code cleanup: it splits one Python/C function that handled three different reboot behaviors into three separate, clearly named functions. There is no change to what the device actually does during a reboot or firmware upgrade, and no security bug is introduced or fixed.