Bluetooth: Fix mgmt response status when removing adapter
authorJohan Hedberg <johan.hedberg@intel.com>
Fri, 12 Dec 2014 09:15:21 +0000 (11:15 +0200)
committerMarcel Holtmann <marcel@holtmann.org>
Fri, 12 Dec 2014 12:20:12 +0000 (13:20 +0100)
commit9845904fd489288bcf693642c1b31cc463c0b660
treeb7964b4e15df38eb53a91c4850f41e3289dd71d3
parentec6f99b807e4bd50566c48fff8994da2fb1bf9fe
Bluetooth: Fix mgmt response status when removing adapter

When an adapter is removed (hci_unregister_dev) any pending mgmt
commands for that adapter should get the appropriate INVALID_INDEX
response. Since hci_unregister_dev() calls hci_dev_do_close() first
that'd so far have caused "not powered" responses to be sent.

Skipping the HCI_UNREGISTER case in mgmt_powered() is also not a
solution since before reaching the mgmt_index_removed() stage any
hci_conn callbacks (e.g. used by pairing) will get called, thereby
causing "disconnected" status responses to be sent.

The fix that covers all scenarios is to handle both INVALID_INDEX and
NOT_POWERED responses through the mgmt_powered() function. The
INVALID_INDEX response sending from mgmt_index_removed() is left
untouched since there are a couple of places not related to powering off
or removing an adapter that call it (e.g. configuring a new bdaddr).

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