]> git.itanic.dy.fi Git - linux-stable/commit
power: supply: da9150: Fix use after free bug in da9150_charger_remove due to race...
authorZheng Wang <zyytlz.wz@163.com>
Sat, 11 Mar 2023 17:46:50 +0000 (01:46 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 5 Apr 2023 09:14:14 +0000 (11:14 +0200)
commitbbf45f079f41efcf1e51bb65a0a45d2b31061bd5
treeeae944f75d3fa5766f3cc7499a05208c258b9bf5
parentef14eeed25df11f2b3a5fd5f90eb3ad1a41d97ef
power: supply: da9150: Fix use after free bug in da9150_charger_remove due to race condition

[ Upstream commit 06615d11cc78162dfd5116efb71f29eb29502d37 ]

In da9150_charger_probe, &charger->otg_work is bound with
da9150_charger_otg_work. da9150_charger_otg_ncb may be
called to start the work.

If we remove the module which will call da9150_charger_remove
to make cleanup, there may be a unfinished work. The possible
sequence is as follows:

Fix it by canceling the work before cleanup in the da9150_charger_remove

CPU0                  CPUc1

                    |da9150_charger_otg_work
da9150_charger_remove      |
power_supply_unregister  |
device_unregister   |
power_supply_dev_release|
kfree(psy)          |
                    |
                    |  power_supply_changed(charger->usb);
                    |   //use

Fixes: c1a281e34dae ("power: Add support for DA9150 Charger")
Signed-off-by: Zheng Wang <zyytlz.wz@163.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/power/supply/da9150-charger.c