summaryrefslogtreecommitdiffstats
path: root/drivers/video
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2012-05-14 22:58:37 +0200
committerFlorian Tobias Schandinat <FlorianSchandinat@gmx.de>2012-05-29 15:00:03 +0200
commitcc4401142c1cbc63b01d6024cbc7a9f804cb3143 (patch)
treefd28d49904845d402661bffa0c618e4448529d83 /drivers/video
parentMerge tag 'omapdss-for-3.5' of git://github.com/tomba/linux into fbdev-next (diff)
downloadlinux-cc4401142c1cbc63b01d6024cbc7a9f804cb3143.tar.xz
linux-cc4401142c1cbc63b01d6024cbc7a9f804cb3143.zip
fb: handle NULL pointers in framebuffer release
This function is called with a potential NULL pointer in picolcd_init_framebuffer() and it causes a static checker warning. This used to handle NULL pointers when the picolcd code was written, but a couple months later we added the "info->apertures" dereference. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Reviewed-by: Marcin Slusarz <marcin.slusarz@gmail.com> Signed-off-by: Florian Tobias Schandinat <FlorianSchandinat@gmx.de>
Diffstat (limited to 'drivers/video')
-rw-r--r--drivers/video/fbsysfs.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/video/fbsysfs.c b/drivers/video/fbsysfs.c
index 67afa9c2289d..a55e3669d135 100644
--- a/drivers/video/fbsysfs.c
+++ b/drivers/video/fbsysfs.c
@@ -80,6 +80,8 @@ EXPORT_SYMBOL(framebuffer_alloc);
*/
void framebuffer_release(struct fb_info *info)
{
+ if (!info)
+ return;
kfree(info->apertures);
kfree(info);
}