Free resources before exit.

Signed-off-by: Alexander Beregalov <[email protected]>
---
 drivers/staging/brcm80211/brcmfmac/dhd_sdio.c |   12 ++++++++----
 1 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/brcm80211/brcmfmac/dhd_sdio.c 
b/drivers/staging/brcm80211/brcmfmac/dhd_sdio.c
index dd2e367..987d025 100644
--- a/drivers/staging/brcm80211/brcmfmac/dhd_sdio.c
+++ b/drivers/staging/brcm80211/brcmfmac/dhd_sdio.c
@@ -5647,6 +5647,10 @@ static int dhdsdio_download_code_array(struct dhd_bus 
*bus)
                unsigned char *ularray;
 
                ularray = kmalloc(bus->ramsize, GFP_ATOMIC);
+               if (!ularray) {
+                       bcmerror = BCME_NOMEM;
+                       goto err;
+               }
                /* Upload image to verify downloaded contents. */
                offset = 0;
                memset(ularray, 0xaa, bus->ramsize);
@@ -5658,7 +5662,7 @@ static int dhdsdio_download_code_array(struct dhd_bus 
*bus)
                                DHD_ERROR(("%s: error %d on reading %d membytes"
                                        " at 0x%08x\n",
                                        __func__, bcmerror, MEMBLOCK, offset));
-                               goto err;
+                               goto free;
                        }
 
                        offset += MEMBLOCK;
@@ -5672,7 +5676,7 @@ static int dhdsdio_download_code_array(struct dhd_bus 
*bus)
                                DHD_ERROR(("%s: error %d on reading %d membytes 
at 0x%08x\n",
                                __func__, bcmerror,
                                sizeof(dlarray) - offset, offset));
-                               goto err;
+                               goto free;
                        }
                }
 
@@ -5680,11 +5684,11 @@ static int dhdsdio_download_code_array(struct dhd_bus 
*bus)
                        DHD_ERROR(("%s: Downloaded image is corrupted.\n",
                                   __func__));
                        ASSERT(0);
-                       goto err;
+                       goto free;
                } else
                        DHD_ERROR(("%s: Download/Upload/Compare succeeded.\n",
                                __func__));
-
+free:
                kfree(ularray);
        }
 #endif                         /* DHD_DEBUG */
-- 
1.7.4.1

_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to