summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2012-05-14 23:58:37 +0300
committerFlorian Tobias Schandinat <FlorianSchandinat@gmx.de>2012-05-29 13:00:03 +0000
commitcc4401142c1cbc63b01d6024cbc7a9f804cb3143 (patch)
treefd28d49904845d402661bffa0c618e4448529d83
parentd85d135d8babbc917b370f36cbc02b7b4a2f2d99 (diff)
downloadop-kernel-dev-cc4401142c1cbc63b01d6024cbc7a9f804cb3143.zip
op-kernel-dev-cc4401142c1cbc63b01d6024cbc7a9f804cb3143.tar.gz
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>
-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 67afa9c..a55e366 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);
}
OpenPOWER on IntegriCloud