Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f6baf153 authored by Dan Carpenter's avatar Dan Carpenter Committed by John W. Linville
Browse files

ath6kl: small cleanup in ath6kl_htc_pipe_rx_complete() 



It's harmless, but Smatch complains if we use "htc_hdr->eid" before
doing the bounds check.

Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 6f56b06e

drivers/net/wireless/ath/ath6kl/htc_pipe.c

+1 −2
Original line number Diff line number Diff line
@@ -988,8 +988,6 @@ static int ath6kl_htc_pipe_rx_complete(struct ath6kl *ar, struct sk_buff *skb, 


	htc_hdr = (struct htc_frame_hdr *) netdata;



	ep = &target->endpoint[htc_hdr->eid];



	if (htc_hdr->eid >= ENDPOINT_MAX) {

		ath6kl_dbg(ATH6KL_DBG_HTC,

			   "HTC Rx: invalid EndpointID=%d\n",
@@ -997,6 +995,7 @@ static int ath6kl_htc_pipe_rx_complete(struct ath6kl *ar, struct sk_buff *skb, 
		status = -EINVAL;

		goto free_skb;

	}

	ep = &target->endpoint[htc_hdr->eid];



	payload_len = le16_to_cpu(get_unaligned(&htc_hdr->payld_len));