TZ
← All projectsTrezor

Trezor firmware

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

BitcoinHardware walletsNormal
Repository coverage

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

923security candidates311second-pass queue2662AI analyses
323commits · 30 days
552commits · 60 days
1337commits · 180 days
2644commits · 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
482Strong · 80–100
1323Adequate · 60–79
863Thin · 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.
tychovrahe292104292461
Roman Zeyde579182576272
PrisionMike945994273
Andrew Kozlik712470265
obrusvit21376213164
Jakub Janků22919182
Martin Pastyřík24823173
cepetr19682196059
Ioan Bizău23076230059
M1nd3r20772207071
Lukas Bielesch846784067
Martin Milata17446174063
Analysis record

Published AI watches

Last scanned 7 minutes ago

Informational 15 AI analysisMessage 72 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

test(core/ethereum): EIP-7702 tuple nonce must be non-zero

This commit only changes test data and expected screen fingerprints for Trezor's Ethereum EIP-7702 signing tests. It replaces one test case that used a nonce of 0 with the same scenario using a nonce of 67, because the real-world protocol …

Test-only change with no production code modificationEIP-7702 tuple nonce constraint reflected in test dataExpected UI test hashes regenerated for renamed test vector
76cd2ed4by Roman Zeyde+28−282 files
No security note in commit
Informational 15 AI analysisMessage 85 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

build(core): propagate emulator feature from projects

This commit is a build-system cleanup for the Trezor firmware's Rust code. It changes how the 'emulator' feature flag is passed between different software components. The firmware project no longer supports the emulator feature (which make…

5642074cby matejcik+5−44 files
No security note in commit
Low 27 AI analysisMessage 93 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(core): digest EIP-1559/EIP-7702 magic without RLP encoding

This commit is a small internal cleanup in Trezor's Ethereum signing code. It changes how two special 'magic' bytes (the transaction type markers for newer Ethereum transaction formats) are fed into the cryptographic hash. Previously the c…

Refactor of cryptographic hashing path for Ethereum transaction signingRemoval of implicit assumption that RLP encodes small integers as single bytesChange affects EIP-1559 and EIP-7702 transaction type markers
2362bc6fby Roman Zeyde+6−32 files
No security note in commit
Informational 15 AI analysisMessage 98 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

test(core): test RLP self-encoding bytestrings

This commit only adds a new unit test to the Trezor firmware codebase. It verifies that very short Ethereum RLP-encoded byte strings and small positive integers encode to themselves, which is a known property of the RLP specification. No p…

41538733by Roman Zeyde+16−01 file
No security note in commit
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
Repository ledger

Explore captured commits

Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.

AI review queuedchore: remove unused ThpError - INVALID_DATA [no changelog]by M1nd3r · 72e55603 · Sep 22, 2025 · 3 filesMessage 77 · AdequateInformational 15Details
Commit message · M1nd3r

