diff options
| author |  Abhishek Pandit-Subedi <abhishekpandit@chromium.org>
| 2022-07-25 15:34:21 -0700 |
|---|---|---|
| committer |  Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
| 2022-07-26 13:35:13 -0700 |
| commit | ef61b6ea154464fefd8a6712d7a3b43b445c3d4a (patch) | |
| tree | 38ed3eb7fa840652dd1defceddd99e86e0ae4f69 /net/bluetooth | |
| parent | Bluetooth: mgmt: Fix double free on error path (diff) | |
| download | linux-ef61b6ea154464fefd8a6712d7a3b43b445c3d4a.tar.gz linux-ef61b6ea154464fefd8a6712d7a3b43b445c3d4a.tar.bz2 linux-ef61b6ea154464fefd8a6712d7a3b43b445c3d4a.zip | |
Bluetooth: Always set event mask on suspend
When suspending, always set the event mask once disconnects are
successful. Otherwise, if wakeup is disallowed, the event mask is not
set before suspend continues and can result in an early wakeup.
Fixes: 182ee45da083 ("Bluetooth: hci_sync: Rework hci_suspend_notifier")
Cc: stable@vger.kernel.org
Signed-off-by: Abhishek Pandit-Subedi <abhishekpandit@chromium.org>
Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
Diffstat (limited to 'net/bluetooth')
| -rw-r--r-- | net/bluetooth/hci_sync.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/net/bluetooth/hci_sync.c b/net/bluetooth/hci_sync.c index 1739e8cb3291..c17021642234 100644 --- a/net/bluetooth/hci_sync.c +++ b/net/bluetooth/hci_sync.c @@ -4973,6 +4973,9 @@ int hci_suspend_sync(struct hci_dev *hdev) return err; } + /* Update event mask so only the allowed event can wakeup the host */ + hci_set_event_mask_sync(hdev); + /* Only configure accept list if disconnect succeeded and wake * isn't being prevented. */ @@ -4984,9 +4987,6 @@ int hci_suspend_sync(struct hci_dev *hdev) /* Unpause to take care of updating scanning params */ hdev->scanning_paused = false; - /* Update event mask so only the allowed event can wakeup the host */ - hci_set_event_mask_sync(hdev); - /* Enable event filter for paired devices */ hci_update_event_filter_sync(hdev); |