Bluetooth: btqca: remove null checks on edl->data as it is an array
authorColin Ian King <colin.king@canonical.com>
Tue, 6 Sep 2016 12:15:49 +0000 (13:15 +0100)
committerMarcel Holtmann <marcel@holtmann.org>
Mon, 19 Sep 2016 18:19:34 +0000 (20:19 +0200)
edl->data is an array of __u8 so the null check is unneccessary,
so remove it.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
drivers/bluetooth/btqca.c

index 4a62081..28afd5d 100644 (file)
@@ -55,8 +55,8 @@ static int rome_patch_ver_req(struct hci_dev *hdev, u32 *rome_version)
        }
 
        edl = (struct edl_event_hdr *)(skb->data);
-       if (!edl || !edl->data) {
-               BT_ERR("%s: TLV with no header or no data", hdev->name);
+       if (!edl) {
+               BT_ERR("%s: TLV with no header", hdev->name);
                err = -EILSEQ;
                goto out;
        }
@@ -224,8 +224,8 @@ static int rome_tlv_send_segment(struct hci_dev *hdev, int idx, int seg_size,
        }
 
        edl = (struct edl_event_hdr *)(skb->data);
-       if (!edl || !edl->data) {
-               BT_ERR("%s: TLV with no header or no data", hdev->name);
+       if (!edl) {
+               BT_ERR("%s: TLV with no header", hdev->name);
                err = -EILSEQ;
                goto out;
        }