usb_free_coherent() is safe with NULL addr and this check is
not required.

Signed-off-by: Qinglang Miao <[email protected]>
---
 drivers/media/usb/b2c2/flexcop-usb.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/media/usb/b2c2/flexcop-usb.c 
b/drivers/media/usb/b2c2/flexcop-usb.c
index 198ddfb8d..12c0b40be 100644
--- a/drivers/media/usb/b2c2/flexcop-usb.c
+++ b/drivers/media/usb/b2c2/flexcop-usb.c
@@ -419,10 +419,9 @@ static void flexcop_usb_transfer_exit(struct flexcop_usb 
*fc_usb)
                        usb_free_urb(fc_usb->iso_urb[i]);
                }
 
-       if (fc_usb->iso_buffer != NULL)
-               usb_free_coherent(fc_usb->udev,
-                       fc_usb->buffer_size, fc_usb->iso_buffer,
-                       fc_usb->dma_addr);
+       usb_free_coherent(fc_usb->udev, fc_usb->buffer_size,
+                         fc_usb->iso_buffer, fc_usb->dma_addr);
+
 }
 
 static int flexcop_usb_transfer_init(struct flexcop_usb *fc_usb)
-- 
2.25.1

Reply via email to