aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2015-02-03 12:36:35 -0500
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>2015-03-02 12:53:27 -0500
commit704b3f4c11e0b1265a41472b6b25dd9e3d205674 (patch)
tree559b9fd213f390e00f8dad5e47368a04d597369b
parentd585c1e14a84f08a4a714553d9fc459ee697ff95 (diff)
[media] stk-webcam: Delete an unnecessary check before the function call "vfree"
The vfree() function performs also input parameter validation. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
-rw-r--r--drivers/media/usb/stkwebcam/stk-webcam.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/media/usb/stkwebcam/stk-webcam.c b/drivers/media/usb/stkwebcam/stk-webcam.c
index e08fa587332f..c21c4c004f97 100644
--- a/drivers/media/usb/stkwebcam/stk-webcam.c
+++ b/drivers/media/usb/stkwebcam/stk-webcam.c
@@ -556,10 +556,8 @@ static int stk_free_sio_buffers(struct stk_camera *dev)
556 nbufs = dev->n_sbufs; 556 nbufs = dev->n_sbufs;
557 dev->n_sbufs = 0; 557 dev->n_sbufs = 0;
558 spin_unlock_irqrestore(&dev->spinlock, flags); 558 spin_unlock_irqrestore(&dev->spinlock, flags);
559 for (i = 0; i < nbufs; i++) { 559 for (i = 0; i < nbufs; i++)
560 if (dev->sio_bufs[i].buffer != NULL) 560 vfree(dev->sio_bufs[i].buffer);
561 vfree(dev->sio_bufs[i].buffer);
562 }
563 kfree(dev->sio_bufs); 561 kfree(dev->sio_bufs);
564 dev->sio_bufs = NULL; 562 dev->sio_bufs = NULL;
565 return 0; 563 return 0;