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

[Backport v2.7-branch] Bluetooth: Host: Avoid processing "no change" encryption changes #74122

Open
wants to merge 1 commit into
base: v2.7-branch
Choose a base branch
from

Conversation

zephyrbot
Copy link
Collaborator

@zephyrbot zephyrbot commented Jun 11, 2024

Backport bf363d7 from #73945.

Fixes #74926

@zephyrbot zephyrbot added the Backport Backport PR and backport failure issues label Jun 11, 2024
If the new encryption state is the same as the old one, there's no point in
doing additional processing or callbacks. Simply log a warning and ignore
the HCI event in such a case.

Signed-off-by: Johan Hedberg <[email protected]>
(cherry picked from commit bf363d7)
@jhedberg jhedberg force-pushed the backport-73945-to-v2.7-branch branch from 13519a6 to 8f2b8ba Compare June 12, 2024 08:06
@joerchan joerchan requested review from alwa-nordic and removed request for joerchan June 12, 2024 11:24
@henrikbrixandersen henrikbrixandersen added this to the v2.7.6 milestone Jun 25, 2024
@cfriedt cfriedt modified the milestones: v2.7.6, v2.7.7 Jun 25, 2024
@cfriedt
Copy link
Member

cfriedt commented Jun 26, 2024

I assume something is up with CI and native_posix is no longer available. (had to cancel the job after 24 hrs).

I'll merge, since it has maintainer approval.

Edit - I'm unable to merge to the LTS branch(?)

@cfriedt
Copy link
Member

cfriedt commented Sep 14, 2024

@stephanosio, @nashif - I think the clang workflow has some issues possibly with the chosen GitHub runner. Can you take a look?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth Host area: Bluetooth Backport Backport PR and backport failure issues
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants