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

[Coverity CID: 330735] Uninitialized scalar variable in subsys/bluetooth/controller/ll_sw/ull_filter.c #74718

Closed
zephyrbot opened this issue Jun 22, 2024 · 3 comments · Fixed by #81633
Assignees
Labels
area: Bluetooth Controller area: Bluetooth bug The issue is a bug, or the PR is fixing a bug Coverity A Coverity detected issue or its fix priority: medium Medium impact/importance bug

Comments

@zephyrbot
Copy link
Collaborator

Static code scan issues found in file:

https://github.com/zephyrproject-rtos/zephyr/tree/dcf42917c550714d2457947538b9e29d083e872e/subsys/bluetooth/controller/ll_sw/ull_filter.c

Category: Uninitialized variables
Function: rpa_adv_refresh
Component: Bluetooth
CID: 330735

Details:

lll_adv_aux_data_enqueue(lll_aux, sec_idx);

Please fix or provide comments in coverity using the link:

https://scan9.scan.coverity.com/#/project-view/29271/12996?selectedIssue=330735

For more information about the violation, check the Coverity Reference. (CWE-457)

Note: This issue was created automatically. Priority was set based on classification
of the file affected and the impact field in coverity. Assignees were set using the MAINTAINERS file.

@zephyrbot zephyrbot added area: Bluetooth area: Bluetooth Controller bug The issue is a bug, or the PR is fixing a bug Coverity A Coverity detected issue or its fix priority: high High impact/importance bug labels Jun 22, 2024
@aescolar
Copy link
Member

Closely related to #74716

@aescolar
Copy link
Member

@cvinayak any update on these?

@aescolar aescolar added priority: medium Medium impact/importance bug and removed priority: high High impact/importance bug labels Jul 12, 2024
@aescolar
Copy link
Member

aescolar commented Jul 12, 2024

Lowering priority. This code has run with valgrind and ASAN quite a times and nothing was reported at runtime so it is either not happening often enough or a false positive. More analysis is needed though.

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 Coverity A Coverity detected issue or its fix priority: medium Medium impact/importance bug
Projects
None yet
3 participants