diff options
| author | Marcel Holtmann <marcel@holtmann.org> | 2013-10-11 07:01:04 -0700 |
|---|---|---|
| committer | Johan Hedberg <johan.hedberg@intel.com> | 2013-10-11 17:00:38 +0200 |
| commit | 602987758b0a3bfb0af5b0931c302b8f552c8d2e (patch) | |
| tree | 0deb79e16190c671df099bd95fbbfc8ec8645841 | |
| parent | 52bc423a7e74f5d1dac3ab61e742831e004b8e39 (diff) | |
| download | lwn-602987758b0a3bfb0af5b0931c302b8f552c8d2e.tar.gz lwn-602987758b0a3bfb0af5b0931c302b8f552c8d2e.zip | |
Bluetooth: Remove pointless parameter check in vhci_send_frame()
The hdev parameter of vhci_send_frame() is always valid. If it were
not valid, then it would have crashed earlier in the call chain.
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
| -rw-r--r-- | drivers/bluetooth/hci_vhci.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/drivers/bluetooth/hci_vhci.c b/drivers/bluetooth/hci_vhci.c index e6f591969d95..7b167385a1c4 100644 --- a/drivers/bluetooth/hci_vhci.c +++ b/drivers/bluetooth/hci_vhci.c @@ -83,18 +83,11 @@ static int vhci_flush(struct hci_dev *hdev) static int vhci_send_frame(struct hci_dev *hdev, struct sk_buff *skb) { - struct vhci_data *data; - - if (!hdev) { - BT_ERR("Frame for unknown HCI device (hdev=NULL)"); - return -ENODEV; - } + struct vhci_data *data = hci_get_drvdata(hdev); if (!test_bit(HCI_RUNNING, &hdev->flags)) return -EBUSY; - data = hci_get_drvdata(hdev); - memcpy(skb_push(skb, 1), &bt_cb(skb)->pkt_type, 1); skb_queue_tail(&data->readq, skb); |