BLE pairing on boards without a button accepts any phone for 60 s after every boot
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- rust
- Área
- embedded-iot, security
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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).
- Lenguaje dominante
- Rust
- Estrellas
- 1
- Forks
- 0
- Merge medio
- 25 min
- PR fusionados (30 d)
- 1
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de OpenCarControls/firmware
-
Refresh stale docsAbierto
Dificultad 1/5 1-3 horas Aptitud para principiantes 67/100
OpenCarControls/firmware#35 ·
-
Clean up E-GMP leftoversAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
OpenCarControls/firmware#34 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
OpenCarControls/firmware#31 ·
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
OpenCarControls/firmware#30 ·
-
e-gmp
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
OpenCarControls/firmware#43 ·
Todos los issues de OpenCarControls/firmware
Issues similares
-
corro-client drops the last byte of a response line that has no trailing newlinePosiblemente ocupada @RaphaelFakhri la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
proxy: include denial_category in the "proxy request denied" log linePosiblemente ocupada @sreyas-endor la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
nolabs-ai/nono#2172 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
metasequoiaime/msime#9113 ·
Los mantenedores suelen responder en 1 día
-
area:desktop priority:p2 type:chore
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
area: supply-chain
Dificultad 1/5 Menos de una hora Aptitud para principiantes 73/100
Los mantenedores suelen responder en 1 día