Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ability to set a PIN required for Bluetooth connections #344

Closed
mitchellrj opened this issue Apr 12, 2016 · 2 comments
Closed

Add ability to set a PIN required for Bluetooth connections #344

mitchellrj opened this issue Apr 12, 2016 · 2 comments

Comments

@mitchellrj
Copy link

I'm deeply concerned that currently any device can connect to the OpenXC VI without authentication of any kind. I see in the source that there is some stuff around enabling PIN authentication, but I can't find any way to actually set a PIN, or documentation of this feature.

@peplin
Copy link
Member

peplin commented Apr 12, 2016

Hey Mitchell, there is no custom PIN support implemented at the moment. To be clear, it does use Bluetooth authentication and encryption, but using the simple pairing protocol (or a default PIN as is common with Bluetooth). The data channel is encrypted. However as you point out, if the VI is powered on and your device is not connected, another device could connect and pair.

The fact that this i a research and development platform not indented for production use, and the attack window is fairly limited in time and space, I don't judge this to be a high risk. I think it's more likely a rouge app on your own phone could take over the Bluetooth connection and try to read or write to the stream.

Security has not been a priority here because again this is for R&D only - this is the primary reason why raw CAN reads and writes are disabled by default.

@pjt0620
Copy link
Contributor

pjt0620 commented Dec 20, 2019

As of #425 you can now set the pin used by bluetooth via the build flag DEFAULT_BLUETOOTH_PIN.

@pjt0620 pjt0620 closed this as completed Dec 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants