[Coverity CID :188757]Memory - illegal accesses in /subsys/bluetooth/shell/bt.c #10570
Assignees
Labels
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
mandarcthorat1
added
area: Bluetooth
bug
Vudentz
added a commit
to Vudentz/zephyr
that referenced
this issue
Nov 9, 2018
BT_LE_ADV_CONN_DIR_LOW_DUTY declares a local variable so it cannot go out of scope, so instead of assigning in the scope of the if statement this rework the code to mantain the declaration in the same scope it is used. Fixes zephyrproject-rtos#10570 Signed-off-by: Luiz Augusto von Dentz <[email protected]>
jhedberg
pushed a commit
that referenced
this issue
Nov 9, 2018
BT_LE_ADV_CONN_DIR_LOW_DUTY declares a local variable so it cannot go out of scope, so instead of assigning in the scope of the if statement this rework the code to mantain the declaration in the same scope it is used. Fixes #10570 Signed-off-by: Luiz Augusto von Dentz <[email protected]>
nashif
pushed a commit
that referenced
this issue
Nov 13, 2018
BT_LE_ADV_CONN_DIR_LOW_DUTY declares a local variable so it cannot go out of scope, so instead of assigning in the scope of the if statement this rework the code to mantain the declaration in the same scope it is used. Fixes #10570 Signed-off-by: Luiz Augusto von Dentz <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Static code scan issues seen in File: /subsys/bluetooth/shell/bt.c
Category: Memory - illegal accesses
Function: cmd_directed_adv
Component: Bluetooth
CID: 188757
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996
The text was updated successfully, but these errors were encountered: