diff options
author | Adrian Bunk <bunk@kernel.org> | 2007-10-14 14:51:37 -0300 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2007-10-22 12:01:40 -0200 |
commit | 7e7f05ca156d34b80e53105e4ef9bc1497a68439 (patch) | |
tree | 12ea6c65c60c9fcb29724b4f8f9906a5783f0500 /drivers/media | |
parent | 1419683d85367df90079fb5535865ac0841ebbea (diff) | |
download | op-kernel-dev-7e7f05ca156d34b80e53105e4ef9bc1497a68439.zip op-kernel-dev-7e7f05ca156d34b80e53105e4ef9bc1497a68439.tar.gz |
V4L/DVB (6328): ivtv: fix NULL dereference
We shouldn't dereference "itv" when we know it's NULL...
Spotted by the Coverity checker.
Signed-off-by: Adrian Bunk <bunk@kernel.org>
Reviewed-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Diffstat (limited to 'drivers/media')
-rw-r--r-- | drivers/media/video/ivtv/ivtv-fileops.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/video/ivtv/ivtv-fileops.c b/drivers/media/video/ivtv/ivtv-fileops.c index da50fa4..0e0605c 100644 --- a/drivers/media/video/ivtv/ivtv-fileops.c +++ b/drivers/media/video/ivtv/ivtv-fileops.c @@ -947,7 +947,7 @@ int ivtv_v4l2_open(struct inode *inode, struct file *filp) if (itv == NULL) { /* Couldn't find a device registered on that minor, shouldn't happen! */ - IVTV_WARN("No ivtv device found on minor %d\n", minor); + printk(KERN_WARNING "No ivtv device found on minor %d\n", minor); return -ENXIO; } |