]> git.baikalelectronics.ru Git - kernel.git/commit
Bluetooth: L2CAP: Fix user-after-free
authorLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
Thu, 29 Sep 2022 20:27:13 +0000 (13:27 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 26 Oct 2022 11:22:51 +0000 (13:22 +0200)
commit174e3bc97dfde32b7ddc1741f6da8654ae6beb4f
tree416d7b4bf4538d4fbaf3e407a668ecc4b313b28d
parent72135e38cf03639ba4a9066159f91a950f0a93af
Bluetooth: L2CAP: Fix user-after-free

[ Upstream commit 68a987f1eaeef9df021264dce3386434840160b4 ]

This uses l2cap_chan_hold_unless_zero() after calling
__l2cap_get_chan_blah() to prevent the following trace:

Bluetooth: l2cap_core.c:static void l2cap_chan_destroy(struct kref
*kref)
Bluetooth: chan 0000000023c4974d
Bluetooth: parent 00000000ae861c08
==================================================================
BUG: KASAN: use-after-free in __mutex_waiter_is_first
kernel/locking/mutex.c:191 [inline]
BUG: KASAN: use-after-free in __mutex_lock_common
kernel/locking/mutex.c:671 [inline]
BUG: KASAN: use-after-free in __mutex_lock+0x278/0x400
kernel/locking/mutex.c:729
Read of size 8 at addr ffff888006a49b08 by task kworker/u3:2/389

Link: https://lore.kernel.org/lkml/20220622082716.478486-1-lee.jones@linaro.org
Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
Signed-off-by: Sungwoo Kim <iam@sung-woo.kim>
Signed-off-by: Sasha Levin <sashal@kernel.org>
net/bluetooth/l2cap_core.c