diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-12-16 10:49:23 +0100 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2014-01-17 09:57:48 +0100 |
commit | 0ede5804ca3b42c831f7440ec29bc7187e5376bb (patch) | |
tree | 6760c9fd0cda38527f4d237d2e5125befff5e25a /drivers/video | |
parent | video: mmp: Using plain integer as NULL pointer (diff) | |
download | linux-0ede5804ca3b42c831f7440ec29bc7187e5376bb.tar.xz linux-0ede5804ca3b42c831f7440ec29bc7187e5376bb.zip |
tgafb: potential NULL dereference in init
Static checkers complain that there are paths where "tga_type_name" can
be NULL. I've re-arranged the code slightly so that's impossible.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Diffstat (limited to 'drivers/video')
-rw-r--r-- | drivers/video/tgafb.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/video/tgafb.c b/drivers/video/tgafb.c index 5e94c6e4f682..07c7df9ee77b 100644 --- a/drivers/video/tgafb.c +++ b/drivers/video/tgafb.c @@ -1490,10 +1490,9 @@ tgafb_init_fix(struct fb_info *info) if (tga_bus_tc) tga_type_name = "Digital ZLX-E3"; break; - default: - tga_type_name = "Unknown"; - break; } + if (!tga_type_name) + tga_type_name = "Unknown"; strlcpy(info->fix.id, tga_type_name, sizeof(info->fix.id)); |