commit d1e821dfaf1128b5e0a84c1093dd2f0b5f43726e
author Mahesh A Saptasagar <c_msapta@qti.qualcomm.com> Fri Apr 04 11:42:47 2014 +0530
committer AnjaneeDevi Kapparapu <akappa@codeaurora.org> Mon Apr 07 14:13:32 2014 +0530
tree 3a1723771b3bdc5966a53709f49fae31ee85962c
parent a0498ef814bd4710021d8f4827356852b71e29e4

wlan: Fix for FTM mode host driver memory leak

The Memory allocated is not being freed in case of failure.
Free the memory allocated when return due to failure.

Change-Id: I4189820864672d53aa32c73dbf5ddfe6d93a4b35
CRs-Fixed: 642837

CORE/HDD/src/wlan_hdd_ftm.c

diff --git a/CORE/HDD/src/wlan_hdd_ftm.c b/CORE/HDD/src/wlan_hdd_ftm.c
index 85e3cfa..ad2acc6 100644
--- a/CORE/HDD/src/wlan_hdd_ftm.c
+++ b/CORE/HDD/src/wlan_hdd_ftm.c
@@ -1321,7 +1321,7 @@
     if ( NULL == pHddCtx->hHal )
     {
        hddLog(VOS_TRACE_LEVEL_ERROR,"%s: HAL context is null", __func__);
-       goto err_sal_close;
+       goto err_ftm_vos_close;
     }
 
     pAdapter = hdd_open_adapter( pHddCtx, WLAN_HDD_FTM, "wlan%d",
@@ -1329,13 +1329,13 @@
     if( NULL == pAdapter )
     {
        hddLog(VOS_TRACE_LEVEL_ERROR,"%s: hdd_open_adapter failed", __func__);
-               goto err_adapter_open_failure;
+       goto err_adapter_open_failure;
     }
 
     if( wlan_ftm_register_wext(pAdapter)!= 0 )
     {
        hddLog(VOS_TRACE_LEVEL_ERROR,"%s: hdd_register_wext failed", __func__);
-       goto err_sal_close;
+       goto err_adapter_close;
     }
 
        //Initialize the nlink service
@@ -1408,11 +1408,12 @@
 err_ftm_register_wext_close:
 hdd_UnregisterWext(pAdapter->dev);
 
+err_adapter_close:
 err_adapter_open_failure:
 hdd_close_all_adapters( pHddCtx );
 
-err_sal_close:
-
+err_ftm_vos_close:
+    wlan_ftm_vos_close(pVosContext);
 err_vos_status_failure:
 
     return VOS_STATUS_E_FAILURE;