aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2014-11-23 06:30:33 -0500
committerTomi Valkeinen <tomi.valkeinen@ti.com>2014-12-04 09:16:01 -0500
commit2daff4d48d7e9acde58a68de3cd11f8215780499 (patch)
tree70c0994fe0b31e613d1f5414b035ef8c2b685f30
parent177c0386ea5beb5b8c73d7dbde41487f3f4eabf4 (diff)
video: smscufx: Deletion of unnecessary checks before the function call "vfree"
The vfree() function performs also input parameter validation. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
-rw-r--r--drivers/video/fbdev/smscufx.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/video/fbdev/smscufx.c b/drivers/video/fbdev/smscufx.c
index d513ed6a49f2..9279e5f6696e 100644
--- a/drivers/video/fbdev/smscufx.c
+++ b/drivers/video/fbdev/smscufx.c
@@ -1142,8 +1142,7 @@ static void ufx_free_framebuffer_work(struct work_struct *work)
1142 fb_dealloc_cmap(&info->cmap); 1142 fb_dealloc_cmap(&info->cmap);
1143 if (info->monspecs.modedb) 1143 if (info->monspecs.modedb)
1144 fb_destroy_modedb(info->monspecs.modedb); 1144 fb_destroy_modedb(info->monspecs.modedb);
1145 if (info->screen_base) 1145 vfree(info->screen_base);
1146 vfree(info->screen_base);
1147 1146
1148 fb_destroy_modelist(&info->modelist); 1147 fb_destroy_modelist(&info->modelist);
1149 1148
@@ -1743,8 +1742,7 @@ error:
1743 fb_dealloc_cmap(&info->cmap); 1742 fb_dealloc_cmap(&info->cmap);
1744 if (info->monspecs.modedb) 1743 if (info->monspecs.modedb)
1745 fb_destroy_modedb(info->monspecs.modedb); 1744 fb_destroy_modedb(info->monspecs.modedb);
1746 if (info->screen_base) 1745 vfree(info->screen_base);
1747 vfree(info->screen_base);
1748 1746
1749 fb_destroy_modelist(&info->modelist); 1747 fb_destroy_modelist(&info->modelist);
1750 1748