Skip to content

host/ble_gap.c : Check allowed random address #2047

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rahult-github
Copy link
Contributor

Add a check to validate whether the random address being set is allowed as per spec

@github-actions github-actions bot added host size/S Small PR labels Jun 16, 2025
@rahult-github rahult-github force-pushed the bugfix/check_random_address branch from 9cbba0e to 1ff031a Compare June 16, 2025 11:23
@rahult-github
Copy link
Contributor Author

Hi @sjanc , please take a look.

@rahult-github
Copy link
Contributor Author

Hi @sjanc ,

image

The compliance check issue seems to be failing due to some missing file and not actual coding style. Can you kindly confirm this once ?

@sjanc sjanc added CI and removed CI labels Jun 24, 2025
@sjanc
Copy link
Contributor

sjanc commented Jun 24, 2025

Hi @sjanc ,

image

The compliance check issue seems to be failing due to some missing file and not actual coding style. Can you kindly confirm this once ?

You may have to rebase to latest master, there were some changes in CI (ie. we moved away from uncrustify towards clangd for code style validation)

@rahult-github rahult-github force-pushed the bugfix/check_random_address branch from 1ff031a to 71b7e12 Compare June 24, 2025 07:02
@rahult-github
Copy link
Contributor Author

Hi @sjanc , Thanks for this . Have rebased now.

@rahult-github rahult-github force-pushed the bugfix/check_random_address branch 3 times, most recently from 9e250b5 to 4884247 Compare June 26, 2025 09:55
Add a check to validate whether the random address being set is
allowed as per spec
@rahult-github rahult-github force-pushed the bugfix/check_random_address branch from 4884247 to 667aec2 Compare June 26, 2025 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants