You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mynewt.apache.org by ry...@apache.org on 2019/01/08 13:11:24 UTC

[mynewt-nimble] 02/06: nimble/ll: Fix extended advertising report event length

This is an automated email from the ASF dual-hosted git repository.

rymek pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mynewt-nimble.git

commit 4213e93c0c4d76618fce344cc2d7ab85a2f09fa4
Author: Ɓukasz Rymanowski <lu...@codecoup.pl>
AuthorDate: Thu Jan 3 12:00:33 2019 +0100

    nimble/ll: Fix extended advertising report event length
    
    `struct ble_ll_ext_adv_report` contains event meta code and event
    length (HCI event header) which shall not be calculated into event length.
    This patch fixes it.
---
 nimble/controller/src/ble_ll_scan.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/nimble/controller/src/ble_ll_scan.c b/nimble/controller/src/ble_ll_scan.c
index 0a16060..6498bda 100644
--- a/nimble/controller/src/ble_ll_scan.c
+++ b/nimble/controller/src/ble_ll_scan.c
@@ -471,7 +471,7 @@ ble_ll_scan_send_truncated_if_chained(struct ble_ll_aux_data *aux_data)
         }
     }
 
-    evt->event_len = sizeof(*evt);
+    evt->event_len = sizeof(*evt) - BLE_HCI_EVENT_HDR_LEN;
     evt->evt_type = aux_data->evt_type;
     evt->evt_type |= (BLE_HCI_ADV_DATA_STATUS_TRUNCATED);
     BLE_LL_SET_AUX_FLAG(aux_data, BLE_LL_AUX_TRUNCATED_SENT);
@@ -748,7 +748,7 @@ ble_ll_hci_send_legacy_ext_adv_report(uint8_t evtype,
     evt->addr_type = addr_type;
     memcpy(evt->addr, addr, BLE_DEV_ADDR_LEN);
 
-    evt->event_len = sizeof(*evt);
+    evt->event_len = sizeof(*evt) - BLE_HCI_EVENT_HDR_LEN;
 
     if (inita) {
         evt->dir_addr_type = inita_type;