Commit 60298775 authored by Marcel Holtmann's avatar Marcel Holtmann Committed by Johan Hedberg
Browse files

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: default avatarMarcel Holtmann <marcel@holtmann.org>
Signed-off-by: default avatarJohan Hedberg <johan.hedberg@intel.com>
parent 52bc423a

drivers/bluetooth/hci_vhci.c

+1 −8
Original line number Diff line number Diff line
@@ -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);