[4.4,19/64] ath9k_htc: Use appropriate rs_datalen type

Message ID	20201117122107.077116163@linuxfoundation.org
State	Superseded
Headers	show Return-Path: <stable-owner@kernel.org> From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, stable@vger.kernel.org, kernel test robot <lkp@intel.com>, Masashi Honma <masashi.honma@gmail.com>, Kalle Valo <kvalo@codeaurora.org> Subject: [PATCH 4.4 19/64] ath9k_htc: Use appropriate rs_datalen type Date: Tue, 17 Nov 2020 14:04:42 +0100 Message-Id: <20201117122107.077116163@linuxfoundation.org> In-Reply-To: <20201117122106.144800239@linuxfoundation.org> References: <20201117122106.144800239@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	None \| expand [4.4,03/64] mm: mempolicy: fix potential pte_unmap_unlock pte error [4.4,04/64] time: Prevent undefined behaviour in timespec64_to_ns() [4.4,08/64] ALSA: hda: prevent undefined shift in snd_hdac_ext_bus_get_link() [4.4,11/64] can: can_create_echo_skb(): fix echo skb generation: always use skb_clone() [4.4,13/64] can: peak_usb: peak_usb_get_ts_time(): fix timestamp wrapping [4.4,15/64] pinctrl: devicetree: Avoid taking direct reference to device name string [4.4,17/64] i40e: Fix of memory leak and integer truncation in i40e_virtchnl.c [4.4,19/64] ath9k_htc: Use appropriate rs_datalen type [4.4,20/64] usb: gadget: goku_udc: fix potential crashes in probe [4.4,21/64] gfs2: Free rd_bits later in gfs2_clear_rgrpd to fix use-after-free [4.4,22/64] gfs2: check for live vs. read-only file system in gfs2_fitrim [4.4,26/64] cfg80211: regulatory: Fix inconsistent format argument [4.4,28/64] xfs: fix a missing unlock on error in xfs_fs_map_blocks [4.4,29/64] of/address: Fix of_node memory leak in of_dma_is_coherent [4.4,32/64] ext4: correctly report "not supported" for {usr,grp}jquota when !CONFIG_QUOTA [4.4,33/64] ext4: unlock xattr_sem properly in ext4_inline_data_truncate() [4.4,36/64] ocfs2: initialize ip_next_orphan [4.4,37/64] dont dump the threads that had been already exiting when zapped. [4.4,38/64] drm/gma500: Fix out-of-bounds access to struct drm_device.vblank[] [4.4,40/64] pinctrl: amd: fix incorrect way to disable debounce filter [4.4,44/64] net/x25: Fix null-ptr-deref in x25_connect [4.4,46/64] random32: make prandom_u32() output unpredictable [4.4,47/64] x86/speculation: Allow IBPB to be conditionally enabled on CPUs with always-on STIBP [4.4,49/64] xen/events: add a proper barrier to 2-level uevent unmasking [4.4,50/64] xen/events: fix race in evtchn_fifo_unmask() [4.4,53/64] xen/netback: use lateeoi irq binding [4.4,54/64] xen/scsiback: use lateeoi irq binding [4.4,55/64] xen/pciback: use lateeoi irq binding [4.4,59/64] xen/events: block rogue events for some time [4.4,60/64] perf/core: Fix race in the perf_mmap_close() function [4.4,61/64] Revert "kernel/reboot.c: convert simple_strtoul to kstrtoint" [4.4,64/64] Convert trailing spaces and periods in path components

Message ID

20201117122107.077116163@linuxfoundation.org

State

Superseded

Headers

From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: linux-kernel@vger.kernel.org
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	stable@vger.kernel.org, kernel test robot <lkp@intel.com>,
	Masashi Honma <masashi.honma@gmail.com>,
	Kalle Valo <kvalo@codeaurora.org>
Subject: [PATCH 4.4 19/64] ath9k_htc: Use appropriate rs_datalen type
Date: Tue, 17 Nov 2020 14:04:42 +0100
Message-Id: <20201117122107.077116163@linuxfoundation.org>
In-Reply-To: <20201117122106.144800239@linuxfoundation.org>
References: <20201117122106.144800239@linuxfoundation.org>
User-Agent: quilt/0.66
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Precedence: bulk

Series

None | expand

Commit Message

Greg KH Nov. 17, 2020, 1:04 p.m. UTC

From: Masashi Honma <masashi.honma@gmail.com>

commit 5024f21c159f8c1668f581fff37140741c0b1ba9 upstream.

kernel test robot says:
drivers/net/wireless/ath/ath9k/htc_drv_txrx.c:987:20: sparse: warning: incorrect type in assignment (different base types)
drivers/net/wireless/ath/ath9k/htc_drv_txrx.c:987:20: sparse:    expected restricted __be16 [usertype] rs_datalen
drivers/net/wireless/ath/ath9k/htc_drv_txrx.c:987:20: sparse:    got unsigned short [usertype]
drivers/net/wireless/ath/ath9k/htc_drv_txrx.c:988:13: sparse: warning: restricted __be16 degrades to integer
drivers/net/wireless/ath/ath9k/htc_drv_txrx.c:1001:13: sparse: warning: restricted __be16 degrades to integer

Indeed rs_datalen has host byte order, so modify it's own type.

Reported-by: kernel test robot <lkp@intel.com>
Fixes: cd486e627e67 ("ath9k_htc: Discard undersized packets")
Signed-off-by: Masashi Honma <masashi.honma@gmail.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20200808233258.4596-1-masashi.honma@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

---
 drivers/net/wireless/ath/ath9k/htc_drv_txrx.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- a/drivers/net/wireless/ath/ath9k/htc_drv_txrx.c
+++ b/drivers/net/wireless/ath/ath9k/htc_drv_txrx.c
@@ -972,7 +972,7 @@  static bool ath9k_rx_prepare(struct ath9
 	struct ath_htc_rx_status *rxstatus;
 	struct ath_rx_status rx_stats;
 	bool decrypt_error = false;
-	__be16 rs_datalen;
+	u16 rs_datalen;
 	bool is_phyerr;
 
 	if (skb->len < HTC_RX_FRAME_HEADER_SIZE) {

[4.4,19/64] ath9k_htc: Use appropriate rs_datalen type

Commit Message

Patch