TZ
← All projectsTrezor

Trezor firmware

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

BitcoinHardware walletsNormal
Repository coverage

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

919security candidates311second-pass queue2658AI analyses
332commits · 30 days
573commits · 60 days
1366commits · 180 days
2650commits · 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
476Strong · 80–100
1321Adequate · 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 Zeyde575179573271
PrisionMike945994273
Andrew Kozlik712470265
obrusvit21376213164
Jakub Janků22919182
Martin Pastyřík23823173
cepetr19682196059
Ioan Bizău23076230059
M1nd3r20772207071
Lukas Bielesch846784067
Martin Milata17446174063
Analysis record

Published AI watches

Last scanned 33 minutes ago

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
Informational 15 AI analysisMessage 47 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

docs: add xtask documentation

This commit only adds documentation. It updates the table of contents and adds a new markdown page plus a diagram explaining the project's 'xtask' build helper. No code, build scripts, or firmware logic was changed, so it cannot introduce …

4879dababy cepetr+296−03 files
No security note in commit
Informational 17 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

feat(xtask): introduce project-toml build-options

This commit is a build-system refactoring for Trezor firmware. It moves the mapping of command-line build options to Rust/cargo features out of hard-coded Rust logic and into per-project TOML files. It also adds a small change so debug bui…

Build-option mapping now lives in project.toml files, increasing the attack surface for supply-chain/build-configuration tamperingMakefile change automatically enables debug-link for PYOPT=0 debug buildsValidation added to reject storage_insecure_testing_mode in production builds
d44de007by cepetr+681−40817 files
No security note in commit
Repository ledger

Explore captured commits

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

Lower-prioritybuild(core): warn when translation blob size become too highby Roman Zeyde · ed8ab863 · Mar 26, 2026 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

build(core): warn when translation blob size become too high

