commit 2d590283a771b3a9738ad6381ab525e2cf2c5025
author Abhinav Kumar <abhikuma@codeaurora.org> Thu Jun 28 11:45:15 2018 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Mon Jul 02 04:28:27 2018 -0700
tree 4afb7a4225e1ea04d58a94e2ab1178cdf476857a
parent 07c9558e59b3042930f19fba7451b2feba46a76d

wlan: Compare ssid IE length in limLookupNaddHashEntry

In commit I5c79bff3427a842036af788fea5003a96c7696a6,
ssid IE length address is compared in limLookupNaddHashEntry,
which results memory leak. Compare ssid IE length to fix
this issue.

Change-Id: I8fbf12b612297443319a9f5ff17140758200721b
CRs-Fixed: 2269285

CORE/MAC/src/pe/lim/limScanResultUtils.c

diff --git a/CORE/MAC/src/pe/lim/limScanResultUtils.c b/CORE/MAC/src/pe/lim/limScanResultUtils.c
index e81ed07..f166ec7 100644
--- a/CORE/MAC/src/pe/lim/limScanResultUtils.c
+++ b/CORE/MAC/src/pe/lim/limScanResultUtils.c
@@ -736,8 +736,8 @@
              // matching band to update new channel info
             (vos_chan_to_band(pBssDescr->bssDescription.channelId) ==
                       vos_chan_to_band(ptemp->bssDescription.channelId)) &&
-            (((tANI_U8 *) &pBssDescr->bssDescription.ieFields + 1) ==
-              ((tANI_U8 *) &ptemp->bssDescription.ieFields + 1)) &&
+            (*((tANI_U8 *) &pBssDescr->bssDescription.ieFields + 1) ==
+              *((tANI_U8 *) &ptemp->bssDescription.ieFields + 1)) &&
             vos_mem_compare( ((tANI_U8 *) &pBssDescr->bssDescription.ieFields + 1),
                            ((tANI_U8 *) &ptemp->bssDescription.ieFields + 1),
                            (tANI_U8) (ssidLen + 1)) &&
@@ -992,8 +992,8 @@
                       sizeof(tSirMacAddr))) &&   //matching BSSID
             (pBssDescr->bssDescription.channelId ==
                                       ptemp->bssDescription.channelId) &&
-            (((tANI_U8 *) &pBssDescr->bssDescription.ieFields + 1) ==
-              ((tANI_U8 *) &ptemp->bssDescription.ieFields + 1)) &&
+            (*((tANI_U8 *) &pBssDescr->bssDescription.ieFields + 1) ==
+              *((tANI_U8 *) &ptemp->bssDescription.ieFields + 1)) &&
             vos_mem_compare( ((tANI_U8 *) &pBssDescr->bssDescription.ieFields + 1),
                            ((tANI_U8 *) &ptemp->bssDescription.ieFields + 1),
                            (tANI_U8) (ssidLen + 1)) &&