Skip to content

Commit 5c73eef

Browse files
fix(esp_wifi): Partially revert NAN event API change from b049856
1 parent 783b80f commit 5c73eef

File tree

2 files changed

+5
-10
lines changed

2 files changed

+5
-10
lines changed

components/esp_wifi/include/esp_wifi_types_generic.h

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1366,7 +1366,7 @@ typedef struct {
13661366
uint32_t reserved_2; /**< Reserved */
13671367
uint8_t ssi_version; /**< Indicates version of SSI in Publish instance, 0 if not available */
13681368
uint16_t ssi_len; /**< Length of service specific info */
1369-
uint8_t *ssi; /**< Service specific info of Publisher */
1369+
uint8_t ssi[]; /**< Service specific info of Publisher */
13701370
} wifi_event_nan_svc_match_t;
13711371

13721372
/**
@@ -1379,7 +1379,7 @@ typedef struct {
13791379
uint32_t reserved_1; /**< Reserved */
13801380
uint32_t reserved_2; /**< Reserved */
13811381
uint16_t ssi_len; /**< Length of service specific info */
1382-
uint8_t *ssi; /**< Service specific info of Subscriber */
1382+
uint8_t ssi[]; /**< Service specific info of Subscriber */
13831383
} wifi_event_nan_replied_t;
13841384

13851385
/**
@@ -1392,7 +1392,7 @@ typedef struct {
13921392
uint32_t reserved_1; /**< Reserved */
13931393
uint32_t reserved_2; /**< Reserved */
13941394
uint16_t ssi_len; /**< Length of service specific info */
1395-
uint8_t *ssi; /**< Service specific info from Follow-up */
1395+
uint8_t ssi[]; /**< Service specific info from Follow-up */
13961396
} wifi_event_nan_receive_t;
13971397

13981398
/**
@@ -1406,7 +1406,7 @@ typedef struct {
14061406
uint32_t reserved_1; /**< Reserved */
14071407
uint32_t reserved_2; /**< Reserved */
14081408
uint16_t ssi_len; /**< Length of service specific info */
1409-
uint8_t *ssi; /**< Service specific info from NDP/NDPE Attribute */
1409+
uint8_t ssi[]; /**< Service specific info from NDP/NDPE Attribute */
14101410
} wifi_event_ndp_indication_t;
14111411

14121412
/**
@@ -1421,7 +1421,7 @@ typedef struct {
14211421
uint32_t reserved_1; /**< Reserved */
14221422
uint32_t reserved_2; /**< Reserved */
14231423
uint16_t ssi_len; /**< Length of Service Specific Info */
1424-
uint8_t *ssi; /**< Service specific info from NDP/NDPE Attribute */
1424+
uint8_t ssi[]; /**< Service specific info from NDP/NDPE Attribute */
14251425
} wifi_event_ndp_confirm_t;
14261426

14271427
/**

components/esp_wifi/wifi_apps/nan_app/src/nan_app.c

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -474,7 +474,6 @@ void nan_app_service_match_cb(uint8_t sub_id, uint8_t pub_id, uint8_t pub_mac[6]
474474
if (ssi_ver) {
475475
evt->ssi_version = ssi_ver;
476476
}
477-
evt->ssi = (uint8_t *)(evt + 1);
478477
memcpy(evt->ssi, ssi, ssi_len);
479478
evt->ssi_len = ssi_len;
480479
ESP_LOG_BUFFER_HEXDUMP(TAG, ssi, ssi_len, ESP_LOG_DEBUG);
@@ -508,7 +507,6 @@ void nan_app_replied_cb(uint8_t pub_id, uint8_t sub_id, uint8_t sub_nmi[6], uint
508507

509508
ESP_LOGI(TAG, "Sent Publish to Peer "MACSTR" [Peer Subscribe id - %d]", MAC2STR(sub_nmi), sub_id);
510509
if (ssi && ssi_len) {
511-
evt->ssi = (uint8_t *)(evt + 1);
512510
memcpy(evt->ssi, ssi, ssi_len);
513511
evt->ssi_len = ssi_len;
514512
ESP_LOG_BUFFER_HEXDUMP(TAG, ssi, ssi_len, ESP_LOG_DEBUG);
@@ -537,7 +535,6 @@ void nan_app_receive_cb(uint8_t svc_id, uint8_t peer_svc_id, uint8_t peer_mac[6]
537535
evt->peer_inst_id = peer_svc_id;
538536
MACADDR_COPY(evt->peer_if_mac, peer_mac);
539537
if (ssi && ssi_len) {
540-
evt->ssi = (uint8_t *)(evt + 1);
541538
memcpy(evt->ssi, ssi, ssi_len);
542539
evt->ssi_len = ssi_len;
543540
ESP_LOGD(TAG, "Received payload from Peer "MACSTR" [Peer Service id - %d] - ", MAC2STR(peer_mac), peer_svc_id);
@@ -593,7 +590,6 @@ void nan_app_ndp_indication_cb(uint8_t pub_id, uint8_t ndp_id, uint8_t peer_nmi[
593590
MACADDR_COPY(evt->peer_nmi, peer_nmi);
594591
MACADDR_COPY(evt->peer_ndi, peer_ndi);
595592
if (ssi && ssi_len) {
596-
evt->ssi = (uint8_t *)(evt + 1);
597593
memcpy(evt->ssi, ssi, ssi_len);
598594
evt->ssi_len = ssi_len;
599595
ESP_LOG_BUFFER_HEXDUMP(TAG, ssi, ssi_len, ESP_LOG_DEBUG);
@@ -653,7 +649,6 @@ void nan_app_ndp_confirm_cb(uint8_t status, uint8_t ndp_id, uint8_t peer_nmi[6],
653649
MACADDR_COPY(evt->peer_ndi, peer_ndi);
654650
MACADDR_COPY(evt->own_ndi, own_ndi);
655651
if (ssi && ssi_len) {
656-
evt->ssi = (uint8_t *)(evt + 1);
657652
memcpy(evt->ssi, ssi, ssi_len);
658653
evt->ssi_len = ssi_len;
659654
ESP_LOG_BUFFER_HEXDUMP(TAG, ssi, ssi_len, ESP_LOG_DEBUG);

0 commit comments

Comments
 (0)