[media] dvb-usb-v2: remove dvb_usb_device NULL check

Reported by Dan Carpenter:

The patch d10d1b9ac97b: "[media] dvb_usb_v2: use dev_* logging
macros" from Jun 26, 2012, leads to the following Smatch complaint:

drivers/media/usb/dvb-usb-v2/dvb_usb_urb.c:31 dvb_usb_v2_generic_io()
	 error: we previously assumed 'd' could be null (see line 29)

...
Remove whole check as it must not happen in any case. Driver is
totally broken if it does not have valid pointer to device.

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Antti Palosaari <crope@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

diff --git a/drivers/media/usb/dvb-usb-v2/dvb_usb_urb.c b/drivers/media/usb/dvb-usb-v2/dvb_usb_urb.c
index 33ff97e708e3a8c2defa22491ab56cd1619fac8a..22bdce15ecf31fcd351309ca7fa5794cb228869d 100644 (file)
--- a/drivers/media/usb/dvb-usb-v2/dvb_usb_urb.c
+++ b/drivers/media/usb/dvb-usb-v2/dvb_usb_urb.c
@@ -26,7 +26,7 @@ static int dvb_usb_v2_generic_io(struct dvb_usb_device *d,
 {
        int ret, actual_length;
 
-       if (!d || !wbuf || !wlen || !d->props->generic_bulk_ctrl_endpoint ||
+       if (!wbuf || !wlen || !d->props->generic_bulk_ctrl_endpoint ||
                        !d->props->generic_bulk_ctrl_endpoint_response) {
                dev_dbg(&d->udev->dev, "%s: failed=%d\n", __func__, -EINVAL);
                return -EINVAL;