staging: r8188eu: Remove a test from usbctrl_vendorreq()
authorFabio M. De Francesco <fmdefrancesco@gmail.com>
Fri, 24 Sep 2021 12:27:01 +0000 (14:27 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 27 Sep 2021 15:37:12 +0000 (17:37 +0200)
Remove an unnecessary 'if' test from usbctrl_vendorreq() because
"length" is never greater than MAX_VENDOR_REQ_CMD_SIZE.

Co-developed-by: Pavel Skripkin <paskripkin@gmail.com>
Signed-off-by: Pavel Skripkin <paskripkin@gmail.com>
Signed-off-by: Fabio M. De Francesco <fmdefrancesco@gmail.com>
Link: https://lore.kernel.org/r/20210924122705.3781-13-fmdefrancesco@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/r8188eu/hal/usb_ops_linux.c

index 04a878c..b3f8a76 100644 (file)
@@ -23,12 +23,6 @@ static int usbctrl_vendorreq(struct intf_hdl *intf, u16 value, void *data, u16 l
                goto exit;
        }
 
-       if (len > MAX_VENDOR_REQ_CMD_SIZE) {
-               DBG_88E("[%s] Buffer len error ,vendor request failed\n", __func__);
-               status = -EINVAL;
-               goto exit;
-       }
-
        mutex_lock(&dvobjpriv->usb_vendor_req_mutex);
 
        /*  Acquire IO memory for vendorreq */