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 v3.7-branch] Bluetooth: L2CAP: Downgrade user_data API error to a warning #77239

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

zephyrbot
Copy link
Collaborator

@zephyrbot zephyrbot commented Aug 19, 2024

Backport 6fa6c4c from #77113.

Fixes #77088

When user_data is not zeroed-out, the API returns an error. Downgrade
the API error to a warning log instead.

Introducing this check (#76489) broke a few PTS tests, as user_data is
not initialized by `net_buf_alloc()`. Doing so is in discussion:

#77088

Signed-off-by: Jonathan Rico <[email protected]>
(cherry picked from commit 6fa6c4c)
@zephyrbot zephyrbot added Backport Backport PR and backport failure issues area: Bluetooth Host Bluetooth Host (excluding BR/EDR) area: Bluetooth platform: nRF BSIM Nordic Semiconductors, nRF BabbleSim labels Aug 19, 2024
@jhedberg
Copy link
Member

@jori-nordic this one needs a respective issue before it can be accepted as a backport

@jori-nordic
Copy link
Collaborator

I would but I don't know which PTS tests specifically failed.
@sjanc would you have the time to open a bug listing a few tests that failed because of that change?

@Thalley
Copy link
Collaborator

Thalley commented Sep 16, 2024

Ping @jori-nordic @sjanc

@henrikbrixandersen henrikbrixandersen added this to the v3.7.1 milestone Nov 4, 2024
@henrikbrixandersen
Copy link
Member

This needs a bug report in order to get backported.

Copy link

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Jan 19, 2025
@dleach02 dleach02 modified the milestones: v3.7.1, v3.7.2 Jan 20, 2025
@github-actions github-actions bot removed the Stale label Jan 21, 2025
@nashif nashif merged commit e659803 into v3.7-branch Jan 23, 2025
33 of 34 checks passed
@nashif nashif deleted the backport-77113-to-v3.7-branch branch January 23, 2025 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth Host Bluetooth Host (excluding BR/EDR) area: Bluetooth Backport Backport PR and backport failure issues platform: nRF BSIM Nordic Semiconductors, nRF BabbleSim
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

8 participants