diff options
author | Adrian Bunk <bunk@kernel.org> | 2007-10-14 13:51:37 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2007-10-22 10:01:40 -0400 |
commit | 7e7f05ca156d34b80e53105e4ef9bc1497a68439 (patch) | |
tree | 12ea6c65c60c9fcb29724b4f8f9906a5783f0500 /drivers/media/video/ivtv/ivtv-fileops.c | |
parent | 1419683d85367df90079fb5535865ac0841ebbea (diff) |
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/video/ivtv/ivtv-fileops.c')
-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 da50fa4a72a5..0e0605cc0bd3 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) | |||
947 | if (itv == NULL) { | 947 | if (itv == NULL) { |
948 | /* Couldn't find a device registered | 948 | /* Couldn't find a device registered |
949 | on that minor, shouldn't happen! */ | 949 | on that minor, shouldn't happen! */ |
950 | IVTV_WARN("No ivtv device found on minor %d\n", minor); | 950 | printk(KERN_WARNING "No ivtv device found on minor %d\n", minor); |
951 | return -ENXIO; | 951 | return -ENXIO; |
952 | } | 952 | } |
953 | 953 | ||