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

Bluetooth: ll_sync_create_cancel fails with BT_HCI_ERR_CMD_DISALLOWED before BT_HCI_EVT_LE_PER_ADV_SYNC_ESTABLISHED is generated #40204

Closed
ppryga-nordic opened this issue Nov 9, 2021 · 0 comments · Fixed by #45696
Assignees
Labels
area: Bluetooth Controller area: Bluetooth bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug

Comments

@ppryga-nordic
Copy link
Collaborator

Describe the bug
After change of moment when BT_HCI_EVT_LE_PER_ADV_SYNC_ESTABLISHED event is send to host (when first periodic advertising PDU is received) the ll_sync_create_cancel stopped to correctly. It can't be called after ull_sync_setup is called because scan->per_scan.sync is set to NULL in ull_sync_setup.

To Reproduce

  1. Build direction_finding_connectionless_tx sample as an advertiser.
  2. Build direction_finding_connectionless_rx sample as a scanned.
  3. Run both.
  4. Reset advertiser once sync is created reset advertiser multiple times with very short delays.
  5. After periodic advertising sync timeouts there should be a message on scanner output that Deleting Periodic Advertising Sync... failed

Expected behavior
Periodic advertising should be successfully cancelled or deleted before BT_HCI_EVT_LE_PER_ADV_SYNC_ESTABLISHED event is generated.

Impact
Moderate, it is related with periodic advertising only.

@ppryga-nordic ppryga-nordic added the bug The issue is a bug, or the PR is fixing a bug label Nov 9, 2021
@carlescufi carlescufi changed the title [BT] ll_sync_create_cancel fails with BT_HCI_ERR_CMD_DISALLOWED before BT_HCI_EVT_LE_PER_ADV_SYNC_ESTABLISHED is generated Bluetooth: ll_sync_create_cancel fails with BT_HCI_ERR_CMD_DISALLOWED before BT_HCI_EVT_LE_PER_ADV_SYNC_ESTABLISHED is generated Nov 9, 2021
@github-actions github-actions bot added the Stale label Jan 9, 2022
@cvinayak cvinayak removed the Stale label Jan 10, 2022
@zephyrproject-rtos zephyrproject-rtos deleted a comment from github-actions bot Jan 10, 2022
@github-actions github-actions bot added the Stale label Mar 12, 2022
@zephyrproject-rtos zephyrproject-rtos deleted a comment from github-actions bot Mar 22, 2022
@cvinayak cvinayak removed the Stale label Mar 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment