diff options
| author | Mauro Carvalho Chehab <mchehab@infradead.org> | 2006-12-20 10:34:32 -0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2006-12-20 10:34:32 -0200 |
| commit | 607d813a90cf1bb17bfbf70a23cd8c4b3f4055fd (patch) | |
| tree | 269b2af29c47f2ab4650db138088bdcb963f3a0c | |
| parent | 529d4a90d815016b14241e3473fe222df8739d32 (diff) | |
| download | mediapointer-dvb-s2-607d813a90cf1bb17bfbf70a23cd8c4b3f4055fd.tar.gz mediapointer-dvb-s2-607d813a90cf1bb17bfbf70a23cd8c4b3f4055fd.tar.bz2 | |
Cafe_ccic.c: fix NULL dereference
From: Adrian Bunk <bunk@stusta.de>
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>
| -rw-r--r-- | linux/drivers/media/video/cafe_ccic.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/linux/drivers/media/video/cafe_ccic.c b/linux/drivers/media/video/cafe_ccic.c index 46010ca07..68f2326dc 100644 --- a/linux/drivers/media/video/cafe_ccic.c +++ b/linux/drivers/media/video/cafe_ccic.c @@ -2173,7 +2173,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); |