Commit b6cfa007 authored by Yang Yingliang's avatar Yang Yingliang Committed by Sebastian Reichel

power: supply: generic-adc-battery: fix possible use-after-free in gab_remove()

This driver's remove path calls cancel_delayed_work(). However, that
function does not wait until the work function finishes. This means
that the callback function may still be running after the driver's
remove function has finished, which would result in a use-after-free.

Fix by calling cancel_delayed_work_sync(), which ensures that
the work is properly cancelled, no longer running, and unable
to re-schedule itself.
Reported-by: default avatarHulk Robot <hulkci@huawei.com>
Signed-off-by: default avatarYang Yingliang <yangyingliang@huawei.com>
Signed-off-by: default avatarSebastian Reichel <sebastian.reichel@collabora.com>
parent e61ffb34
...@@ -373,7 +373,7 @@ static int gab_remove(struct platform_device *pdev) ...@@ -373,7 +373,7 @@ static int gab_remove(struct platform_device *pdev)
} }
kfree(adc_bat->psy_desc.properties); kfree(adc_bat->psy_desc.properties);
cancel_delayed_work(&adc_bat->bat_work); cancel_delayed_work_sync(&adc_bat->bat_work);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment