Skip to content

Commit c24e5ed

Browse files
szymon-czaprackisjanc
authored andcommitted
nimble/btp: Fix address assignment in padv operations
In periodic sync procedure assignment was wrong due to device not being connected - this resulted in assert condition trigger. Also during periodic transfer reception we now assign proper address to send to tester. Previous lookup for connection was wrong, due to invalid parameter passed into `ble_gap_conn_find` function.
1 parent c648458 commit c24e5ed

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

apps/bttester/src/btp_gap.c

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1202,17 +1202,11 @@ bond_lost(uint16_t conn_handle)
12021202
static void
12031203
sync_established(struct ble_gap_event *event)
12041204
{
1205-
int rc;
1206-
struct ble_gap_conn_desc desc;
12071205
struct gap_periodic_sync_est_ev ev;
12081206

12091207
ev.status = event->periodic_sync.status;
12101208
ev.sync_handle = event->periodic_sync.sync_handle;
1211-
1212-
rc = ble_gap_conn_find(ev.sync_handle, &desc);
1213-
assert(rc == 0);
1214-
1215-
ev.peer_addr = desc.peer_id_addr;
1209+
ev.peer_addr = event->periodic_sync.adv_addr;
12161210

12171211
tester_event(BTP_SERVICE_ID_GAP, GAP_EV_PERIODIC_SYNC_ESTABLISHED,
12181212
(uint8_t *) &ev, sizeof(ev));
@@ -1261,7 +1255,7 @@ periodic_transfer_received(struct ble_gap_event *event)
12611255
ev.sync_handle = event->periodic_transfer.sync_handle;
12621256
ev.status = event->periodic_transfer.status;
12631257

1264-
rc = ble_gap_conn_find(ev.sync_handle, &desc);
1258+
rc = ble_gap_conn_find(event->periodic_transfer.conn_handle, &desc);
12651259
assert(rc == 0);
12661260

12671261
ev.peer_addr = desc.peer_id_addr;

0 commit comments

Comments
 (0)