staging: gdm724x: cleanup alloc_tx_sdu_struct()
authorDan Carpenter <dan.carpenter@oracle.com>
Tue, 18 Feb 2014 12:18:10 +0000 (15:18 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 18 Feb 2014 19:16:56 +0000 (11:16 -0800)
The kfree(t_sdu->buf) sets off a private static checker warning because
"t_sdu->buf" is always NULL.

This function just allocates two pointers so we can re-write it to be
simpler.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/gdm724x/gdm_usb.c

index 33458a583142db31db6c1b4c722849bb1e462372..03b43056fcf23ea415910d989c29f88b067c7348 100644 (file)
@@ -119,28 +119,15 @@ out:
 
 static struct usb_tx_sdu *alloc_tx_sdu_struct(void)
 {
-       struct usb_tx_sdu *t_sdu = NULL;
-       int ret = 0;
-
+       struct usb_tx_sdu *t_sdu;
 
        t_sdu = kzalloc(sizeof(struct usb_tx_sdu), GFP_ATOMIC);
-       if (!t_sdu) {
-               ret = -ENOMEM;
-               goto out;
-       }
+       if (!t_sdu)
+               return NULL;
 
        t_sdu->buf = kmalloc(SDU_BUF_SIZE, GFP_ATOMIC);
        if (!t_sdu->buf) {
-               ret = -ENOMEM;
-               goto out;
-       }
-out:
-
-       if (ret < 0) {
-               if (t_sdu) {
-                       kfree(t_sdu->buf);
-                       kfree(t_sdu);
-               }
+               kfree(t_sdu);
                return NULL;
        }