Skip to content

Conversation

cvinayak
Copy link
Contributor

Fix the leak of node rx buffer used to generate the
connection complete and CSA#2 event introduced in the
commit 4a5f263 ("Bluetooth: controller: split: Validate
chan map and hop value") and the
commit 94d5f08 ("Bluetooth: controller: fixing error
re. all zero chmap in conn-ind").

Bluetooth: controller: Fix interval check in CONNECT_IND PDU

Check for interval value in received CONNECT_IND PDU and
ignore connection setup.

Signed-off-by: Vinayak Kariappa Chettimada [email protected]

Fix the leak of node rx buffer used to generate the
connection complete and CSA#2 event introduced in the
commit 4a5f263 ("Bluetooth: controller: split: Validate
chan map and hop value") and the
commit 94d5f08 ("Bluetooth: controller: fixing error
re. all zero chmap in conn-ind").

Signed-off-by: Vinayak Kariappa Chettimada <[email protected]>
Check for interval value in received CONNECT_IND PDU and
ignore connection setup.

Signed-off-by: Vinayak Kariappa Chettimada <[email protected]>
Copy link
Contributor

@wopu-ot wopu-ot left a comment

Choose a reason for hiding this comment

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

LGTM

@carlescufi carlescufi merged commit 813b241 into zephyrproject-rtos:master Mar 16, 2021
@zephyrbot
Copy link

The backport to v1.14-branch failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-v1.14-branch v1.14-branch
# Navigate to the new working tree
cd .worktrees/backport-v1.14-branch
# Create a new branch
git switch --create backport-33278-to-v1.14-branch
# Cherry-pick the merged commits of this pull request and resolve the conflicts
git cherry-pick 813b24157b3e6bd51250cadabd96d087bf284caa~2..813b24157b3e6bd51250cadabd96d087bf284caa
# Push it to GitHub
git push --set-upstream origin backport-33278-to-v1.14-branch
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-v1.14-branch

Then, create a pull request where the base branch is v1.14-branch and the compare/head branch is backport-33278-to-v1.14-branch.

@Matheus-Garbelini
Copy link

@cvinayak nice 👍

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.

6 participants