AI-generated analysisPublished automatically and not human-verified. Validated context appears in community notes below.
← Watch feed
Informational 20 Bitcoin

feat(core/prodtest): add ble-monitor

Public commit record

What the developer wrote

Authored by tychovrahe

57/100 · Thin
feat(core/prodtest): add ble-monitor
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
The short version

What changed, and why it matters

This commit adds a new factory-testing command called ble-monitor to Trezor hardware wallets. It lets a technician make the device advertise over Bluetooth, watch connection events, and manually approve or reject pairing requests. The change is confined to the production-test firmware (prodtest), not the normal user wallet firmware, and requires physical CLI access to use. It introduces a global flag that stops the periodic Bluetooth timer from auto-handling events while the monitor is running, so pairing must be confirmed by an operator typing y/n.

Recommended action

Treat as a low-risk feature addition in the production-test environment. Reviewers should confirm that g_ble_monitor_active is always cleared on exit paths (including errors and CLI abort), that ble_switch_off() reliably stops advertising and disconnects, and that the shared CLI read path cannot block other critical tasks. No immediate security patch is indicated.

Security signals we found

01

New Bluetooth pairing/connection surface added to prodtest firmware

02

Manual pairing confirmation replaces automatic pairing acceptance while monitor is active

03

Global flag g_ble_monitor_active suppresses periodic timer event handling

04

Operator-controlled advertising name and mode switching

05

No input validation beyond length truncation for advertising name

06

Change is scoped to production-test project, not main firmware

Risk score

Why this scored 20/100

Our methodology →
Potential impact 2/30
Exploitability 3/25
Stealth signal 2/15
Affected reach 3/15
Confidence 7/10
Evidence quality 3/5
Human-validated context

Community notes

Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.

No validated notes yet.

The AI analysis stands alone for now. Submit a note if you can add evidence or important context.