Every captured commit receives deterministic security triage and a separate communication-quality score. Security candidates and broader second-pass signals receive full-patch Ollama analysis.
Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.
This commit removes the SHA-1 hashing function from the regular Trezor firmware. SHA-1 is an old, weak hash algorithm that is no longer considered secure for sensitive uses. The change deletes the code that exposes SHA-1 to apps running on…
Removal of a deprecated cryptographic primitive (SHA-1) from the firmware API surfaceReduction of attack surface and prevention of future misuse of a collision-vulnerable hashNo direct vulnerability patch or memory-safety bug is present in the diff
This commit changes how a cryptographic library sets up secure connections. Previously, the code automatically calculated the public key from the private key. Now, the caller must provide the public key directly. This is a code-quality ref…
Cryptographic key handling changed: public key is now supplied rather than derivedPotential reduction of key-mismatch risk if caller provides correct public keyNew null-pointer check added for static_public_key
This commit simply reorders the arguments of an internal function called dh() and updates every place that calls it. The actual math and security behavior are unchanged; it is a code cleanup with no security effect.
This commit simply renames two groups of internal status labels (called enums) in the Trezor firmware's cryptographic code. The old names like WAITING_FOR_REQUEST1 were shared between two different parts of the code, so the developer gave …
This commit fixes three places in the Trezor firmware's cryptographic code where a memory-clearing function could be called with a NULL pointer. In practice, passing NULL to memzero is harmless on Trezor's platform (it does nothing), but i…
NULL pointer passed to memory-zeroing helper in cryptographic codeUndefined behavior in C standard library contractDefensive hardening in Noise protocol implementation
This commit is a code cleanup (refactor) for the cryptographic handshake code used in Trezor devices. It changes how the other party's long-term public key is returned to the caller: instead of storing it inside an internal state structure…
Removal of long-term public key storage from internal handshake stateCaller-supplied output buffer for remote static public key reduces internal secret retentionError-path memzero of returned key material on failure
This commit is a build-system refactoring for Trezor firmware. It moves the mapping of command-line build options to Rust/cargo features out of hard-coded Rust logic and into per-project TOML files. It also adds a small change so debug bui…
Build-option mapping now lives in project.toml files, increasing the attack surface for supply-chain/build-configuration tamperingMakefile change automatically enables debug-link for PYOPT=0 debug buildsValidation added to reject storage_insecure_testing_mode in production builds
This commit only updates test data. It corrects the capitalization (checksum) of Ethereum addresses used in automated tests and refreshes the expected screen-shot hashes those tests compare against. There are no changes to the actual Trezo…
This commit changes the Trezor Python library so that, when setting up a Trezor Model One (the original Trezor 1 device), it now performs an entropy check by default if the device runs firmware 1.13.1 or newer. Previously, the library only…
Enables a previously disabled security/validation feature (entropy check) for a specific device modelAdds version-gated behavior to avoid errors on older firmwareDefensive hardening of wallet setup randomness verification
This commit is a routine cleanup that removes the old SCons build system files from the Trezor Core firmware repository. It deletes Makefiles, SConscripts, and related Python helper scripts, but does not change any firmware source code, cr…
This commit updates Trezor's embedded MicroPython interpreter from an older version to 1.28.0. It pulls in several upstream MicroPython bug fixes, including stricter buffer-size checks for converting integers to bytes, a new stack-safety A…
Synchronizes upstream MicroPython fixes that include buffer-size and stack-safety hardeningPrevents Ctrl+C interruption of frozen boot code, reducing denial-of-service/control-flow risk during bootFixes sys.stdout.buffer.write() return value, which could affect code relying on correct I/O semantics
This commit is a simple code cleanup that replaces a common MicroPython string-creation pattern with a new helper function. It does not change what the code does, only how it is written. There is no security issue visible in the change.
This is a large but straightforward internal cleanup: the project switched from using the MicroPython `ubinascii` module's `hexlify`/`unhexlify` functions to the standard Python `bytes.hex()` and `bytes.fromhex()` methods. The change remov…
This commit is a routine code cleanup in the Trezor firmware's embedded MicroPython modules. It replaces an older, two-step object allocation pattern with a newer helper that allocates memory and sets the object type in one step. The chang…
No security-relevant logic changes observedNo input validation changesNo memory safety bug fixes (e.g., no overflow, use-after-free, or uninitialized memory fixes)
This commit is a routine maintenance refactor. MicroPython removed its built-in 'utimeq' module, so Trezor copied the same code into its own firmware tree and renamed it to 'timeq'. There is no security bug being fixed and no new vulnerabi…
This is a code cleanup change in Trezor's firmware that swaps one MicroPython internal helper for another. It replaces calls that create byte or string objects from a vstr buffer with newer, purpose-built helpers. The commit message says t…
Refactor only: helper function renames with equivalent semanticsNew str helper adds UTF-8 validation; bytes helper does not validateNo input validation, length, or error-handling changes observed
This commit is a routine internal cleanup that switches how Trezor's firmware defines built-in MicroPython object types. It replaces older, manually-written type structures with a newer macro provided by the upstream MicroPython project. T…
This commit is a straightforward internal code refactor. It adds a small C wrapper file so that Rust code can call two specific internal MicroPython functions that are declared as static/inline in headers. There is no user-facing change, n…
This commit is a routine build-system update for the Trezor firmware. It changes where the build process looks for the MicroPython cross-compiler binary (mpy-cross) and synchronizes a configuration header with newer MicroPython code. There…
This commit updates the Trezor firmware build system to use a newer MicroPython mechanism for tracking special memory pointers called 'root pointers.' It removes hard-coded lists of these pointers from configuration files and instead colle…
GC root pointer handling changedBuild system now auto-collects root pointers instead of hard-coding themRemoves manual MICROPY_PORT_ROOT_POINTERS lists
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100
This commit fixes a spelling mistake in the legal license text at the top of six source files. The typo 'HMAC_SHALL' is corrected to 'SHALL'. It does not change any program logic, security behavior, or compiled code.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100
This commit only adds new automated tests for an existing cryptographic handshake protocol called noise_xxpsk3. It does not change the actual security code, fix a bug, or introduce a vulnerability. The tests verify that the protocol works correctly, that tampered messages are rejected, and that keys are wiped after use.
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 7/100
This commit adds a brand-new cryptographic module implementing the Noise_XXpsk3_25519_AESGCM_SHA256 protocol for secure encrypted communication. It is a feature addition with no changelog entry and no accompanying security advisory. The code itself is a clean implementation of a well-known cryptographic handshake, but because it is new code handling private keys, pre-shared keys, and AES-GCM encryption, it is security-relevant by nature. There is no direct evidence in the commit or supplied references that this fixes a known vulnerability or introduces a bug.
Security candidatefix(crypto): minor fixes in `noise_kk1`by M1nd3r · 1b60dbbf · Jul 3, 2026 · 1 fileMessage 80 · StrongLow 31Details
Commit message · M1nd3r
fix(crypto): minor fixes in `noise_kk1`
- Mark `split` as static in `noise_kk1`. - Add missing memzero to `noise_kk1`. - Return false on encryption failure in `noise_kk1`. - Prevent using memcpy with NULL in noise. - Using `memcpy` to copy data from NULL buffer results in potentially undefined behavior. This raises a runtime error when using UB sanitizer.
[no changelog]
80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
defensive validationcryptography-sensitive path
AI analysis · Low 31/100
This commit fixes several minor issues in a cryptographic module used for secure device communication. The changes include preventing undefined behavior when copying from NULL pointers, zeroing out sensitive key material after use, and properly returning errors when encryption fails. These are defensive hardening fixes rather than a clear, exploitable vulnerability.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100
This commit is a simple rename of the cryptographic module from 'noise' to 'noise_kk1' across file names, function names, constants, and build scripts. No security behavior changes are visible in the diff. It is a code-maintenance refactor, not a security fix.
where some other completely unrelated package provides a "version.h" file that happens to come first on the include path.
"version.h" is too generic a name to generally rely on C's imperfect header resolution; a redirect via a symlink seems to be the most correct solution
80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
boot or update path
AI analysis · Informational 17/100
This commit fixes a build problem in Trezor's firmware. The code used a very common filename, version.h, which could accidentally pull in the wrong file from another software package during compilation. The fix creates a symlink with a more specific name, firmware_version.h, and uses that instead. There is no direct evidence this is a security vulnerability; it appears to be a build reliability fix.
build(core): avoid unused variable warning in rng_fill_buffer_strong
When neither USE_OPTIGA nor USE_TROPIC, rng_fill_buffer_strong is just rng_fill_buffer with extra steps. Make that explicit in code; otherwise the `dst` variable is never read from and we get a warning under clang.
97/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
entropy or randomnessseed or entropy path
AI analysis · Informational 15/100
This is a minor build cleanup. A developer added a compile-time branch so that when neither the Optiga nor Tropic security chips are used, the 'strong' random-number wrapper simply calls the normal random-number function and returns. The only reason for the change is to silence a compiler warning about an unused variable. There is no functional security change.
Security candidatefeat(ethereum): allow approve and transfer to send ethby PrisionMike · e785f6c6 · Jul 2, 2026 · 8 filesMessage 62 · AdequateModerate 64Details
Commit message · PrisionMike
feat(ethereum): allow approve and transfer to send eth
[no changelog]
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
cryptography-sensitive pathsigning or wallet path
AI analysis · Moderate 64/100
This commit changes how Trezor devices display Ethereum transactions when someone sends ERC-20 token approval or transfer calls that also include native ETH. Previously, the device might not clearly show the attached ETH, so a user could unknowingly approve sending tokens plus real ETH. The patch makes the device surface that native ETH amount on the confirmation screen. It is a defensive fix that improves transparency rather than introducing a vulnerability.
feat(clear_signing, ethereum) add new provider names.
[no changelog]
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing boundarysigning or wallet path
AI analysis · Informational 15/100
This commit is a routine feature update for Trezor's Ethereum 'clear signing' feature. It adds human-readable names for more DeFi and staking providers (like Lido, Morpho, Kiln) and makes the address lookup aware of which blockchain network (chain ID) an address belongs to. There is no security vulnerability or bug fix here.
feat(core): add chunkify support for Solana SignTx
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
cryptography-sensitive pathsigning or wallet path
AI analysis · Informational 18/100
This commit adds an optional user-interface feature for Solana transactions on Trezor hardware wallets. When enabled, long Solana addresses are shown on the device screen in smaller groups of characters (chunks of 4) instead of one long string, making them easier for users to read and verify. There is no indication this change fixes a security vulnerability; it is a usability improvement.
AI review queuedfeat(core): sign rotation index into Evolu sign registrationby Martin Pastyřík · 4db3a69d · Jul 1, 2026 · 8 filesMessage 77 · AdequateInformational 24Details
Commit message · Martin Pastyřík
feat(core): sign rotation index into Evolu sign registration
This enables the Quota Manager to distinguish the indices and store the current index.
[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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 24/100
This commit updates the Trezor firmware's Evolu (delegated identity) registration signing feature so that the device now includes a 'rotation index' in the signed registration request. The rotation index tells the Quota Manager which version of the device's delegated identity key was used. The signature format was also bumped from V1 to V2 by adding the rotation index into the signed data. This is a feature enhancement, not a fix for an obvious security vulnerability, but it touches cryptographic signing and identity-key rotation, which are security-sensitive areas.
✓ 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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 17/100
This change moves some internal test-only Ethereum transaction display definitions from being permanently added to a global list to being generated on demand only when the device is running in debug/test mode. It is a memory-footprint and code-hygiene cleanup, not a fix for an exploitable security bug. There is no indication it addresses a vulnerability.
AI review queuedtest(core): drop `test_descriptor_all_formatters` unit testby Roman Zeyde · 3704e769 · Jul 1, 2026 · 1 fileMessage 87 · StrongInformational 15Details
Commit message · Roman Zeyde
test(core): drop `test_descriptor_all_formatters` unit test
It should be covered by device tests.
[no changelog]
87/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Mentions testing or verification
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit simply removes a single unit test from the Trezor firmware test suite. The test covered Ethereum 'clear signing' formatting logic, and the commit message says it is being dropped because it should be covered by device tests instead. No production code was changed, so this does not introduce or fix a security vulnerability by itself.
Security candidatefeat(clear_signing): add date and raw formatters. - core tests added. - proto updated. [no changelog]by PrisionMike · 600cf34e · Jun 30, 2026 · 7 filesMessage 87 · StrongInformational 21Details
Commit message · PrisionMike
feat(clear_signing): add date and raw formatters. - core tests added. - proto updated. [no changelog]
87/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Mentions testing or verification
Why it was queued
signing boundarysigning or wallet path
AI analysis · Informational 21/100
This commit adds two new ways to display Ethereum transaction details on a Trezor screen: a 'raw' formatter that shows values as-is (numbers, text, or hex bytes) and a 'date' formatter that turns a Unix timestamp into a readable date. It also wraps some existing parsing and display steps in try/except blocks so failures are logged and re-raised. There is no direct evidence this fixes an active security bug; it looks like a feature addition for clearer transaction signing.
Security candidatefeat(clear_signing): support multi-value (array) fieldsby PrisionMike · f9f6eb97 · Jun 30, 2026 · 2 filesMessage 85 · StrongInformational 20Details
Commit message · PrisionMike
feat(clear_signing): support multi-value (array) fields
When a field's path resolves to an array, the formatter is applied to each element and the rendered values are joined with newlines (one value per line). Works for any formatter pointed at an array. Also renders booleans in RawFormatter (bool is not an int subclass under MicroPython).
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
signing boundarysigning or wallet path
AI analysis · Informational 20/100
This commit adds support in Trezor's Ethereum 'clear signing' feature for displaying array (multi-value) fields on the device screen. It also fixes a minor display quirk where booleans were being treated as numbers. There is no direct evidence in the commit that this fixes an active security vulnerability; it reads as a feature improvement with some defensive hardening against malformed display definitions.
Security candidatefeat(clear_signing): identify a tokenAmount token by a constant addressby PrisionMike · c5aa2851 · Jun 30, 2026 · 5 filesMessage 85 · StrongInformational 19Details
Commit message · PrisionMike
feat(clear_signing): identify a tokenAmount token by a constant address
Adds an additive proto field (EthereumERC7730FieldInfo.const_token_address) and lets TokenAmountFormatter resolve its token from that literal address instead of a token_path - for descriptors whose token is a `$.metadata.constants.*` reference rather than a calldata parameter.
[no changelog]
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
signing boundarysigning or wallet path
AI analysis · Informational 19/100
This commit adds a new optional field to Trezor's Ethereum clear-signing protocol that lets a token amount be tied to a fixed contract address rather than pulled from transaction data. It is a feature addition, not a bug fix, and there is no evidence in the commit that it addresses a security flaw. The change does not appear to introduce an obvious vulnerability, but it slightly expands the attack surface by adding another way token addresses are resolved.
Security candidatetest(clear_signing): add Trezor Test descriptor and clear-signing testsby PrisionMike · c11f7477 · Jun 30, 2026 · 6 filesMessage 87 · StrongInformational 15Details
Commit message · PrisionMike
test(clear_signing): add Trezor Test descriptor and clear-signing tests
- Add test descriptors. - core tests.
[no changelog]
87/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Mentions testing or verification
Why it was queued
signing boundarysigning or wallet path
AI analysis · Informational 15/100
This commit only adds test code for Trezor's Ethereum clear-signing feature. It introduces debug-only test descriptors, unit tests, device tests, and expected screen snapshots. There is no change to production behavior or any fix for a security issue.
Security candidatefix(core/caesar): fix `confirm_properties()` handling of `PageMsg::Info`by Roman Zeyde · 04621573 · Jun 30, 2026 · 1 fileMessage 62 · AdequateLow 33Details
Commit message · Roman Zeyde
fix(core/caesar): fix `confirm_properties()` handling of `PageMsg::Info`
[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 fixes a small Rust UI bug in Trezor's 'Caesar' firmware layout. When a user viewed an 'info' page during a confirmation prompt, the device previously failed to return the expected 'INFO' response and instead threw a type error. The patch adds the missing case so the info page is handled correctly. This is likely a user-experience or defensive bug fix rather than a critical security vulnerability, but it could affect how device confirmations behave.
Security candidatetest(core/caesar): visit menu only on last `ButtonPage`by Roman Zeyde · a0fe1ecc · Jun 30, 2026 · 2 filesMessage 87 · StrongInformational 15Details
Commit message · Roman Zeyde
test(core/caesar): visit menu only on last `ButtonPage`
Non-last pages don't have an info button on Caesar.
[no changelog]
87/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Mentions testing or verification
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100
This is a minor test-only adjustment for the Caesar user interface layout. It changes a test helper so that it only tries to open the menu on the last page of a multi-page screen, because earlier pages do not have a menu/info button. There is no change to production firmware behavior or security.
Security candidatechore(core): return to autolock menu after setting a new valueby Roman Zeyde · 62382b34 · Jun 30, 2026 · 3 filesMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde
chore(core): return to autolock menu after setting a new value
[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 15/100
This commit is a minor user-interface polish change. After a user changes the auto-lock timeout setting on a Trezor device, the device now returns to the dedicated Auto-Lock menu instead of the broader Security menu. It does not change any security policy, cryptographic behavior, or data handling. The accompanying test updates simply expect the new screen.
Security candidatefeat(tron): add support for `WithdrawBalanceContract`by PrisionMike · c687e153 · Jun 29, 2026 · 13 filesMessage 62 · AdequateInformational 12Details
Commit message · PrisionMike
feat(tron): add support for `WithdrawBalanceContract`
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
cryptography-sensitive pathsigning or wallet path
AI analysis · Informational 12/100
This commit adds support for a new Tron blockchain transaction type called WithdrawBalanceContract, which lets users claim their voting rewards. It also improves the on-device confirmation flow for claim transactions so the device warns and shows the owner address when the transaction is being signed on behalf of a different account. There is no indication of a security vulnerability in the change itself; it appears to be a normal feature addition with stronger user confirmation.
AI review queuedchore(tron): scaffolding for `WithdrawBalanceContract`by PrisionMike · 2b93024e · Jun 29, 2026 · 18 filesMessage 62 · AdequateInformational 15Details
Commit message · PrisionMike
chore(tron): scaffolding for `WithdrawBalanceContract`
[no changelog]
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit adds support for a new Tron blockchain operation called 'WithdrawBalanceContract' (claiming voting rewards). It is a feature addition: new message types, generated code, translations, and a test fixture. There is no indication of a security bug or vulnerability fix in the diff.
Security candidatechore(tron): use pubhash for address derivation.by PrisionMike · 4b204ec9 · Jun 29, 2026 · 3 filesMessage 95 · StrongInformational 16Details
Commit message · PrisionMike
chore(tron): use pubhash for address derivation.
- avoids calling private key in python for public key derivation. - removing a function that was only used in tests.
[no changelog]
95/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Names security-relevant behavior explicitly
Why it was queued
secret or key material
AI analysis · Informational 16/100
This commit changes how Trezor hardware wallets derive TRON addresses. Previously, the code loaded the private key into Python memory and computed the public key from it. Now it uses a precomputed public-key hash already available from the keychain. This is a defensive hardening change that reduces the chance of private-key exposure in the Python layer, but it does not by itself fix a known exploitable vulnerability.
Security candidatefix(core): keep vertical offset when reloading device menuby Roman Zeyde · f383055e · Jun 29, 2026 · 13 filesMessage 62 · AdequateInformational 18Details
Commit message · Roman Zeyde
fix(core): keep vertical offset when reloading device menu
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
cryptography-sensitive pathboot or update path
AI analysis · Informational 18/100
This commit fixes a visual glitch on the Trezor T3W1 device where the device menu would visibly jump or flicker when it refreshed. The fix preserves the menu's current scroll position across refreshes by passing an extra offset value through the UI API. There is no indication this change addresses a security vulnerability.
Security candidaterefactor(core/eckhart): simplify device menu handlingby Roman Zeyde · e6e514c9 · Jun 29, 2026 · 7 filesMessage 85 · StrongInformational 15Details
Commit message · Roman Zeyde
refactor(core/eckhart): simplify device menu handling
- use Qstrs as DeviceMenuResult variants - replace `result_arg` by an explicit `DeviceMenuMsg` variant - don't replace `DeviceMenuMsg::Close` by `CANCELLED` - unify parent menu ID & menu refresh logic
[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
boot or update path
AI analysis · Informational 15/100
This is a code cleanup (refactor) for the device menu on a specific Trezor hardware wallet model. It changes how menu actions are represented internally (from numbers to named strings) and simplifies how the menu decides which submenu to show next. There is no indication this fixes a security bug or introduces a vulnerability.