diff options
| author | Marcel Holtmann <marcel@holtmann.org> | 2013-10-10 23:54:18 +0200 |
|---|---|---|
| committer | Johan Hedberg <johan.hedberg@intel.com> | 2013-10-11 00:10:11 +0200 |
| commit | 3d386acea090e7af3346de59d9e322f92128a4b1 (patch) | |
| tree | 94c9131baab089e14ba9d8574f67be404ffb1a46 | |
| parent | Bluetooth: Move skb->dev assignment for hdev->send into central place (diff) | |
| download | linux-3d386acea090e7af3346de59d9e322f92128a4b1.tar.xz linux-3d386acea090e7af3346de59d9e322f92128a4b1.zip | |
Bluetooth: Remove pointless check of hci_send_frame parameter
The hdev parameter of hci_send_frame must be always valid. If the hdev
is not valid, it would not even make it to this stage. The callers
will have already accessed hdev at that point many times.
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
| -rw-r--r-- | net/bluetooth/hci_core.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c index 925bd47c4a3c..47cf3a95dac0 100644 --- a/net/bluetooth/hci_core.c +++ b/net/bluetooth/hci_core.c @@ -2699,11 +2699,6 @@ EXPORT_SYMBOL(hci_unregister_cb); static int hci_send_frame(struct hci_dev *hdev, struct sk_buff *skb) { - if (!hdev) { - kfree_skb(skb); - return -ENODEV; - } - BT_DBG("%s type %d len %d", hdev->name, bt_cb(skb)->pkt_type, skb->len); /* Time stamp */ |