aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/media
diff options
context:
space:
mode:
authorDan Carpenter <error27@gmail.com>2010-05-05 01:59:48 -0400
committerMauro Carvalho Chehab <mchehab@redhat.com>2010-05-19 11:59:13 -0400
commitb9245d80c375b93db79475ca231276e1fa5a7649 (patch)
treee0647a8c551ca982be1df56d07ca1b3d689d158b /drivers/media
parent16ee9bb100340f9f775ab334acb645f54642f312 (diff)
V4L/DVB: ov511: cleanup: remove unneeded null check
We dereference "ov" unconditionally throughout the function so there is no way it can be NULL here. This code has been around for ages so if it were possible for "ov" to be NULL someone would have complained. Signed-off-by: Dan Carpenter <error27@gmail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media')
-rw-r--r--drivers/media/video/ov511.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/video/ov511.c b/drivers/media/video/ov511.c
index 6085d5582555..a10912097b7a 100644
--- a/drivers/media/video/ov511.c
+++ b/drivers/media/video/ov511.c
@@ -5940,7 +5940,7 @@ ov51x_disconnect(struct usb_interface *intf)
5940 ov->dev = NULL; 5940 ov->dev = NULL;
5941 5941
5942 /* Free the memory */ 5942 /* Free the memory */
5943 if (ov && !ov->user) { 5943 if (!ov->user) {
5944 mutex_lock(&ov->cbuf_lock); 5944 mutex_lock(&ov->cbuf_lock);
5945 kfree(ov->cbuf); 5945 kfree(ov->cbuf);
5946 ov->cbuf = NULL; 5946 ov->cbuf = NULL;