michaelh/zephyr
1
0
Fork 0
Code Releases Activity

Bluetooth: Fix clearing advertising & scanning parameters

Browse source 
The parameter structs for advertising and scanning contain many
members that may not get explicitly set when enabling these states. Do
a memset to zero on them to make sure we don't operate on
uninitialized memory.

Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
This commit is contained in:
Johan Hedberg 2017-11-12 21:15:04 +02:00 • committed by Johan Hedberg
commit d43a92b1c8
1 changed files with 4 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

4
subsys/bluetooth/host/hci_core.c
View file

@ -2891,6 +2891,8 @@ static int start_le_scan(u8_t scan_type, u16_t interval, u16_t window)
struct net_buf *buf;
int err;
memset(&set_param, 0, sizeof(set_param));
set_param.scan_type = scan_type;
/* for the rest parameters apply default values according to
@ -4646,6 +4648,8 @@ int bt_le_adv_start(const struct bt_le_adv_param *param,
}
}
memset(&set_param, 0, sizeof(set_param));
set_param.min_interval = sys_cpu_to_le16(param->interval_min);
set_param.max_interval = sys_cpu_to_le16(param->interval_max);
set_param.channel_map = 0x07;