Commit 8f96c956 authored by KAMBAROV, ZAUR's avatar KAMBAROV, ZAUR Committed by Linus Torvalds

[PATCH] coverity: fix fbsysfs null pointer check

Correctly test for a null pointer before going and dereferencing it.

This defect was found automatically by Coverity Prevent, a static analysis
tool.
Signed-off-by: default avatarZaur Kambarov <zkambarov@coverity.com>
Cc: <linux-fbdev-devel@lists.sourceforge.net>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 5bbcfd90
...@@ -256,7 +256,7 @@ static ssize_t show_cmap(struct class_device *class_device, char *buf) ...@@ -256,7 +256,7 @@ static ssize_t show_cmap(struct class_device *class_device, char *buf)
unsigned int offset = 0, i; unsigned int offset = 0, i;
if (!fb_info->cmap.red || !fb_info->cmap.blue || if (!fb_info->cmap.red || !fb_info->cmap.blue ||
fb_info->cmap.green || fb_info->cmap.transp) !fb_info->cmap.green || !fb_info->cmap.transp)
return -EINVAL; return -EINVAL;
for (i = 0; i < fb_info->cmap.len; i++) { for (i = 0; i < fb_info->cmap.len; i++) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment