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

NimBLE: Fix bug in protocomm_nimble chararcteristic access callback

Browse Source 
Fixes bug in `protocomm_nimble` while writing to characteristic with length greater than MTU value.
This commit is contained in:
Prasad Alatkar 2019-08-15 10:50:30 +08:00 committed by Angus Gratton
parent 0781868c36
commit 4c7b83defc
1 changed files with 25 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
components/protocomm/src/transports/protocomm_nimble.c
View File

@ -287,7 +287,7 @@ gatt_svr_dsc_access(uint16_t conn_handle, uint16_t attr_handle, struct
char *temp_outbuf = strdup(ctxt->dsc->arg); char *temp_outbuf = strdup(ctxt->dsc->arg);
if (temp_outbuf == NULL) { if (temp_outbuf == NULL) {
ESP_LOGE(TAG, "Error duplicating user description of characteristic"); ESP_LOGE(TAG, "Error duplicating user description of characteristic");
return ESP_ERR_NO_MEM; return BLE_ATT_ERR_INSUFFICIENT_RES;
} }
ssize_t temp_outlen = strlen(temp_outbuf); ssize_t temp_outlen = strlen(temp_outbuf);
@ -308,6 +308,9 @@ gatt_svr_chr_access(uint16_t conn_handle, uint16_t attr_handle,
ssize_t temp_outlen = 0; ssize_t temp_outlen = 0;
uint8_t *temp_outbuf = NULL; uint8_t *temp_outbuf = NULL;
uint8_t *uuid = NULL; uint8_t *uuid = NULL;
uint8_t *data_buf = NULL;
uint16_t data_len = 0;
uint16_t data_buf_len = 0;
switch (ctxt->op) { switch (ctxt->op) {
case BLE_GATT_ACCESS_OP_READ_CHR: case BLE_GATT_ACCESS_OP_READ_CHR:
@ -328,7 +331,7 @@ gatt_svr_chr_access(uint16_t conn_handle, uint16_t attr_handle,
uuid = (uint8_t *) calloc(BLE_UUID128_VAL_LENGTH, sizeof(uint8_t)); uuid = (uint8_t *) calloc(BLE_UUID128_VAL_LENGTH, sizeof(uint8_t));
if (!uuid) { if (!uuid) {
ESP_LOGE(TAG, "Error allocating memory for 128 bit UUID"); ESP_LOGE(TAG, "Error allocating memory for 128 bit UUID");
return ESP_ERR_NO_MEM; return BLE_ATT_ERR_INSUFFICIENT_RES;
} }
rc = ble_uuid_flat(ctxt->chr->uuid, uuid); rc = ble_uuid_flat(ctxt->chr->uuid, uuid);
@ -338,16 +341,32 @@ gatt_svr_chr_access(uint16_t conn_handle, uint16_t attr_handle,
return rc; return rc;
} }
ESP_LOGD(TAG, "Write attempt for uuid = %s, attr_handle = %d, om_len = %d", /* Save the length of entire data */
ble_uuid_to_str(ctxt->chr->uuid, buf), attr_handle, ctxt->om->om_len); data_len = OS_MBUF_PKTLEN(ctxt->om);
ESP_LOGD(TAG, "Write attempt for uuid = %s, attr_handle = %d, data_len = %d",
ble_uuid_to_str(ctxt->chr->uuid, buf), attr_handle, data_len);
data_buf = calloc(1, data_len);
if (data_buf == NULL) {
ESP_LOGE(TAG, "Error allocating memory for characteristic value");
return BLE_ATT_ERR_INSUFFICIENT_RES;
}
rc = ble_hs_mbuf_to_flat(ctxt->om, data_buf, data_len, &data_buf_len);
if (rc != 0) {
ESP_LOGE(TAG, "Error getting data from memory buffers");
return BLE_ATT_ERR_UNLIKELY;
}
ret = protocomm_req_handle(protoble_internal->pc_ble, ret = protocomm_req_handle(protoble_internal->pc_ble,
uuid128_to_handler(uuid), uuid128_to_handler(uuid),
conn_handle, conn_handle,
ctxt->om->om_data, data_buf,
ctxt->om->om_len, data_buf_len,
&temp_outbuf, &temp_outlen); &temp_outbuf, &temp_outlen);
/* Release the 16 bytes allocated for uuid*/ /* Release the 16 bytes allocated for uuid*/
free(uuid); free(uuid);
free(data_buf);
if (ret == ESP_OK) { if (ret == ESP_OK) {
/* Save data address and length outbuf and outlen internally */ /* Save data address and length outbuf and outlen internally */