commit 1e2bfa32d015cf7ee67c944ccbccc7c8e30c71ce
author Abhishek Singh <absingh@qti.qualcomm.com> Thu Jan 02 15:44:15 2014 +0530
committer Kiet Lam <c_kietl@qca.qualcomm.com> Fri Jan 03 11:22:59 2014 -0800
tree cc26d2e00852e5c0e1dcb21dab184afa5bc175b0
parent b20fd748ca0f39783dffcca484afcae9ec22a2bd

wlan: Fix null pointer references found in static analysis.

Fixed potential null pointer references and
memory leaks find by static code analysis.

Change-Id: Ib13533257a23fb0801a287a256a9faa5c7ffa4f4
CRs-Fixed: 594219

CORE/WDA/src/wlan_qct_wda.c

diff --git a/CORE/WDA/src/wlan_qct_wda.c b/CORE/WDA/src/wlan_qct_wda.c
index 30420e8..5c63135 100644
--- a/CORE/WDA/src/wlan_qct_wda.c
+++ b/CORE/WDA/src/wlan_qct_wda.c
@@ -5331,10 +5331,10 @@
           VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
                            "%s: VOS MEM Alloc Failure", __func__);
           VOS_ASSERT(0);
+          vos_mem_free(pGetRoamRssiParams);
+          vos_mem_free(pWdaParams);
           return VOS_STATUS_E_NOMEM;
       }
-      vos_mem_free(pGetRoamRssiParams);
-      vos_mem_free(pWdaParams);
       pGetRoamRssiRspParams->staId = pGetRoamRssiParams->staId;
       pGetRoamRssiRspParams->rc    = eSIR_FAILURE;
       pGetRoamRssiRspParams->rssi    = 0;