diff options
author | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2012-12-13 05:08:21 -0500 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2012-12-13 05:08:21 -0500 |
commit | 057eeaee6803dafbbf2f79345643530e731ba7cb (patch) | |
tree | f9a2452a84d21d182a32fd2c92e5929a8e6f34b4 /drivers/video | |
parent | 8105c94bc4f94478c67afb7593724e4d684dd9c0 (diff) |
OMAPFB: use devm_kzalloc to allocate omapfb2_device
Use devm_kzalloc to allocate omapfb2_device. This fixes possible memory
leak:
drivers/video/omap2/omapfb/omapfb-main.c:2553 omapfb_probe() warn:
possible memory leak of 'fbdev'
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Diffstat (limited to 'drivers/video')
-rw-r--r-- | drivers/video/omap2/omapfb/omapfb-main.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/video/omap2/omapfb/omapfb-main.c b/drivers/video/omap2/omapfb/omapfb-main.c index 4fa9487a8720..477a20817d08 100644 --- a/drivers/video/omap2/omapfb/omapfb-main.c +++ b/drivers/video/omap2/omapfb/omapfb-main.c | |||
@@ -1868,7 +1868,6 @@ static void omapfb_free_resources(struct omapfb2_device *fbdev) | |||
1868 | } | 1868 | } |
1869 | 1869 | ||
1870 | dev_set_drvdata(fbdev->dev, NULL); | 1870 | dev_set_drvdata(fbdev->dev, NULL); |
1871 | kfree(fbdev); | ||
1872 | } | 1871 | } |
1873 | 1872 | ||
1874 | static int omapfb_create_framebuffers(struct omapfb2_device *fbdev) | 1873 | static int omapfb_create_framebuffers(struct omapfb2_device *fbdev) |
@@ -2413,7 +2412,8 @@ static int __init omapfb_probe(struct platform_device *pdev) | |||
2413 | goto err0; | 2412 | goto err0; |
2414 | } | 2413 | } |
2415 | 2414 | ||
2416 | fbdev = kzalloc(sizeof(struct omapfb2_device), GFP_KERNEL); | 2415 | fbdev = devm_kzalloc(&pdev->dev, sizeof(struct omapfb2_device), |
2416 | GFP_KERNEL); | ||
2417 | if (fbdev == NULL) { | 2417 | if (fbdev == NULL) { |
2418 | r = -ENOMEM; | 2418 | r = -ENOMEM; |
2419 | goto err0; | 2419 | goto err0; |