diff options
author | Christophe JAILLET <christophe.jaillet@wanadoo.fr> | 2017-11-09 18:09:28 +0100 |
---|---|---|
committer | Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> | 2017-11-09 18:09:28 +0100 |
commit | 8cae353e6b01ac3f18097f631cdbceb5ff28c7f3 (patch) | |
tree | 6e33ed0ebd4d330d55347d117207b69d693ac587 | |
parent | video: fbdev: au1200fb: Fix a potential double free (diff) | |
download | linux-8cae353e6b01ac3f18097f631cdbceb5ff28c7f3.tar.xz linux-8cae353e6b01ac3f18097f631cdbceb5ff28c7f3.zip |
video: fbdev: au1200fb: Return an error code if a memory allocation fails
'ret' is known to be 0 at this point.
In case of memory allocation error in 'framebuffer_alloc()', return
-ENOMEM instead.
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Cc: Tejun Heo <tj@kernel.org>
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
-rw-r--r-- | drivers/video/fbdev/au1200fb.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/video/fbdev/au1200fb.c b/drivers/video/fbdev/au1200fb.c index a5facc2ad90b..7fa41026984d 100644 --- a/drivers/video/fbdev/au1200fb.c +++ b/drivers/video/fbdev/au1200fb.c @@ -1680,8 +1680,10 @@ static int au1200fb_drv_probe(struct platform_device *dev) fbi = framebuffer_alloc(sizeof(struct au1200fb_device), &dev->dev); - if (!fbi) + if (!fbi) { + ret = -ENOMEM; goto failed; + } _au1200fb_infos[plane] = fbi; fbdev = fbi->par; |