Check for kzalloc failure and directly return from the error patch thus
simplifying the success path.

Suggested-by: Dan Carpenter <dan.carpen...@oracle.com>
Signed-off-by: Sudip Mukherjee <su...@vectorindia.org>
---
 drivers/staging/rtl8188eu/os_dep/usb_intf.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/rtl8188eu/os_dep/usb_intf.c 
b/drivers/staging/rtl8188eu/os_dep/usb_intf.c
index 09a918e..1c9558f 100644
--- a/drivers/staging/rtl8188eu/os_dep/usb_intf.c
+++ b/drivers/staging/rtl8188eu/os_dep/usb_intf.c
@@ -114,13 +114,12 @@ static struct dvobj_priv *usb_dvobj_init(struct 
usb_interface *usb_intf)
        mutex_init(&pdvobjpriv->usb_vendor_req_mutex);
        pdvobjpriv->usb_vendor_req_buf = kzalloc(MAX_USB_IO_CTL_SIZE, 
GFP_KERNEL);
 
-       if (pdvobjpriv->usb_vendor_req_buf) {
-               usb_get_dev(pusbd);
-       } else {
+       if (!pdvobjpriv->usb_vendor_req_buf) {
                usb_set_intfdata(usb_intf, NULL);
                kfree(pdvobjpriv);
-               pdvobjpriv = NULL;
+               return NULL;
        }
+       usb_get_dev(pusbd);
 
        return pdvobjpriv;
 }
-- 
1.8.1.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to