diff options
| author | Johan Hedberg | 2015-10-16 09:07:52 +0200 |
|---|---|---|
| committer | Marcel Holtmann | 2015-10-16 09:24:41 +0200 |
| commit | 679d2b6f9d742b3f091868bd9a0634647ce7e782 (patch) | |
| tree | 187fa4a6c496d44748e751cb51e47222a7d644bc /net/bluetooth | |
| parent | Bluetooth: Fix LE reconnection logic (diff) | |
| download | kernel-qcow2-linux-679d2b6f9d742b3f091868bd9a0634647ce7e782.tar.gz kernel-qcow2-linux-679d2b6f9d742b3f091868bd9a0634647ce7e782.tar.xz kernel-qcow2-linux-679d2b6f9d742b3f091868bd9a0634647ce7e782.zip | |
Bluetooth: Fix remove_device behavior for explicit connects
Devices undergoing an explicit connect should not have their
conn_params struct removed by the mgmt Remove Device command. This
patch fixes the necessary checks in the command handler to correct the
behavior.
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'net/bluetooth')
| -rw-r--r-- | net/bluetooth/mgmt.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c index 4dbfe01546b3..0ed94e6f4de9 100644 --- a/net/bluetooth/mgmt.c +++ b/net/bluetooth/mgmt.c @@ -6386,7 +6386,8 @@ static int remove_device(struct sock *sk, struct hci_dev *hdev, goto unlock; } - if (params->auto_connect == HCI_AUTO_CONN_DISABLED) { + if (params->auto_connect == HCI_AUTO_CONN_DISABLED || + params->auto_connect == HCI_AUTO_CONN_EXPLICIT) { err = cmd->cmd_complete(cmd, MGMT_STATUS_INVALID_PARAMS); mgmt_pending_remove(cmd); @@ -6422,6 +6423,10 @@ static int remove_device(struct sock *sk, struct hci_dev *hdev, if (p->auto_connect == HCI_AUTO_CONN_DISABLED) continue; device_removed(sk, hdev, &p->addr, p->addr_type); + if (p->explicit_connect) { + p->auto_connect = HCI_AUTO_CONN_EXPLICIT; + continue; + } list_del(&p->action); list_del(&p->list); kfree(p); |