mirrors/linux
1
0
Fork 1
mirror of https://github.com/torvalds/linux.git synced 2025-11-04 10:40:15 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

rtc: mxc_v2: fix possible race condition

Browse source 
The IRQ is requested before the struct rtc is allocated and registered, but
this struct is used in the IRQ handler. This may lead to a NULL pointer
dereference.

Switch to devm_rtc_allocate_device/rtc_register_device to allocate the rtc
before requesting the IRQ.

Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
This commit is contained in:
Alexandre Belloni 2018-05-19 10:01:42 +02:00
parent d759924500
commit 5490a1e018
1 changed files with 9 additions and 7 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

16
drivers/rtc/rtc-mxc_v2.c
View file

@ -343,6 +343,12 @@ static int mxc_rtc_probe(struct platform_device *pdev)
return ret; return ret;
} }
pdata->rtc = devm_rtc_allocate_device(&pdev->dev);
if (IS_ERR(pdata->rtc))
return PTR_ERR(pdata->rtc);
pdata->rtc->ops = &mxc_rtc_ops;
clk_disable(pdata->clk); clk_disable(pdata->clk);
platform_set_drvdata(pdev, pdata); platform_set_drvdata(pdev, pdata);
ret = ret =
@ -354,15 +360,11 @@ static int mxc_rtc_probe(struct platform_device *pdev)
return ret; return ret;
} }
pdata->rtc = ret = rtc_register_device(pdata->rtc);
devm_rtc_device_register(&pdev->dev, pdev->name, &mxc_rtc_ops, if (ret < 0)
THIS_MODULE);
if (IS_ERR(pdata->rtc)) {
clk_unprepare(pdata->clk); clk_unprepare(pdata->clk);
return PTR_ERR(pdata->rtc);
}
return 0; return ret;
} }
static int mxc_rtc_remove(struct platform_device *pdev) static int mxc_rtc_remove(struct platform_device *pdev)