[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
translation-only discount
AI analysis · Informational 15/100

This commit adds a build-time warning to the Trezor firmware translation tool. It checks how large each language translation file is compared to the maximum space available on each device model, and logs a yellow warning or red error if the file is getting too close to the limit. There is no change to device runtime code, no user-facing behavior change, and no security fix or vulnerability.

Lower-priorityrefactor(rust/trezor-thp): always validate control byteby Martin Milata · b48c35d3 · Mar 26, 2026 · 3 filesMessage 62 · AdequateLow 37Details
Commit message · Martin Milata

refactor(rust/trezor-thp): always validate control byte

[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
defensive validation
AI analysis · Low 37/100

This commit refactors how the Trezor hardware wallet's Rust transport protocol code validates the 'control byte' in incoming USB packets. Previously, the control byte was accepted from raw bytes without checking whether it was a known/valid value, and different parts of the code independently decided what the byte meant. Now, every incoming control byte is centrally validated against a list of allowed packet types before being used. The change also moves responsibility for handling some packet types (channel allocation and legacy codec v1) out of the channel layer into the multiplexer layer. This is a defensive hardening change that reduces the chance that a malformed or unexpected control byte could confuse the device or be misinterpreted as a different packet type.

Lower-priorityrefactor(rust/trezor-thp): error constructorsby Martin Milata · f590edcd · Mar 26, 2026 · 8 filesMessage 80 · StrongInformational 14Details
Commit message · Martin Milata

refactor(rust/trezor-thp): error constructors

You can set a breakpoint on these to find out where an error is raised.

Also remove Error::OutOfBounds.

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

This commit is a straightforward internal code cleanup in Trezor's Rust transport protocol library. It replaces direct creation of error values with calls to new helper functions (constructors) so developers can set breakpoints to trace where errors originate. It also removes one unused error variant (OutOfBounds) and maps those cases to MalformedData instead. There is no functional security change or bug fix visible in the diff.

Lower-priorityci(rust): propagate unit test failureby Martin Milata · fb1307a6 · Mar 26, 2026 · 1 fileMessage 67 · AdequateInformational 15Details
Commit message · Martin Milata

ci(rust): propagate unit test failure

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

This commit fixes a CI build script so that Rust test, style-check, and lint failures are properly reported. It does not change any firmware code or fix a security vulnerability in the product itself.

Lower-priorityrefactor(rust/trezor-thp): add use_std featureby Martin Milata · 7796bc64 · Mar 26, 2026 · 5 filesMessage 72 · AdequateInformational 15Details
Commit message · Martin Milata

refactor(rust/trezor-thp): add use_std feature

Provide optional buffer handling using std::vec::Vec.

[no changelog]

72/100 · AdequateMessage clarity
✓ 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 is a routine code refactor for the Trezor hardware wallet firmware. It adds an optional 'use_std' feature to a Rust library so it can use standard Rust vector-based buffers when running on a full computer (for example, in test/example code), while keeping the no-standard-library mode for the embedded device. There is no indication of a security fix or vulnerability.

Lower-priorityrefactor(rust/trezor-thp): more detailed PacketInResultby Martin Milata · 91c90f8e · Mar 26, 2026 · 4 filesMessage 62 · AdequateLow 27Details
Commit message · Martin Milata

refactor(rust/trezor-thp): more detailed PacketInResult

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

This commit refactors how a Trezor hardware wallet's Rust transport layer reports the outcome of receiving a network or USB packet. It replaces a simple success/failure result with a richer set of statuses (accepted, buffer too small, peer error, ignored, failed, route to another channel, etc.) and changes several error-handling paths so that malformed or unexpected packets are now explicitly marked as ignored or failed rather than silently treated as 'nothing happened.' The commit is labeled a refactor with no changelog and no security disclosure. The changes appear to be defensive hardening, but because the patch is partial and the surrounding protocol code is not fully shown, it is not possible to confirm whether it fixes an exploitable vulnerability or merely improves robustness.

Lower-priorityrefactor(rust/trezor-thp): separate handling for host-side broadcast messagesby Martin Milata · fa45095d · Mar 26, 2026 · 9 filesMessage 62 · AdequateInformational 11Details
Commit message · Martin Milata

refactor(rust/trezor-thp): separate handling for host-side broadcast messages

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

This commit is a code refactor in Trezor's experimental THP (Trezor Host Protocol) Rust library. It splits the handling of broadcast-channel messages (channel allocation and ping/pong) into a new 'Mux' component, separate from the main encrypted channel handshake. There is no indication in the commit that this fixes a security bug; it appears to be an internal restructuring with added tests and cleaner state management.

Security candidatefeat(rust/trezor-thp): secure channel layer: device sideby Martin Milata · db56317e · Mar 26, 2026 · 14 filesMessage 62 · AdequateLow 28Details
Commit message · Martin Milata

feat(rust/trezor-thp): secure channel layer: device side

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

This commit adds the device-side implementation of a new secure communication layer (THP) for Trezor hardware wallets. It introduces code that handles encrypted channels, pairing handshakes, and legacy protocol fallback. The change is a feature addition rather than a documented security fix, and the diff itself does not show an exploitable vulnerability. However, it is a large, security-critical change to firmware that will protect future host-device communication.

Lower-priorityfix(tests): Fix Evolu nightly testsby Martin Pastyřík · 80a023ae · Mar 26, 2026 · 2 filesMessage 94 · StrongInformational 15Details
Commit message · Martin Pastyřík

fix(tests): Fix Evolu nightly tests

Compute the proofs of delegated identity dynamically so that it does not fail in hardware tests

[no changelog]

94/100 · StrongMessage clarity
✓ 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
AI analysis · Informational 15/100

This commit only changes automated test code for the Evolu feature. It replaces hard-coded cryptographic proof values with dynamically generated ones so the tests pass on real hardware. There is no change to the actual Trezor firmware or any user-facing security behavior.

Lower-prioritychore: bump prodtest version to 0.3.8by obrusvit · 7733d73d · Mar 25, 2026 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · obrusvit

chore: bump prodtest version to 0.3.8

[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 is a routine version number bump for an internal production-testing tool (prodtest), changing only the patch version from 0.3.7 to 0.3.8. There is no functional code change and no security relevance visible in the diff or commit message.

Lower-prioritydocs(core/prodtest): changelog for 0.3.7by obrusvit · a4fcf345 · Mar 25, 2026 · 3 filesMessage 72 · AdequateInformational 15Details
Commit message · obrusvit

docs(core/prodtest): changelog for 0.3.7

[no changelog]

(cherry picked from commit 5f4a38fe35bf9bdad01472e83772b7b242fb86e3)

72/100 · AdequateMessage clarity
✓ 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 is purely a documentation update for the Trezor hardware wallet's production-test firmware. It moves two changelog entries from draft files into the published CHANGELOG.md for version 0.3.7. No code was changed, and there is nothing here that affects device security or user funds.

AI review queuedfeat(core): Update RFAL library to version 4.0.2by kopecdav · 151ff869 · Mar 25, 2026 · 55 filesMessage 57 · ThinModerate 59Details
Commit message · kopecdav

feat(core): Update RFAL library to version 4.0.2

[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
second-pass: unusually broad change
AI analysis · Moderate 59/100

This commit updates the third-party STMicroelectronics RFAL NFC library inside Trezor firmware from version 3.0.1 to 4.0.2. The vendor's release notes for version 4.0.2 explicitly mention fixing a 'vulnerability during EMD handling on fast systems' that was introduced in version 4.0.0. EMD (Electro-Magnetic Disturbance) handling is part of the NFC radio stack, so the flaw is likely a low-level radio/communication issue rather than something that directly exposes cryptocurrency private keys. Because the commit is a large library drop (almost 8,000 lines changed across 55 files) and the actual code diff is not shown, we cannot independently verify the exact nature or severity of the vulnerability from this commit alone.

Lower-priorityfeat(core): LOCAL_CHANGES.md update.by kopecdav · 13c40c8d · Mar 25, 2026 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · kopecdav

feat(core): LOCAL_CHANGES.md update.

[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
documentation-only discount
AI analysis · Informational 15/100

This commit only adds a documentation note inside a markdown file. It records that the RFAL NFC middleware was updated from version 3.0.1 to 4.0.2. No code, build scripts, or configuration files were changed.

Security candidatefix(core): correct a typo in recovery-related layout function nameby Roman Zeyde · 8ef2caeb · Mar 25, 2026 · 5 filesMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

fix(core): correct a typo in recovery-related layout function name

[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 commit fixes a spelling mistake in a function name used during wallet recovery on Trezor hardware wallets. The old misspelled name 'show_group_thresholod' is replaced with the correct 'show_group_threshold' across all device layouts. There is no security issue here—just a typo cleanup.

Lower-priorityrefactor(core): refactor backup flow to support more backup methodsby Roman Zeyde · a892407d · Mar 25, 2026 · 1 fileMessage 85 · StrongInformational 12Details
Commit message · Roman Zeyde

refactor(core): refactor backup flow to support more backup methods

`display_mnemonics()` can be now replaced by a factory function which
would allow using other backup methods.

[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
AI analysis · Informational 12/100

This commit is a straightforward internal code cleanup in the Trezor hardware wallet's backup process. It reorganizes how backup words are displayed and confirmed, without changing what the user sees or the security checks performed. There is no indication this fixes a security bug or introduces a vulnerability.

Security candidatefeat(core): Evolu key rotationby Martin Pastyřík · 8807fec0 · Mar 24, 2026 · 57 filesMessage 72 · AdequateLow 37Details
Commit message · Martin Pastyřík

feat(core): Evolu key rotation

Add an index to the generation of Evolu keys and Delegated identity key.

[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
signing or wallet pathboot or update path
AI analysis · Low 37/100

This commit adds a key-rotation feature for the 'Evolu' / 'Suite Sync' delegated identity key used by Trezor. It introduces a rotation index that is mixed into the key derivation, lets users rotate the key on demand, and stores the current index in device storage. The change is a feature addition rather than a fix for an obvious vulnerability, but it touches sensitive key-derivation code and adds new protocol messages.

Security candidatefeat(ethereum): clear signing some swapsby Ioan Bizău · bb54d580 · Mar 24, 2026 · 2 filesMessage 72 · AdequateInformational 23Details
Commit message · Ioan Bizău

feat(ethereum): clear signing some swaps

Add 1inch, LiFi and Uniswap swaps from the clear signing registry to our
definitions.

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
signing boundarysigning or wallet path
AI analysis · Informational 23/100

This commit adds human-readable labels for certain cryptocurrency swap transactions on Trezor hardware wallets. It does not change how transactions are validated or signed; it only improves what the user sees on the device screen. There is no direct security vulnerability in the change itself, but any mistake in the labels could mislead a user about what a transaction does.

Security candidaterefactor: prepare for ERC-7730 clear signingby Ioan Bizău · 0d1d8d12 · Mar 24, 2026 · 7 filesMessage 57 · ThinLow 34Details
Commit message · Ioan Bizău

refactor: prepare for ERC-7730 clear signing

[no changelog]

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

This commit is a code refactor that reorganizes how Trezor handles Ethereum transaction display logic. It splits existing code into new modules for 'clear signing' and staking, and changes how token amounts and addresses are formatted before being shown to the user. There is no direct evidence in the commit that this fixes a security vulnerability; it appears to be preparation for supporting a new Ethereum standard (ERC-7730). However, because it touches the code that decides what users see on screen before signing transactions, any bug introduced here could affect how clearly users understand what they are approving.

Security candidaterefactor: move clear signing UI to layoutsby Ioan Bizău · 205c094a · Mar 24, 2026 · 6 filesMessage 72 · AdequateInformational 18Details
Commit message · Ioan Bizău

refactor: move clear signing UI to layouts

* also include the summary screen

[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
signing boundarycryptography-sensitive pathsigning or wallet path
AI analysis · Informational 18/100

This commit is a code cleanup that moves the on-screen confirmation flow for Ethereum 'clear signing' into the device's UI layout modules. It also adds a final summary screen showing the maximum transaction fee. There is no direct evidence of a security vulnerability being fixed; it appears to be a user-interface refactor. However, adding the fee summary screen is a security-relevant improvement because it gives users more complete information before they approve a transaction.

AI review queuedchore(ethereum): clear sign only first data chunkby Ioan Bizău · 73eaef8c · Mar 24, 2026 · 3 filesMessage 57 · ThinLow 40Details
Commit message · Ioan Bizău

chore(ethereum): clear sign only first data chunk

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 40/100

This commit changes how Trezor hardware wallets display and confirm Ethereum transactions that use 'clear signing' (human-readable descriptions of smart-contract calls). Previously, clear signing could be attempted for transactions whose data is split across multiple chunks. Now, clear signing is used only when the entire transaction data fits in the first chunk. If the clear-signing parser fails on that first chunk, the device falls back to the standard raw-data confirmation. The change is described as a feature limitation, not a security fix, but it removes a code path where multi-chunk data might be parsed before all chunks are available or confirmed.

AI review queuedfeat(ethereum): token amount native currencyby Ioan Bizău · d1760fc8 · Mar 24, 2026 · 1 fileMessage 57 · ThinInformational 16Details
Commit message · Ioan Bizău

feat(ethereum): token amount native currency

[no changelog]

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

This commit changes how the Trezor hardware wallet formats token amounts shown on screen during Ethereum transactions. It adds support for treating certain addresses as the network's native currency (like ETH on Ethereum mainnet) rather than as ERC-20 tokens when displaying amounts. There is no indication in the commit that this fixes a security vulnerability; it appears to be a user-interface correctness feature.

AI review queuedchore: drop 1inch definitions for nowby Ioan Bizău · dc1f3a1b · Mar 24, 2026 · 3 filesMessage 57 · ThinInformational 17Details
Commit message · Ioan Bizău

chore: drop 1inch definitions for now

[no changelog]

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

This commit removes built-in 'clear signing' support for the 1inch decentralized exchange from Trezor Ethereum firmware. Clear signing makes complex crypto transactions easier to read and verify on the device screen. Removing it does not create a new security hole, but users interacting with 1inch will now see less human-readable transaction details, which can make it harder to spot malicious or mistaken transactions. It is a feature rollback, not a vulnerability fix.

AI review queuedchore: enable SLIP-24 for ERC-20 `transfer`by Ioan Bizău · 795d5d6d · Mar 24, 2026 · 2 filesMessage 57 · ThinLow 30Details
Commit message · Ioan Bizău

chore: enable SLIP-24 for ERC-20 `transfer`

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 30/100

This commit enables Trezor hardware wallets to use SLIP-24 payment requests when signing ERC-20 token transfers. Previously, payment requests were rejected for any contract interaction, including ERC-20 transfers. The change adds a new code path that verifies the token transfer amount and recipient against a signed payment request before showing the confirmation screen. It is a feature-enablement change rather than a fix for an active vulnerability, but it touches security-critical signing flow code.

AI review queuedchore(tests): test ETH calldata parsingby Ioan Bizău · e3253010 · Mar 24, 2026 · 1 fileMessage 67 · AdequateInformational 12Details
Commit message · Ioan Bizău

chore(tests): test ETH calldata parsing

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

This commit only adds new automated tests for Ethereum transaction data parsing. It does not change any production code, so it cannot introduce a security vulnerability by itself. The tests verify that the device correctly rejects malformed Ethereum addresses and oversized numeric values.

AI review queuedchore: process up to 4kb of calldataby Ioan Bizău · d4609489 · Mar 24, 2026 · 1 fileMessage 57 · ThinLow 35Details
Commit message · Ioan Bizău

chore: process up to 4kb of calldata

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 35/100

This commit changes how Trezor firmware handles Ethereum transaction data (calldata). Previously, the code only tried to 'clear sign' transactions where the entire calldata fit in the first chunk. Now it stores and processes up to 4 KB of calldata across multiple chunks, and gives up gracefully if the data is too large. The change appears to be a hardening/capacity improvement rather than a fix for an active exploit, but it removes a risky assumption that all data is present in the initial chunk.