diff options
| author | Mauro Carvalho Chehab <mchehab@infradead.org> | 2007-08-01 10:13:36 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2007-08-01 10:13:36 -0300 |
| commit | f99add164ce3676cfb08aa46d689d7186317111c (patch) | |
| tree | 7e4db20c036dc5a7109616952e36f257d9de1f67 /linux/drivers | |
| parent | c117650e0ec2d29959fdbb91e043eb7bd28df429 (diff) | |
| download | mediapointer-dvb-s2-f99add164ce3676cfb08aa46d689d7186317111c.tar.gz mediapointer-dvb-s2-f99add164ce3676cfb08aa46d689d7186317111c.tar.bz2 | |
Frontend_ioctl(): fix check-after-use
From: Adrian Bunk <bunk@stusta.de>
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>
Diffstat (limited to 'linux/drivers')
| -rw-r--r-- | linux/drivers/media/dvb/dvb-core/dvb_frontend.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/linux/drivers/media/dvb/dvb-core/dvb_frontend.c b/linux/drivers/media/dvb/dvb-core/dvb_frontend.c index a05f4267a..e39820d23 100644 --- a/linux/drivers/media/dvb/dvb-core/dvb_frontend.c +++ b/linux/drivers/media/dvb/dvb-core/dvb_frontend.c @@ -753,7 +753,7 @@ static int dvb_frontend_ioctl(struct inode *inode, struct file *file, dprintk ("%s\n", __FUNCTION__); - if (!fe || fepriv->exit) + if (fepriv->exit) return -ENODEV; if ((file->f_flags & O_ACCMODE) == O_RDONLY && |