diff options
| author | Adrian Bunk <bunk@stusta.de> | 2007-08-01 09:13:36 -0400 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2007-10-09 21:04:22 -0400 |
| commit | 813ce47cee33964ff710d2c91063548773cb4cd5 (patch) | |
| tree | 2f1378a8304b9f399c317309604b9f09c79c694f | |
| parent | 89f4267d6fa96cb3db053d5183558c94ad5f46e5 (diff) | |
V4L/DVB (5965): Frontend_ioctl(): fix check-after-use
The Coverity checker spotted that we have already oops'ed if "fe" was NULL.
Since "fe" being NULL seems impossible at this point this patch removes
the NULL check.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Acked-by: Manu Abraham <manu@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
| -rw-r--r-- | drivers/media/dvb/dvb-core/dvb_frontend.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/dvb/dvb-core/dvb_frontend.c b/drivers/media/dvb/dvb-core/dvb_frontend.c index afe797b75de4..fd9bac5eb25e 100644 --- a/drivers/media/dvb/dvb-core/dvb_frontend.c +++ b/drivers/media/dvb/dvb-core/dvb_frontend.c | |||
| @@ -748,7 +748,7 @@ static int dvb_frontend_ioctl(struct inode *inode, struct file *file, | |||
| 748 | 748 | ||
| 749 | dprintk ("%s\n", __FUNCTION__); | 749 | dprintk ("%s\n", __FUNCTION__); |
| 750 | 750 | ||
| 751 | if (!fe || fepriv->exit) | 751 | if (fepriv->exit) |
| 752 | return -ENODEV; | 752 | return -ENODEV; |
| 753 | 753 | ||
| 754 | if ((file->f_flags & O_ACCMODE) == O_RDONLY && | 754 | if ((file->f_flags & O_ACCMODE) == O_RDONLY && |