diff options
| author | Adrian Bunk <bunk@stusta.de> | 2006-12-20 09:34:32 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2006-12-27 14:19:56 -0200 |
| commit | d4f60baf7827c6a46b7ee2ed0a2a32a373febfcb (patch) | |
| tree | 3b4e0b7677cc022fc776f9016e323db42555b99d /drivers/media | |
| parent | d82d418a3af4d7d07151f9d45ca20f2ce61289a0 (diff) | |
| download | op-kernel-dev-d4f60baf7827c6a46b7ee2ed0a2a32a373febfcb.zip op-kernel-dev-d4f60baf7827c6a46b7ee2ed0a2a32a373febfcb.tar.gz | |
V4L/DVB (4991): Cafe_ccic.c: fix NULL dereference
We shouldn't dereference "cam" when we already know it's NULL.
Spotted by the Coverity checker.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Diffstat (limited to 'drivers/media')
| -rw-r--r-- | drivers/media/video/cafe_ccic.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/video/cafe_ccic.c b/drivers/media/video/cafe_ccic.c index e347c7e..3083c80 100644 --- a/drivers/media/video/cafe_ccic.c +++ b/drivers/media/video/cafe_ccic.c @@ -2166,7 +2166,7 @@ static void cafe_pci_remove(struct pci_dev *pdev) struct cafe_camera *cam = cafe_find_by_pdev(pdev); if (cam == NULL) { - cam_warn(cam, "pci_remove on unknown pdev %p\n", pdev); + printk(KERN_WARNING "pci_remove on unknown pdev %p\n", pdev); return; } mutex_lock(&cam->s_mutex); |
