Bluetooth: Fix using test_and_clear instead of test_and_set

The code for updating the HCI_CONNECTABLE flag was incorrectly using
test_and_set_bit instead of test_and_clear_bit when HCI_CONNECTABLE is
to be cleared.

Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>

diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c
index 84431b8..8f9df76 100644 (file)
--- a/net/bluetooth/hci_core.c
+++ b/net/bluetooth/hci_core.c
@@ -2726,8 +2726,8 @@ int hci_dev_cmd(unsigned int cmd, void __user *arg)
                                changed = !test_and_set_bit(HCI_CONNECTABLE,
                                                            &hdev->dev_flags);
                        else
-                               changed = test_and_set_bit(HCI_CONNECTABLE,
-                                                          &hdev->dev_flags);
+                               changed = test_and_clear_bit(HCI_CONNECTABLE,
+                                                            &hdev->dev_flags);
 
                        if (changed)
                                mgmt_new_settings(hdev);