Skip to content

Commit 165e3ed

Browse files
committed
Merge branch 'bugfix/btdm_mem_leak_of_bt' into 'master'
component/bt: Fix mem leak of bt See merge request idf/esp-idf!1924
2 parents 2abba13 + 0519a73 commit 165e3ed

File tree

5 files changed

+19
-5
lines changed

5 files changed

+19
-5
lines changed

components/bt/bluedroid/bta/av/bta_av_act.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1847,6 +1847,8 @@ void bta_av_dereg_comp(tBTA_AV_DATA *p_data)
18471847

18481848
/* make sure that the timer is not active */
18491849
bta_sys_stop_timer(&p_scb->timer);
1850+
list_free(p_scb->a2d_list);
1851+
p_scb->a2d_list = NULL;
18501852
utl_freebuf((void **)&p_cb->p_scb[p_scb->hdi]);
18511853
}
18521854

components/bt/bluedroid/btc/profile/std/a2dp/btc_a2dp_sink.c

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -273,7 +273,10 @@ error_exit:;
273273
vQueueDelete(btc_aa_snk_ctrl_queue);
274274
btc_aa_snk_ctrl_queue = NULL;
275275
}
276-
276+
if (btc_aa_snk_queue_set) {
277+
vQueueDelete(btc_aa_snk_queue_set);
278+
btc_aa_snk_queue_set = NULL;
279+
}
277280
return false;
278281
}
279282

@@ -292,6 +295,9 @@ void btc_a2dp_sink_shutdown(void)
292295

293296
vQueueDelete(btc_aa_snk_ctrl_queue);
294297
btc_aa_snk_ctrl_queue = NULL;
298+
299+
vQueueDelete(btc_aa_snk_queue_set);
300+
btc_aa_snk_queue_set = NULL;
295301
}
296302

297303
/*****************************************************************************

components/bt/bluedroid/btc/profile/std/a2dp/btc_a2dp_source.c

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -348,7 +348,10 @@ error_exit:;
348348
vQueueDelete(btc_aa_src_ctrl_queue);
349349
btc_aa_src_ctrl_queue = NULL;
350350
}
351-
351+
if (btc_aa_src_queue_set) {
352+
vQueueDelete(btc_aa_src_queue_set);
353+
btc_aa_src_queue_set = NULL;
354+
}
352355
return false;
353356
}
354357

@@ -367,6 +370,9 @@ void btc_a2dp_source_shutdown(void)
367370

368371
vQueueDelete(btc_aa_src_ctrl_queue);
369372
btc_aa_src_ctrl_queue = NULL;
373+
374+
vQueueDelete(btc_aa_src_queue_set);
375+
btc_aa_src_queue_set = NULL;
370376
}
371377

372378
/*****************************************************************************

examples/bluetooth/a2dp_sink/main/bt_app_core.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ static xTaskHandle bt_app_task_handle = NULL;
2727
bool bt_app_work_dispatch(bt_app_cb_t p_cback, uint16_t event, void *p_params, int param_len, bt_app_copy_cb_t p_copy_cback)
2828
{
2929
ESP_LOGD(BT_APP_CORE_TAG, "%s event 0x%x, param len %d", __func__, event, param_len);
30-
30+
3131
bt_app_msg_t msg;
3232
memset(&msg, 0, sizeof(bt_app_msg_t));
3333

@@ -96,7 +96,7 @@ static void bt_app_task_handler(void *arg)
9696
void bt_app_task_start_up(void)
9797
{
9898
bt_app_task_queue = xQueueCreate(10, sizeof(bt_app_msg_t));
99-
xTaskCreate(bt_app_task_handler, "BtAppT", 2048, NULL, configMAX_PRIORITIES - 3, bt_app_task_handle);
99+
xTaskCreate(bt_app_task_handler, "BtAppT", 2048, NULL, configMAX_PRIORITIES - 3, &bt_app_task_handle);
100100
return;
101101
}
102102

examples/bluetooth/a2dp_source/main/bt_app_core.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ static void bt_app_task_handler(void *arg)
9696
void bt_app_task_start_up(void)
9797
{
9898
bt_app_task_queue = xQueueCreate(10, sizeof(bt_app_msg_t));
99-
xTaskCreate(bt_app_task_handler, "BtAppT", 2048, NULL, configMAX_PRIORITIES - 3, bt_app_task_handle);
99+
xTaskCreate(bt_app_task_handler, "BtAppT", 2048, NULL, configMAX_PRIORITIES - 3, &bt_app_task_handle);
100100
return;
101101
}
102102

0 commit comments

Comments
 (0)