espressif/esp-nimble
1
0
Fork 0
mirror of https://github.com/espressif/esp-nimble.git synced 2026-09-10 19:20:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

nimble/ll: Fix HCI_Host_Buffer_Size

Browse Source 
Any non-zero value for num ACL packets and packet length is valid. The
quoted text from spec referrerd to command and event packets, not ACL
packets.
This commit is contained in:
Andrzej Kaczmarek
2023-04-17 13:12:52 +02:00
parent 465de20085
commit d8dfb16991
1 changed files with 1 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
nimble/controller/src/ble_ll_hci.c
+1 -9
View File
@@ -1464,17 +1464,9 @@ ble_ll_hci_cb_host_buf_size(const uint8_t *cmdbuf, uint8_t len)
return BLE_ERR_INV_HCI_CMD_PARMS;
}
/*
* Core 5.2 Vol 4 Part E section 7.3.39 states that "Both the Host and the
* Controller shall support command and event packets, where the data portion
* (excluding header) contained in the packets is 255 octets in size.".
* This means we can basically accept any allowed value since LL does not
* reassemble incoming data thus will not send more than 255 octets in single
* data packet.
*/
acl_num = le16toh(cmd->acl_num);
acl_data_len = le16toh(cmd->acl_data_len);
if (acl_data_len < 255) {
if ((acl_num < 1) || (acl_data_len < 1)) {
return BLE_ERR_INV_HCI_CMD_PARMS;
}