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: controller: Fix adv/scan context access post release #35014

Merged

Conversation

cvinayak
Copy link
Contributor

@cvinayak cvinayak commented May 9, 2021

Fix advertiser and scanning context being accessed on done
event when connection complete node rx that is processed
earlier has release them.

Relates to #30735.
Fixes #35013.

Signed-off-by: Vinayak Kariappa Chettimada vich@nordicsemi.no

Fix advertiser and scanning context being accessed on done
event when connection complete node rx that is processed
earlier has release them.

Relates to zephyrproject-rtos#30735.
Fixes zephyrproject-rtos#35013.

Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
@cvinayak cvinayak requested review from wopu-ot and mtpr-ot May 9, 2021 06:39
@cvinayak cvinayak added the bug The issue is a bug, or the PR is fixing a bug label May 9, 2021
@cvinayak cvinayak self-assigned this May 10, 2021
Copy link
Collaborator

@mtpr-ot mtpr-ot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@mtpr-ot
Copy link
Collaborator

mtpr-ot commented May 10, 2021

Looks good!

Also, this is something we have observed, and could be related to what was attempted to fix in #34123.

@carlescufi carlescufi merged commit 30f260d into zephyrproject-rtos:master May 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth Controller area: Bluetooth bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bluetooth: Controller: Out-of-Bound ULL context access during connection completion
3 participants