arno/trezor-firmware
1
0
Fork 0
mirror of https://github.com/trezor/trezor-firmware.git synced 2025-07-20 13:38:11 +00:00
Code Issues Releases Wiki Activity

fixup! feat(core): prevent interruption of workflows from other communication interfaces

Browse Source
This commit is contained in:
tychovrahe 2023-04-25 21:44:58 +02:00
parent a3e8f191a6
commit a65ab6bad0
3 changed files with 12 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
core/src/apps/webauthn/__init__.py
View File

@ -1,6 +1,6 @@
def boot() -> None:
def boot(mutex) -> None:
from trezor import loop
import usb
from .fido2 import handle_reports
loop.schedule(handle_reports(usb.iface_webauthn))
loop.schedule(handle_reports(usb.iface_webauthn, mutex))

6
core/src/apps/webauthn/fido2.py
View File

@ -538,7 +538,7 @@ def send_cmd_sync(cmd: Cmd, iface: HID) -> None:
seq += 1
async def handle_reports(iface: HID) -> None:
async def handle_reports(iface: HID, mutex) -> None:
dialog_mgr = DialogManager(iface)
while True:
@ -546,6 +546,10 @@ async def handle_reports(iface: HID) -> None:
req = await _read_cmd(iface)
if req is None:
continue
if mutex is not None and mutex.get_busy(iface.iface_num()):
resp: Cmd | None = cmd_error(req.cid, _ERR_CHANNEL_BUSY)
else:
mutex.set_busy(iface.iface_num())
if not dialog_mgr.allow_cid(req.cid):
resp: Cmd | None = cmd_error(req.cid, _ERR_CHANNEL_BUSY)
else:

6
core/src/session.py
View File

@ -7,10 +7,12 @@ import usb
apps.base.boot()
mutex = Mutex()
if not utils.BITCOIN_ONLY and usb.ENABLE_IFACE_WEBAUTHN:
import apps.webauthn
apps.webauthn.boot()
apps.webauthn.boot(mutex)
if __debug__:
import apps.debug
@ -22,8 +24,6 @@ apps.base.set_homescreen()
workflow.start_default()
mutex = Mutex()
mutex.add(usb.iface_wire.iface_num())
mutex.add(usb.iface_debug.iface_num())