]> git.baikalelectronics.ru Git - kernel.git/commit
Bluetooth: use hdev->workqueue when queuing hdev->{cmd,ncmd}_timer works
authorTetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Fri, 2 Sep 2022 11:23:48 +0000 (20:23 +0900)
committerLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
Mon, 19 Sep 2022 17:24:24 +0000 (10:24 -0700)
commit4d0dc3362334df7604e7bca6742bef7597450d04
tree088ffbba0066f818a36861a74a91f3ef45989ba5
parent52decf9776f6366c481623b781011cc3cd1224df
Bluetooth: use hdev->workqueue when queuing hdev->{cmd,ncmd}_timer works

syzbot is reporting attempt to schedule hdev->cmd_work work from system_wq
WQ into hdev->workqueue WQ which is under draining operation [1], for
commit 857275110960ebb2 ("workqueue: allow chained queueing during
destruction") does not allow such operation.

The check introduced by commit b419cf7078d6cf30 ("Bluetooth: When HCI work
queue is drained, only queue chained work") was incomplete.

Use hdev->workqueue WQ when queuing hdev->{cmd,ncmd}_timer works because
hci_{cmd,ncmd}_timeout() calls queue_work(hdev->workqueue). Also, protect
the queuing operation with RCU read lock in order to avoid calling
queue_delayed_work() after cancel_delayed_work() completed.

Link: https://syzkaller.appspot.com/bug?extid=243b7d89777f90f7613b
Reported-by: syzbot <syzbot+243b7d89777f90f7613b@syzkaller.appspotmail.com>
Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Fixes: b419cf7078d6cf30 ("Bluetooth: When HCI work queue is drained, only queue chained work")
Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
net/bluetooth/hci_core.c
net/bluetooth/hci_event.c