BLE pairing on boards without a button accepts any phone for 60 s after every boot
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 35/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- rust
- 領域
- embedded-iot, security
調査の方向性
Read boards/esp/src/ble/lifecycle.rs and boards/esp/src/ble/transport.rs, then review the WiCAN profile at hardware-profiles/meatpi-wican-esp32c3.toml and the pairing guidance in docs/ble.md. Decide how pairing should be gated and authenticated across supported hardware, and check the setup wizard issue at OpenCarControls/contracts#10. Done means strangers cannot bond during a normal reboot and the pairing flow is documented.
索引モデルが issue の本文から書いたものです。
説明
Problem
- When
pairing_button_pinisn't set (the WiCAN profile sets it to-1, inhardware-profiles/meatpi-wican-esp32c3.toml:32),ble_lifecycle_taskopens the pairing window forpairing_window_seconds(60 s) on every boot (boards/esp/src/ble/lifecycle.rs:91). - Pairing uses
IoCapabilities::NoInputNoOutput(boards/esp/src/ble/transport.rs:173). That's Just Works pairing: unauthenticated, with no MITM protection and no user confirmation. - So any phone in range after a reboot, watchdog reset or brownout can bond, and bonded phones are trusted from then on.
docs/ble.mddoesn't describe the automatic opening on boot yet.
Impact
Low while v1 is read-only: someone can read the car's state. It must be fixed before E-GMP commands are enabled, because then a bonded phone can control the car.
Ideas
- Open the window at boot only while no phone is bonded (first setup), and require a deliberate action after that: a bonded phone's command, or a physical action such as unplugging and replugging the device a set number of times.
- Use authenticated pairing (passkey or numeric comparison) where the hardware allows it, e.g. a code shown in the app and confirmed with a pattern on the device's LED.
- Cover this in the setup wizard (https://github.com/OpenCarControls/contracts/issues/10).
- Document the behaviour in
docs/ble.md.
Done when
A stranger near the car can't bond during a normal reboot, and the pairing flow is documented.
From the audit
- While the window is open, any already-bonded phone can also clear, remove or upsert paired phones, so a window opened at boot widens that too.
- Related: removing a phone doesn't revoke its bond on ESP (https://github.com/OpenCarControls/firmware/issues/9).
- 主要言語
- Rust
- スター
- 1
- フォーク
- 0
- 平均マージ
- 25分
- マージ済み PR（30日）
- 1
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
OpenCarControls/firmware のほかの issue
-
Refresh stale docsオープン
難易度 1/5 1〜3時間 初心者へのやさしさ 67/100
OpenCarControls/firmware#35 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
OpenCarControls/firmware#34 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
OpenCarControls/firmware#31 ·
-
good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
OpenCarControls/firmware#30 ·
-
e-gmp
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
OpenCarControls/firmware#43 ·
OpenCarControls/firmware の issue をすべて見る
似ている issue
-
proxy: include denial_category in the "proxy request denied" log line対応中かも @sreyas-endor が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
nolabs-ai/nono#2172 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
metasequoiaime/msime#9113 ·
メンテナーはふだん 1 日以内に返信
-
area:desktop priority:p2 type:chore
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
メンテナーはふだん 1 日以内に返信
-
area: supply-chain
難易度 1/5 1時間未満 初心者へのやさしさ 73/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100