chore: remove unused ThpError - INVALID_DATA
[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
parser or protocol pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit simply removes an unused error code named INVALID_DATA from the Trezor hardware wallet's new THP (Trezor Host Protocol) code. It deletes the numeric constant, the matching Python exception class, and the mapping that translated the error number into that exception. There is no change to how data is validated or how errors are handled; it only cleans up code that was never referenced.

Security candidatefix(nordic/ble): fix stack overflow crashby tychovrahe · 9935c83c · Sep 22, 2025 · 2 filesMessage 62 · AdequateLow 42Details
Commit message · tychovrahe

fix(nordic/ble): fix stack overflow crash

[no changelog]

62/100 · AdequateMessage clarity
✓ 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 Bluetooth Low Energy (BLE) crash in Trezor hardware wallets caused by a stack overflow. The patch does two things: it stops placing the device advertising name on the thread stack by using a static buffer instead, and it increases the BLE management thread's stack size from the default to 2048 bytes. The change reduces the chance that a long or specially crafted device name could corrupt nearby memory or crash the BLE subsystem. There is no vendor statement that this is a security vulnerability, and no independent researcher is credited.

Security candidatefix(legacy): always use Gwei for ETH fee ratesby Roman Zeyde · a5fb5adb · Sep 21, 2025 · 2 filesMessage 72 · AdequateLow 39Details
Commit message · Roman Zeyde

fix(legacy): always use Gwei for ETH fee rates

Similar to how it's done in core.

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

This commit changes how the older Trezor One (legacy) firmware displays Ethereum transaction fees on the device screen. Previously, small per-gas fee values could be shown in 'Wei' (the smallest Ethereum unit) instead of always using 'Gwei'. The fix makes per-gas fees consistently display in Gwei, matching the newer Trezor Core firmware. This is a user-interface consistency fix that could have minor security relevance because confusing fee units might lead a user to misread or misapprove a transaction fee, but the commit itself does not change any signing logic or cryptographic checks.

Lower-priorityfeat(core/eckhart): cache THP app/host name along with MACby Lukas Bielesch · a80a1d00 · Sep 21, 2025 · 7 filesMessage 77 · AdequateInformational 19Details
Commit message · Lukas Bielesch

feat(core/eckhart): cache THP app/host name along with MAC

[no changelog]

Co-authored-by: Martin Milata <martin@martinmilata.cz>

77/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
AI analysis · Informational 19/100

This commit is a small feature update for the Trezor hardware wallet's Bluetooth pairing system. It adds a new 'app_name' field to the cached information stored alongside each paired device's MAC address and host name. The change is purely additive and does not appear to fix or introduce any security vulnerability. It updates protocol definitions, device menu display helpers, pairing logic, and generated code in multiple languages, plus related tests.

Lower-prioritychore(tools): bump-version bumps VERSION filesby obrusvit · 97c6e141 · Sep 21, 2025 · 1 fileMessage 80 · StrongInformational 15Details
Commit message · obrusvit

chore(tools): bump-version bumps VERSION files

- modify the tool to also work with VERSION files which are not
C-headers. Used e.g. in nordic/trezor/trezor-ble project.

[no changelog]

80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This is a routine developer tooling change. It extends an internal version-bumping script so it can also update plain-text VERSION files (used by a new Bluetooth project), in addition to the existing C header version files. There is no user-facing or security-relevant change to the firmware itself.

Security candidatechore: bump versionsby obrusvit · 9be734f6 · Sep 21, 2025 · 4 filesMessage 55 · ThinInformational 15Details
Commit message · obrusvit

chore: bump versions

- prodtest
- secmon
- bootloader
- nordic/trezor/trezor-ble

[no changelog]

55/100 · ThinMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
update trustboot or update path
AI analysis · Informational 15/100

This commit only increases version numbers (patch levels) for four Trezor firmware components: the bootloader, production test tool, security monitor, and Bluetooth Low Energy module. There are no code logic changes, bug fixes, or security-related modifications in the diff itself.

Lower-priorityfix(core): make app_name required for THP pairingby Martin Milata · ed1063b7 · Sep 19, 2025 · 7 filesMessage 57 · ThinLow 37Details
Commit message · Martin Milata

fix(core): make app_name required for THP pairing

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

This commit tightens the rules for a Trezor pairing request by making both the host name and app name mandatory. Previously, the app name was optional, which could have led to a confusing or misleading pairing screen on the device. The change helps ensure users see complete, accurate information before approving a connection.

Lower-priorityfix(python): include hostname during THP pairingby Martin Milata · 29baee0d · Sep 19, 2025 · 3 filesMessage 57 · ThinInformational 18Details
Commit message · Martin Milata

fix(python): include hostname during THP pairing

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

This is a small Python library fix for the Trezor hardware wallet's new Bluetooth-style pairing protocol (THP). It makes the client send the computer's real hostname and the application's name during pairing, instead of always sending a generic placeholder. This helps users see which app and computer they are pairing with on their Trezor screen. It is a usability and correctness improvement, not a security vulnerability fix.

Lower-prioritybuild: add secmon target to build-docker.shby obrusvit · cf1fa82a · Sep 18, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · obrusvit

build: add secmon target to build-docker.sh

[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 simply adds a new build target called 'secmon' to a Docker build script. It changes two lines in a help message and a default list of build targets. There is no indication this is a security fix or introduces a vulnerability.

Security candidatefix(core): don't request anim frames if text fitsby Ioan Bizău · f6d6787d · Sep 18, 2025 · 3 filesMessage 57 · ThinInformational 18Details
Commit message · Ioan Bizău

fix(core): don't request anim frames if text fits

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

This commit is a small UI cleanup for Trezor hardware wallets. It stops on-screen scrolling text (marquees) from requesting animation frames when the text already fits in its allotted space. This saves battery and avoids unnecessary screen redraws, but it does not fix a security vulnerability.

Security candidatechore(core/eckhart): overlap vertical menu and headerby Lukas Bielesch · ec21fb80 · Sep 18, 2025 · 4 filesMessage 77 · AdequateInformational 15Details
Commit message · Lukas Bielesch

chore(core/eckhart): overlap vertical menu and header

- Overlap the first button top padding with header when no subtitle is present
[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
boot or update path
AI analysis · Informational 15/100

This is a user-interface layout tweak for the Trezor hardware wallet's new 'Eckhart' design. It makes the first menu button visually overlap the screen header when there is no subtitle, and adds logic so the header is drawn on top if its buttons are being pressed. There is no security-relevant change here.

Security candidatechore(core/eckhart): do not disable partially hidden itemsby Lukas Bielesch · 4862cc9d · Sep 18, 2025 · 3 filesMessage 77 · AdequateLow 27Details
Commit message · Lukas Bielesch

chore(core/eckhart): do not disable partially hidden items

- rather limit their touch area not to overlap with the top component
[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
boot or update path
AI analysis · Low 27/100

This commit tweaks how menu buttons behave on the Trezor Safe 5 (Eckhart layout). Previously, any menu button that was not fully visible on screen was completely disabled, so you could not tap it. Now only buttons that are fully hidden are disabled; buttons that are partially visible remain active, but their touchable area is trimmed so it does not overlap the top header. This is a user-interface polish change, not a fix for a cryptographic or firmware security flaw. It could slightly reduce the chance of a user accidentally tapping a half-hidden button, but it does not introduce or remove a meaningful security vulnerability on its own.

Security candidatechore(core/eckhart): update overflow arrow positionby Lukas Bielesch · b304af9e · Sep 18, 2025 · 2 filesMessage 100 · StrongInformational 15Details
Commit message · Lukas Bielesch

chore(core/eckhart): update overflow arrow position

- add test so that the overflow arrow doesn't overlap the text
[no changelog]

100/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✓ Mentions testing or verification✓ Names security-relevant behavior explicitly
Why it was queued
memory safetyboot or update path
AI analysis · Informational 15/100

This commit is a minor visual polish change for the Trezor hardware wallet's user interface. It moves a small downward arrow icon slightly upward so it no longer overlaps menu text, and adds an automated test to ensure that spacing rule stays correct. There is no security-relevant change.

Security candidatefeat(core): Read Tropic certificates.by Andrew Kozlik · d30cc324 · Sep 18, 2025 · 4 filesMessage 57 · ThinInformational 17Details
Commit message · Andrew Kozlik

feat(core): Read Tropic certificates.

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

This commit adds a new feature to read pre-stored digital certificates from the Tropic secure chip inside Trezor hardware wallets. It does not change how keys are generated or signed, and it does not appear to fix a known bug or vulnerability. The new code is mostly about safely reading fixed-size data slots from the chip and exposing them to Python applications.

Security candidaterefactor(core): Move Tropic constants to tropic.h.by Andrew Kozlik · 37b0c368 · Sep 18, 2025 · 5 filesMessage 62 · AdequateInformational 15Details
Commit message · Andrew Kozlik

refactor(core): Move Tropic constants to tropic.h.

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

This is a routine code cleanup that moves Tropic chip constants (like which key slot is used for what) into a shared header file so multiple parts of the firmware can use the same definitions. It also updates the Python binding for signing to use the correct Ed25519 signature size and exposes two key-slot constants to Python. There is no security vulnerability here.

Security candidatefeat(core): Use correct pairing keys in Tropic handshake.by Andrew Kozlik · 41cd4bbf · Sep 18, 2025 · 2 filesMessage 62 · AdequateModerate 57Details
Commit message · Andrew Kozlik

feat(core): Use correct pairing keys in Tropic handshake.

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

This commit fixes how a Trezor hardware wallet selects cryptographic pairing keys when establishing a secure session with the Tropic secure chip. Previously the code always used a single fixed key slot (likely a factory/unprivileged key). After the change, it tries to use a privileged pairing key if available, and falls back to an unprivileged key if not. It also adds a small delay to avoid a 'chip busy' error from the Tropic chip. The change is security-relevant because using the wrong pairing key could weaken the secure channel or cause it to rely on a less trusted factory key, but the commit itself does not describe an active vulnerability or attack.

Security candidatefeat(python): Check Tropic fields to AuthenticityProof.by Andrew Kozlik · bb1d21b5 · Sep 18, 2025 · 3 filesMessage 62 · AdequateLow 26Details
Commit message · Andrew Kozlik

feat(python): Check Tropic fields to AuthenticityProof.

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
authentication path
AI analysis · Low 26/100

This commit updates the Trezor Python library's device-authentication code to support a new hardware security chip (Tropic) in addition to the existing Optiga chip. It adds Ed25519 signature verification, new root public keys for the T3W1 device, and a check that both chips' certificate chains are issued by the same root authority. The change is a feature addition rather than a fix for a known vulnerability, but it strengthens the authentication process by making it harder for a device with a valid certificate from one root to pass authentication using a mismatched certificate from another root.

Lower-priorityfeat(core): Add tropic_data_read().by Andrew Kozlik · f436a808 · Sep 18, 2025 · 12 filesMessage 47 · ThinLow 27Details
Commit message · Andrew Kozlik

feat(core): Add tropic_data_read().

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

This commit adds a new secure hardware feature that lets the device read small protected data slots from a Tropic security chip. It is a feature addition, not a bug fix. The code includes the usual memory-access checks, so there is no obvious security flaw in the diff itself. However, it expands the attack surface by exposing another privileged operation to less-trusted firmware, and the review is limited because the underlying Tropic library code is not shown.

Lower-priorityfix(core): Fix dysfunctional utils.USE_TROPIC.by Andrew Kozlik · cc4b24fa · Sep 18, 2025 · 3 filesMessage 57 · ThinInformational 12Details
Commit message · Andrew Kozlik

fix(core): Fix dysfunctional utils.USE_TROPIC.

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

This commit fixes a build configuration bug for the Trezor T3W1 hardware wallet. The Tropic secure-element feature was being compiled in, but the feature flag that tells the rest of the firmware to actually use it was not being registered. The change adds 'tropic' to the list of available features so the firmware can correctly enable Tropic support. There is no direct evidence in the commit of a security vulnerability or exploit.

Security candidatefeat(core): Use Tropic in AuthenticateDevice.by Andrew Kozlik · 1940bd56 · Sep 18, 2025 · 3 filesMessage 57 · ThinLow 26Details
Commit message · Andrew Kozlik

feat(core): Use Tropic in AuthenticateDevice.

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
authentication path
AI analysis · Low 26/100

This commit updates the Trezor hardware wallet's device-authentication feature so it can optionally use a second secure chip (Tropic) in addition to the existing Optiga chip. It refactors certificate parsing into a helper and renames the proof fields from generic names to optiga_* and tropic_*. There is no direct evidence in the commit that this fixes a security vulnerability; it appears to be a feature addition enabling a new hardware variant.

Lower-priorityfeat(core): Optimize and generalize tropic_data_multi_read().by Andrew Kozlik · 8f894185 · Sep 18, 2025 · 1 fileMessage 62 · AdequateLow 41Details
Commit message · Andrew Kozlik

feat(core): Optimize and generalize tropic_data_multi_read().

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

This commit rewrites a function that reads data stored across multiple secure memory slots on Trezor's Tropic secure element. The rewrite reduces memory usage and avoids reading unused slots, but it also changes how the code checks slot boundaries and how it validates that the declared data length actually fits within the requested slot range. A subtle bug in the new code could allow reading more slots than intended or mishandling short terminal slots, though the commit appears aimed at hardening rather than introducing a vulnerability.

Lower-prioritychore(core): Add Tropic fields to AuthenticityProof.by Andrew Kozlik · 24659231 · Sep 18, 2025 · 4 filesMessage 62 · AdequateInformational 18Details
Commit message · Andrew Kozlik

chore(core): Add Tropic fields to AuthenticityProof.

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

This commit renames existing fields in a device-authentication proof message and adds optional fields for a second security chip (Tropic). It is a schema-only change across generated protobuf bindings; there is no new executable logic shown. By itself it does not create a vulnerability, but it changes the public API used to verify whether a Trezor device is genuine.

Lower-priorityrefactor(core): improve checksum handling in THP [no changelog]by M1nd3r · 1de3b2ba · Sep 18, 2025 · 2 filesMessage 77 · AdequateInformational 12Details
Commit message · M1nd3r

refactor(core): improve checksum handling in THP
[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
AI analysis · Informational 12/100

This commit is a straightforward code cleanup in the Trezor firmware's THP (Trezor Host Protocol) layer. It renames a function and changes how CRC checksums are computed, but does not appear to fix or introduce a security vulnerability. The behavior remains functionally equivalent: invalid checksums are still rejected, and valid payloads are still accepted.

Lower-prioritychore(core): increase retransmission timeout [no changelog]by M1nd3r · 462e435c · Sep 18, 2025 · 1 fileMessage 77 · AdequateInformational 15Details
Commit message · M1nd3r

chore(core): increase retransmission timeout
[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
AI analysis · Informational 15/100

This commit simply doubles the initial wait time between retries when sending data over the Trezor's THP (Trezor Host Protocol) channel and slightly adjusts the maximum timeout. It is a routine tuning change with no visible security implications.

Security candidatechore(core): update T3W1 embedded binariesby tychovrahe · ea3d5066 · Sep 18, 2025 · 3 filesMessage 57 · ThinInformational 11Details
Commit message · tychovrahe

chore(core): update T3W1 embedded binaries

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
boot or update path
AI analysis · Informational 11/100

This commit swaps out two pre-built binary firmware files for the T3W1 hardware model and updates the matching cryptographic hash values in a header file. The change is described as a routine chore with no changelog entry. There is no visible source code change, no security explanation, and no disclosed vulnerability.