-
Notifications
You must be signed in to change notification settings - Fork 8.3k
Closed
Labels
area: Bluetootharea: Bluetooth HostBluetooth Host (excluding BR/EDR)Bluetooth Host (excluding BR/EDR)bugThe issue is a bug, or the PR is fixing a bugThe issue is a bug, or the PR is fixing a bugpriority: lowLow impact/importance bugLow impact/importance bug
Description
The documentation on _bt_gatt_ccc.value specifies that only connected peers contribute to that value. But if CONFIG_BT_SETTINGS_CCC_LAZY_LOADING=n, the implementation summarizes over all stores CCC values, including (bonded) disconnected peers.
zephyr/subsys/bluetooth/host/gatt.c
Lines 2125 to 2129 in 5c7c099
| for (i = 0; i < ARRAY_SIZE(ccc->cfg); i++) { | |
| if (ccc->cfg[i].value > value) { | |
| value = ccc->cfg[i].value; | |
| } | |
| } |
Expected behavior
Only connected peers should be included in the summarized value.
Metadata
Metadata
Assignees
Labels
area: Bluetootharea: Bluetooth HostBluetooth Host (excluding BR/EDR)Bluetooth Host (excluding BR/EDR)bugThe issue is a bug, or the PR is fixing a bugThe issue is a bug, or the PR is fixing a bugpriority: lowLow impact/importance bugLow impact/importance bug