michaelh/zephyr
1
0
Fork 0
Code Releases Activity

Bluetooth: Controller: Fix coverity issue 340844

Browse source 
Fix coverity issue 340844, Uninitialized pointer read.

Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
This commit is contained in:
Vinayak Kariappa Chettimada 2024-01-29 12:17:18 +01:00 • committed by Carles Cufí
commit 83321eed41
1 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
subsys/bluetooth/controller/ll_sw/ull_df.c
View file

@ -1073,13 +1073,15 @@ static uint8_t cte_info_clear(struct ll_adv_set *adv, struct lll_df_adv_cfg *df_
/* NOTE: ULL_ADV_PDU_EXTRA_DATA_ALLOC_NEVER is just information that extra_data /* NOTE: ULL_ADV_PDU_EXTRA_DATA_ALLOC_NEVER is just information that extra_data
* should be removed in case of this call ull_adv_sync_pdu_alloc. * should be removed in case of this call ull_adv_sync_pdu_alloc.
*/ */
extra_data_prev = NULL;
extra_data = NULL;
err = ull_adv_sync_pdu_alloc(adv, ULL_ADV_PDU_EXTRA_DATA_ALLOC_NEVER, &pdu_prev, &pdu, err = ull_adv_sync_pdu_alloc(adv, ULL_ADV_PDU_EXTRA_DATA_ALLOC_NEVER, &pdu_prev, &pdu,
&extra_data_prev, &extra_data, ter_idx); &extra_data_prev, &extra_data, ter_idx);
if (err != BT_HCI_ERR_SUCCESS) { if (err != BT_HCI_ERR_SUCCESS) {
return err; return err;
} }
if (extra_data) { if (extra_data_prev && extra_data) {
ull_adv_sync_extra_data_set_clear(extra_data_prev, extra_data, 0, ull_adv_sync_extra_data_set_clear(extra_data_prev, extra_data, 0,
ULL_ADV_PDU_HDR_FIELD_CTE_INFO, NULL); ULL_ADV_PDU_HDR_FIELD_CTE_INFO, NULL);
} }