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: Host: df: Uninitialized variable used to assign length of antenna identifiers #42735

Closed
ppryga-nordic opened this issue Feb 11, 2022 · 0 comments · Fixed by #42734
Closed
Assignees
Labels
bug The issue is a bug, or the PR is fixing a bug
Milestone

Comments

@ppryga-nordic
Copy link
Collaborator

Describe the bug
Wrong variable is used to assign length of antenna identifiers for CTE reception in AoA mode.
The variable used is cp->switch_pattern_len pointer instead of function argument params->num_ant_ids.

To Reproduce
Steps to reproduce the behavior:

  1. Build direction_finding_connectionless_rx sample for nRF53 and run on the device.
  2. When synchronized with periodic advertising, the CTE reception is enabled. Controller stops responding for messages due to wrong length of the HCI command.
  3. Host asserts due to lack of response from Controller.

Expected behavior
CTE is successfully received and reported to Host. Controller is still responsive.

Impact
DF samples are not usable.

@ppryga-nordic ppryga-nordic added bug The issue is a bug, or the PR is fixing a bug has-pr labels Feb 11, 2022
@ppryga-nordic ppryga-nordic self-assigned this Feb 11, 2022
@ppryga-nordic ppryga-nordic added this to the v3.0.0 milestone Feb 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant