diff options
| author | Marcel Holtmann <marcel@holtmann.org> | 2013-10-14 09:55:32 -0700 |
|---|---|---|
| committer | Johan Hedberg <johan.hedberg@intel.com> | 2013-10-14 21:35:47 +0300 |
| commit | 899e107577a8de67b97a877ed90bbfbfefb1916e (patch) | |
| tree | 41f80f4a002967bc6d4f5750bd652e4cf439e708 /net/bluetooth/mgmt.c | |
| parent | 1f209383f2be94e15e179da3a4cbcfeabcc2efc2 (diff) | |
| download | op-kernel-dev-899e107577a8de67b97a877ed90bbfbfefb1916e.zip op-kernel-dev-899e107577a8de67b97a877ed90bbfbfefb1916e.tar.gz | |
Bluetooth: Check that scan window is smaller or equal than scan interval
The scan window parameter for connection establishment and passive
scanning needs to be smaller or equal than the scan interval.
Instead of waiting for a controller to reject these values later on,
just reject them right away.
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Diffstat (limited to 'net/bluetooth/mgmt.c')
| -rw-r--r-- | net/bluetooth/mgmt.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c index 813012f..861e389 100644 --- a/net/bluetooth/mgmt.c +++ b/net/bluetooth/mgmt.c @@ -3493,6 +3493,10 @@ static int set_scan_params(struct sock *sk, struct hci_dev *hdev, return cmd_status(sk, hdev->id, MGMT_OP_SET_SCAN_PARAMS, MGMT_STATUS_INVALID_PARAMS); + if (window > interval) + return cmd_status(sk, hdev->id, MGMT_OP_SET_SCAN_PARAMS, + MGMT_STATUS_INVALID_PARAMS); + hci_dev_lock(hdev); hdev->le_scan_interval = interval; |
