Skip to content

Conversation

hedger
Copy link
Member

@hedger hedger commented Jun 1, 2025

What's new

  • ble: use unique root security keys for new pairings after pairing reset;
  • ble: added migrations for existing pairing data;
  • unit_tests: added migration tests

NB: Pairing information is upgraded to new format with this PR, it's not backwards compatible! Downgrading will lead to reset of BLE pairings.

Kudos to daw10 on Discord for providing report on potential security issues with old key usage model

Verification

  • Upgrade to this branch, pairings from apps & main system should still work
  • Check that keys file format is upgraded
  • Reset pairing multiple times and validate that root keys stored at the start of pairing file are unique

Checklist (For Reviewer)

  • PR has description of feature/bug or link to Confluence/Jira task
  • Description contains actions to verify feature/bugfix
  • I've built this code, uploaded it to the device and verified feature/bugfix

…et; added migrations for existing pairing data; unit_tests: added migration tests
Copy link

github-actions bot commented Jun 1, 2025

Compiled f7 firmware for commit 6b02ca4f:

@hedger hedger added New Feature Contains an IMPLEMENTATION of a new feature Bluetooth labels Jun 1, 2025
@hedger hedger marked this pull request as ready for review June 1, 2025 15:11

This comment was marked as outdated.

@hedger hedger merged commit 0d5beed into dev Sep 24, 2025
11 checks passed
@hedger hedger deleted the hedger/ble_keys_migration branch September 24, 2025 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bluetooth New Feature Contains an IMPLEMENTATION of a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants