diff mbox series

[5.10,40/48] usb: mtu3: fix list_head check warning

Message ID	20220103142054.832901817@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, Yuwen Ng <yuwen.ng@mediatek.com>, Chunfeng Yun <chunfeng.yun@mediatek.com> Subject: [PATCH 5.10 40/48] usb: mtu3: fix list_head check warning Date: Mon, 3 Jan 2022 15:24:17 +0100 Message-Id: <20220103142054.832901817@linuxfoundation.org> In-Reply-To: <20220103142053.466768714@linuxfoundation.org> References: <20220103142053.466768714@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.10,02/48] Input: i8042 - enable deferred probe quirk for ASUS UM325UA [5.10,03/48] tomoyo: Check exceeded quota early in tomoyo_domain_quota_is_ok(). [5.10,04/48] tomoyo: use hwight16() in tomoyo_domain_quota_is_ok() [5.10,05/48] parisc: Clear stale IIR value on instruction access rights trap [5.10,06/48] platform/x86: apple-gmux: use resource_size() with res [5.10,07/48] memblock: fix memblock_phys_alloc() section mismatch error [5.10,08/48] recordmcount.pl: fix typo in s390 mcount regex [5.10,09/48] selinux: initialize proto variable in selinux_ip_postroute_compat() [5.10,10/48] scsi: lpfc: Terminate string in lpfc_debugfs_nvmeio_trc_write() [5.10,11/48] net/mlx5: DR, Fix NULL vs IS_ERR checking in dr_domain_init_resources [5.10,12/48] net/mlx5e: Wrap the tx reporter dump callback to extract the sq [5.10,13/48] net/mlx5e: Fix ICOSQ recovery flow for XSK [5.10,14/48] udp: using datalen to cap ipv6 udp max gso segments [5.10,15/48] selftests: Calculate udpgso segment count without header adjustment [5.10,16/48] net: phy: fixed_phy: Fix NULL vs IS_ERR() checking in __fixed_phy_register [5.10,17/48] sctp: use call_rcu to free endpoint [5.10,18/48] net/smc: fix using of uninitialized completions [5.10,19/48] net: usb: pegasus: Do not drop long Ethernet frames [5.10,20/48] net: ag71xx: Fix a potential double free in error handling paths [5.10,21/48] net: lantiq_xrx200: fix statistics of received bytes [5.10,22/48] NFC: st21nfca: Fix memory leak in device probe and remove [5.10,23/48] net/smc: improved fix wait on already cleared link [5.10,24/48] net/smc: dont send CDC/LLC message if link not ready [5.10,25/48] net/smc: fix kernel panic caused by race of smc_sock [5.10,26/48] igc: Fix TX timestamp support for non-MSI-X platforms [5.10,27/48] ionic: Initialize the lif->dbid_inuse bitmap [5.10,28/48] net/mlx5e: Fix wrong features assignment in case of error [5.10,29/48] selftests/net: udpgso_bench_tx: fix dst ip argument [5.10,30/48] net/ncsi: check for error return from call to nla_put_u32 [5.10,31/48] fsl/fman: Fix missing put_device() call in fman_port_probe [5.10,32/48] i2c: validate user data in compat ioctl [5.10,33/48] nfc: uapi: use kernel size_t to fix user-space builds [5.10,34/48] uapi: fix linux/nfc.h userspace compilation errors [5.10,35/48] drm/amdgpu: When the VCN(1.0) block is suspended, powergating is explicitly enabled [5.10,36/48] drm/amdgpu: add support for IP discovery gc_info table v2 [5.10,37/48] xhci: Fresco FL1100 controller should not have BROKEN_MSI quirk set. [5.10,38/48] usb: gadget: f_fs: Clear ffs_eventfd in ffs_data_clear. [5.10,39/48] usb: mtu3: add memory barrier before set GPDs HWO [5.10,40/48] usb: mtu3: fix list_head check warning [5.10,41/48] usb: mtu3: set interval of FS intr and isoc endpoint [5.10,42/48] binder: fix async_free_space accounting for empty parcels [5.10,43/48] scsi: vmw_pvscsi: Set residual data length conditionally [5.10,44/48] Input: appletouch - initialize work before device registration [5.10,45/48] Input: spaceball - fix parsing of movement data packets [5.10,46/48] net: fix use-after-free in tw_timer_handler [5.10,47/48] perf script: Fix CPU filtering of a scripts switch events [5.10,48/48] bpf: Add kconfig knob for disabling unpriv bpf by default

Commit Message

Greg KH Jan. 3, 2022, 2:24 p.m. UTC

From: Chunfeng Yun <chunfeng.yun@mediatek.com>

commit 8c313e3bfd9adae8d5c4ba1cc696dcbc86fbf9bf upstream.

This is caused by uninitialization of list_head.

BUG: KASAN: use-after-free in __list_del_entry_valid+0x34/0xe4

Call trace:
dump_backtrace+0x0/0x298
show_stack+0x24/0x34
dump_stack+0x130/0x1a8
print_address_description+0x88/0x56c
__kasan_report+0x1b8/0x2a0
kasan_report+0x14/0x20
__asan_load8+0x9c/0xa0
__list_del_entry_valid+0x34/0xe4
mtu3_req_complete+0x4c/0x300 [mtu3]
mtu3_gadget_stop+0x168/0x448 [mtu3]
usb_gadget_unregister_driver+0x204/0x3a0
unregister_gadget_item+0x44/0xa4

Fixes: 83374e035b62 ("usb: mtu3: add tracepoints to help debug")
Cc: stable@vger.kernel.org
Reported-by: Yuwen Ng <yuwen.ng@mediatek.com>
Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
Link: https://lore.kernel.org/r/20211218095749.6250-3-chunfeng.yun@mediatek.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
 drivers/usb/mtu3/mtu3_gadget.c |    1 +
 1 file changed, 1 insertion(+)

diff mbox series

Patch

--- a/drivers/usb/mtu3/mtu3_gadget.c
+++ b/drivers/usb/mtu3/mtu3_gadget.c
@@ -235,6 +235,7 @@  struct usb_request *mtu3_alloc_request(s
 	mreq->request.dma = DMA_ADDR_INVALID;
 	mreq->epnum = mep->epnum;
 	mreq->mep = mep;
+	INIT_LIST_HEAD(&mreq->list);
 	trace_mtu3_alloc_request(mreq);
 
 	return &mreq->request;