Roman Zeyde

Results 96 issues of Roman Zeyde

Following the benchmarking results here: https://satoshilabs.slack.com/archives/C078GRAK58U/p1762513026034009 - [ ] measure the benefit of dropping THP ACKs when signing `test_lots_of_inputs` tx on `PYOPT=1` build This change must be coordinated with @trezor/suite-connect...

code

Similar to the UI overlay for debug FW builds.

bootloader core
T3W1

Tested on dc7deec32e with the DEBUG level logging disabled: ```diff diff --git a/core/src/trezor/log.py b/core/src/trezor/log.py index 9d33cc4ecb..2b00c57eb6 100644 --- a/core/src/trezor/log.py +++ b/core/src/trezor/log.py @@ -14,7 +14,7 @@ if __debug__: from trezorlog import...

bitcoin
T3W1

**Describe the bug** Signing BTC transaction with 255 outputs OOMs on T3W1 - similar to https://github.com/trezor/trezor-firmware/issues/5472. **Firmware version and revision** ``` * d983edbdd8 Roman Zeyde: (HEAD -> main) Merge remote-tracking...

bug
T3W1

While debugging https://github.com/trezor/trezor-suite/issues/22878, @marekrjpolak has noticed that THP desync[^1] happens when we are cancelling "dry-run recovery" flow, but not other flows. We have added more debug logs, and discovered that...

bug
T3W1

**Describe the bug** https://github.com/trezor/trezor-firmware/actions/runs/19052439327/job/54415529756 100+ tests failed due to `UnexpectedMagicError`, all with the same packet: `3f047465737460016a145f1b1e112b5c0035bd0f6aad54f80737c240fa09800101980100a00100aa010154ca0113554e534146452c20444f204e4f5420555345` Errors ``` ERROR tests/device_tests/bitcoin/test_signtx_replacement.py::test_attack_fake_ext_input_amount - trezorlib.transport.thp.protocol_v1.UnexpectedMagicError: 3f047465737460016a145f1b1e112b5c0035bd0f6aad54f80737c240fa09800101980100a00100aa010154ca0113554e534146452c20444f204e4f5420555345 ERROR tests/device_tests/bitcoin/test_signtx_invalid_path.py::test_invalid_path_pass_forkid - trezorlib.transport.thp.protocol_v1.UnexpectedMagicError: 3f047465737460016a145f1b1e112b5c0035bd0f6aad54f80737c240fa09800101980100a00100aa010154ca0113554e534146452c20444f204e4f5420555345 ERROR tests/device_tests/bitcoin/test_getpublickey.py::test_get_public_node[Testnet-70617039-path4-tpubDDKn3FtHc74CaRrRbi1WFdJNaaenZkDWqq9NsEhcafnDZ4VuKeuLG2aKHm5SuwuLgAhRkkfHqcCxpnVNSrs5kJYZXwa6Ud431VnevzzzK3U]...

bug
hardware only
tests
T2T1

Following https://github.com/trezor/trezor-firmware/pull/6099

tests
code
T3W1

We require at least 85% coverage - currently T3B1 check is failing: ``` Module statements missing excluded coverage Total 21589 3359 2392 84% ``` [core-coverage-report-T3B1-err.zip](https://github.com/user-attachments/files/22736655/core-coverage-report-T3B1-err.zip) When it passed, the coverage...

ci
tests

Yeah, the issue with Caesar is, if there is no menu, we can't have a layout that has `BACK | CONTINUE | MENU` We need something else... which is `confirm_properties`,...

T2B1
T3B1

**Describe the bug** For example: https://data.trezor.io/dev/firmware/ui_report/18902278163/en_diff.html **Expected behavior** Per-model UI fixture page is OK: https://data.trezor.io/dev/firmware/ui_report/18902278163/T2T1-en-core_device_test-master_diff.html

bug
ci
tests