staging: gdm724x: cleanup alloc_tx_sdu_struct()

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>

diff --git a/drivers/staging/gdm724x/gdm_usb.c b/drivers/staging/gdm724x/gdm_usb.c
index 33458a583142db31db6c1b4c722849bb1e462372..03b43056fcf23ea415910d989c29f88b067c7348 100644 (file)
--- a/drivers/staging/gdm724x/gdm_usb.c
+++ b/drivers/staging/gdm724x/gdm_usb.c
@@ -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;
        }