alex/esp-idf
alex/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf.git synced 2024-10-05 20:47:46 -04:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

components/bt: Fix connection fail and crash when receive unknown AT cmd

Browse Source
This commit is contained in:
baohongde 2020-01-20 11:32:26 +08:00
parent 0c99aef08d
commit 5ca4c85497
3 changed files with 16 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
components/bt/host/bluedroid/api/include/api/esp_hf_ag_api.h
View File

@ -335,7 +335,8 @@ esp_err_t esp_bt_hf_volume_control(esp_bd_addr_t remote_bda, esp_hf_volume_contr
* As a precondition to use this API, Service Level Connection shall exist between AG and HF Client. * As a precondition to use this API, Service Level Connection shall exist between AG and HF Client.
* *
* @param[in] remote_addr: remote bluetooth device address * @param[in] remote_addr: remote bluetooth device address
* @param[in] unat: AT command string from HF Client * @param[in] unat: User AT command response to HF Client.
* It will response "ERROR" by default if unat is NULL.
* @return * @return
* - ESP_OK: disconnect request is sent to lower layer * - ESP_OK: disconnect request is sent to lower layer
* - ESP_INVALID_STATE: if bluetooth stack is not yet enabled * - ESP_INVALID_STATE: if bluetooth stack is not yet enabled

15
components/bt/host/bluedroid/btc/profile/std/hf_ag/btc_hf_ag.c
View File

@ -461,12 +461,18 @@ static bt_status_t btc_hf_unat_response(bt_bdaddr_t *bd_addr, const char *unat)
return BT_STATUS_FAIL; return BT_STATUS_FAIL;
} }
if (is_connected(NULL) && (idx != BTC_HF_INVALID_IDX)) if (is_connected(bd_addr) && (idx != BTC_HF_INVALID_IDX))
{ {
tBTA_AG_RES_DATA ag_res; tBTA_AG_RES_DATA ag_res;
/* Format the response and send */ /* Format the response and send */
memset(&ag_res, 0, sizeof(ag_res)); memset(&ag_res, 0, sizeof(ag_res));
strncpy(ag_res.str, unat, BTA_AG_AT_MAX_LEN); if (unat != NULL) {
strncpy(ag_res.str, unat, BTA_AG_AT_MAX_LEN);
} else {
ag_res.ok_flag = BTA_AG_OK_ERROR;
ag_res.errcode = BTA_AG_ERR_OP_NOT_SUPPORTED;
}
BTA_AgResult(hf_local_param[idx].btc_hf_cb.handle, BTA_AG_UNAT_RES, &ag_res); BTA_AgResult(hf_local_param[idx].btc_hf_cb.handle, BTA_AG_UNAT_RES, &ag_res);
return BT_STATUS_SUCCESS; return BT_STATUS_SUCCESS;
} }
@ -878,6 +884,10 @@ void btc_hf_arg_deep_copy(btc_msg_t *msg, void *p_dest, void *p_src)
switch (msg->act) { switch (msg->act) {
case BTC_HF_UNAT_RESPONSE_EVT: case BTC_HF_UNAT_RESPONSE_EVT:
{ {
if (src->unat_rep.unat == NULL) {
break;
}
dst->unat_rep.unat = (char *)osi_malloc(strlen(src->unat_rep.unat)+1); dst->unat_rep.unat = (char *)osi_malloc(strlen(src->unat_rep.unat)+1);
if(dst->unat_rep.unat) { if(dst->unat_rep.unat) {
memcpy(dst->unat_rep.unat, src->unat_rep.unat, strlen(src->unat_rep.unat)+1); memcpy(dst->unat_rep.unat, src->unat_rep.unat, strlen(src->unat_rep.unat)+1);
@ -1066,6 +1076,7 @@ void btc_hf_call_handler(btc_msg_t *msg)
case BTC_HF_UNAT_RESPONSE_EVT: case BTC_HF_UNAT_RESPONSE_EVT:
{ {
btc_hf_unat_response(&arg->unat_rep.remote_addr, arg->unat_rep.unat); btc_hf_unat_response(&arg->unat_rep.remote_addr, arg->unat_rep.unat);
break;
} }
case BTC_HF_CME_ERR_EVT: case BTC_HF_CME_ERR_EVT:

2
examples/bluetooth/bluedroid/classic_bt/hfp_ag/main/bt_app_hf.c
View File

@ -190,7 +190,7 @@ void bt_app_hf_cb(esp_hf_cb_event_t event, esp_hf_cb_param_t *param)
case ESP_HF_UNAT_RESPONSE_EVT: case ESP_HF_UNAT_RESPONSE_EVT:
{ {
ESP_LOGI(BT_HF_TAG, "--UNKOW AT CMD: %s", param->unat_rep.unat); ESP_LOGI(BT_HF_TAG, "--UNKOW AT CMD: %s", param->unat_rep.unat);
esp_hf_unat_response(hf_peer_addr, param->unat_rep.unat); esp_hf_unat_response(hf_peer_addr, NULL);
break; break;
} }