michaelh/zephyr
1
0
Fork 0
Code Releases Activity

Bluetooth: shell: Fixed pointer in cmd_per_adv_sync_create

Browse source 
The `free_per_adv_sync` pointer in `cmd_per_adv_sync_create`
did not work as intended, as it didn't actually update the
pointer in the array, which caused the
`cmd_per_adv_sync_delete` to not function as intended.

Signed-off-by: Emil Gydesen <emil_gydesen@bose.com>
This commit is contained in:
Emil Gydesen 2020-08-16 23:04:43 +02:00 • committed by Anas Nashif
commit 6cc0487c46
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
subsys/bluetooth/shell/bt.c
View file

@ -1433,12 +1433,12 @@ static int cmd_per_adv_sync_create(const struct shell *shell, size_t argc,
int err; int err;
struct bt_le_per_adv_sync_param create_params = { 0 }; struct bt_le_per_adv_sync_param create_params = { 0 };
uint32_t options = 0; uint32_t options = 0;
struct bt_le_per_adv_sync *free_per_adv_sync = NULL; struct bt_le_per_adv_sync **free_per_adv_sync = NULL;
int i = 0; int i = 0;
for (i = 0; i < ARRAY_SIZE(per_adv_syncs); i++) { for (i = 0; i < ARRAY_SIZE(per_adv_syncs); i++) {
if (per_adv_syncs[i] == NULL) { if (per_adv_syncs[i] == NULL) {
free_per_adv_sync = per_adv_syncs[i]; free_per_adv_sync = &per_adv_syncs[i];
break; break;
} }
} }
@ -1495,7 +1495,7 @@ static int cmd_per_adv_sync_create(const struct shell *shell, size_t argc,
create_params.options = options; create_params.options = options;
err = bt_le_per_adv_sync_create(&create_params, &per_adv_sync_cb, err = bt_le_per_adv_sync_create(&create_params, &per_adv_sync_cb,
&free_per_adv_sync); free_per_adv_sync);
if (err) { if (err) {
shell_error(shell, "Per adv sync failed (%d)", err); shell_error(shell, "Per adv sync failed (%d)", err);
} else { } else {