Bluetooth: Check req->err in hci_req_add
authorAndre Guedes <andre.guedes@openbossa.org>
Fri, 8 Mar 2013 14:20:18 +0000 (11:20 -0300)
committerGustavo Padovan <gustavo.padovan@collabora.co.uk>
Sat, 9 Mar 2013 20:11:26 +0000 (17:11 -0300)
If req->err is set, there is no point in queueing the HCI command
in HCI request command queue since it won't be sent anyway.

Signed-off-by: Andre Guedes <andre.guedes@openbossa.org>
Acked-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
net/bluetooth/hci_core.c

index 1c678757c83a10940c25fb6d8a19279e2e53feeb..02070dcdfbbb6bfbdf981931352eba089fb1d962 100644 (file)
@@ -2536,6 +2536,12 @@ void hci_req_add(struct hci_request *req, u16 opcode, u32 plen, void *param)
 
        BT_DBG("%s opcode 0x%4.4x plen %d", hdev->name, opcode, plen);
 
+       /* If an error occured during request building, there is no point in
+        * queueing the HCI command. We can simply return.
+        */
+       if (req->err)
+               return;
+
        skb = hci_prepare_cmd(hdev, opcode, plen, param);
        if (!skb) {
                BT_ERR("%s no memory for command (opcode 0x%4.4x)",