[5.4,045/151] ath9k_htc: Use appropriate rs_datalen type

Message ID	20201117122123.619251798@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 5.4 045/151] ath9k_htc: Use appropriate rs_datalen type Date: Tue, 17 Nov 2020 14:04:35 +0100 Message-Id: <20201117122123.619251798@linuxfoundation.org> In-Reply-To: <20201117122121.381905960@linuxfoundation.org> References: <20201117122121.381905960@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 [5.4,002/151] time: Prevent undefined behaviour in timespec64_to_ns() [5.4,004/151] KVM: arm64: Force PTE mapping on fault resulting in a device mapping [5.4,006/151] usb: dwc3: gadget: Continue to process pending requests [5.4,007/151] usb: dwc3: gadget: Reclaim extra TRBs after request completion [5.4,008/151] btrfs: tracepoints: output proper root owner for trace_find_free_extent() [5.4,009/151] btrfs: sysfs: init devices outside of the chunk_mutex [5.4,012/151] genirq: Let GENERIC_IRQ_IPI select IRQ_DOMAIN_HIERARCHY [5.4,015/151] ASoC: codecs: wcd9335: Set digital gain range correctly [5.4,016/151] xfs: set xefi_discard when creating a deferred agfl free log intent item [5.4,017/151] netfilter: use actual socket sk rather than skb sk when routing harder [5.4,018/151] netfilter: nf_tables: missing validation from the abort path [5.4,020/151] powerpc/eeh_cache: Fix a possible debugfs deadlock [5.4,021/151] perf trace: Fix segfault when trying to trace events by cgroup [5.4,024/151] iommu/vt-d: Fix a bug for PDP check in prq_event_thread [5.4,026/151] can: rx-offload: dont call kfree_skb() from IRQ context [5.4,027/151] can: dev: can_get_echo_skb(): prevent call to kfree_skb() in hard IRQ context [5.4,030/151] can: j1939: swap addr and pgn in the send example [5.4,032/151] can: ti_hecc: ti_hecc_probe(): add missed clk_disable_unprepare() in error path [5.4,034/151] can: peak_usb: add range checking in decode operations [5.4,038/151] can: flexcan: flexcan_remove(): disable wakeup completely [5.4,040/151] xfs: fix scrub flagging rtinherit even if there is no rt device [5.4,041/151] tpm: efi: Dont create binary_bios_measurements file for an empty log [5.4,043/151] KVM: arm64: ARM_SMCCC_ARCH_WORKAROUND_1 doesnt return SMCCC_RET_NOT_REQUIRED [5.4,045/151] ath9k_htc: Use appropriate rs_datalen type [5.4,046/151] ASoC: qcom: sdm845: set driver name correctly [5.4,047/151] ASoC: cs42l51: manage mclk shutdown delay [5.4,049/151] opp: Reduce the size of critical section in _opp_table_kref_release() [5.4,051/151] selftests/ftrace: check for do_sys_openat2 in user-memory test [5.4,052/151] selftests: pidfd: fix compilation errors due to wait.h [5.4,053/151] ALSA: hda: Separate runtime and system suspend [5.4,055/151] gfs2: Free rd_bits later in gfs2_clear_rgrpd to fix use-after-free [5.4,057/151] gfs2: check for live vs. read-only file system in gfs2_fitrim [5.4,059/151] drm/amdgpu: perform srbm soft reset always on SDMA resume [5.4,062/151] mac80211: fix use of skb payload instead of header [5.4,063/151] cfg80211: initialize wdev data earlier [5.4,065/151] cfg80211: regulatory: Fix inconsistent format argument [5.4,069/151] nvme: introduce nvme_sync_io_queues [5.4,071/151] nvme-tcp: avoid race between time out and tear down [5.4,072/151] nvme-rdma: avoid repeated request completion [5.4,074/151] iommu/amd: Increase interrupt remapping table limit to 512 entries [5.4,075/151] s390/smp: move rcu_cpu_starting() earlier [5.4,078/151] i2c: mediatek: move dma reset before i2c reset [5.4,079/151] amd/amdgpu: Disable VCN DPG mode for Picasso [5.4,084/151] spi: bcm2835: remove use of uninitialized gpio flags variable [5.4,085/151] tick/common: Touch watchdog in tick_unfreeze() on all CPUs [5.4,087/151] pinctrl: intel: Set default bias in case no particular value given [5.4,088/151] ARM: 9019/1: kprobes: Avoid fortify_panic() when copying optprobe template [5.4,092/151] SUNRPC: Fix general protection fault in trace_rpc_xdr_overflow() [5.4,093/151] bpf: Zero-fill re-used per-cpu map element [5.4,095/151] igc: Fix returning wrong statistics [5.4,096/151] xfs: fix flags argument to rmap lookup when converting shared file rmaps [5.4,098/151] xfs: fix rmap key and record comparison functions [5.4,099/151] xfs: fix brainos in the refcount scrubbers rmap fragment processor [5.4,103/151] cosa: Add missing kfree in error path of cosa_write [5.4,105/151] perf: Fix get_recursion_context() [5.4,108/151] ext4: unlock xattr_sem properly in ext4_inline_data_truncate() [5.4,109/151] btrfs: ref-verify: fix memory leak in btrfs_ref_tree_mod [5.4,111/151] btrfs: dev-replace: fail mount if we dont have replace item with target device [5.4,115/151] uio: Fix use-after-free in uio_unregister_device() [5.4,117/151] xhci: hisilicon: fix refercence leak in xhci_histb_probe [5.4,119/151] mei: protect mei_cl_mtu from null dereference [5.4,121/151] jbd2: fix up sparse warnings in checkpoint code [5.4,123/151] Revert "kernel/reboot.c: convert simple_strtoul to kstrtoint" [5.4,126/151] btrfs: fix potential overflow in cluster_pages_for_defrag on 32bit arch [5.4,127/151] selinux: Fix error return code in sel_ib_pkey_sid_slow() [5.4,128/151] gpio: pcie-idio-24: Fix irq mask when masking [5.4,131/151] mmc: sdhci-of-esdhc: Handle pulse width detection erratum for more SoCs [5.4,132/151] mmc: renesas_sdhi_core: Add missing tmio_mmc_host_free() at remove [5.4,134/151] drm/gma500: Fix out-of-bounds access to struct drm_device.vblank[] [5.4,136/151] pinctrl: amd: fix incorrect way to disable debounce filter [5.4,139/151] net/af_iucv: fix null pointer dereference on shutdown [5.4,140/151] net: udp: fix UDP header access on Fast/frag0 UDP GRO [5.4,141/151] net: Update window_clamp if SOCK_RCVBUF is set [5.4,143/151] tipc: fix memory leak in tipc_topsrv_start() [5.4,144/151] r8169: fix potential skb double free in an error path [5.4,145/151] drm/i915: Correctly set SFC capability for video engines [5.4,147/151] x86/speculation: Allow IBPB to be conditionally enabled on CPUs with always-on STIBP [5.4,149/151] perf/core: Fix race in the perf_mmap_close() function [5.4,150/151] net: sch_generic: fix the missing new qdisc assignment bug

Message ID

20201117122123.619251798@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 5.4 045/151] ath9k_htc: Use appropriate rs_datalen type
Date: Tue, 17 Nov 2020 14:04:35 +0100
Message-Id: <20201117122123.619251798@linuxfoundation.org>
In-Reply-To: <20201117122121.381905960@linuxfoundation.org>
References: <20201117122121.381905960@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
@@ -973,7 +973,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) {

[5.4,045/151] ath9k_htc: Use appropriate rs_datalen type

Commit Message

Patch