diff options
| author | Varun Reddy Yeturu <vyeturu@qca.qualcomm.com> | 2014-12-12 18:24:44 -0800 |
|---|---|---|
| committer | AnjaneeDevi Kapparapu <c_akappa@qti.qualcomm.com> | 2014-12-15 14:59:28 +0530 |
| commit | 6f45c5f59fedeecee66f72482ce3bcf66e796e9b (patch) | |
| tree | ae3007dd832fc629e8baad8113f1622e1df9a960 | |
| parent | f3a780b24faa8e34cfadf010d4e5d75d31abf88a (diff) | |
wlan: qcacld: Populate Value only if memory is allocated
Before copying the value of bkid count, it needs to be
checked if it is non-zero, since the memory to hold the
value is allocated only if it is non-zero.
Change-Id: I6b5dce50f2654abaa25c6345a8ff7b9d479ae5f2
CRs-Fixed: 770688
| -rw-r--r-- | CORE/SME/src/csr/csrApiRoam.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/CORE/SME/src/csr/csrApiRoam.c b/CORE/SME/src/csr/csrApiRoam.c index e24e7c811bac..4bfea18957e2 100644 --- a/CORE/SME/src/csr/csrApiRoam.c +++ b/CORE/SME/src/csr/csrApiRoam.c @@ -5333,12 +5333,10 @@ static eHalStatus csrRoamSaveSecurityRspIE(tpAniSirGlobal pMac, tANI_U32 session pIesLocal->WAPI.multicast_cipher_suite + 4, 2); pIeBuf += 2; - //bkid_count - vos_mem_copy(pIeBuf, &pIesLocal->WAPI.bkid_count, 2); - - pIeBuf += 2; - if( pIesLocal->WAPI.bkid_count ) - { + if (pIesLocal->WAPI.bkid_count) { + /* bkid_count */ + vos_mem_copy(pIeBuf, &pIesLocal->WAPI.bkid_count, 2); + pIeBuf += 2; //copy akm_suites vos_mem_copy(pIeBuf, pIesLocal->WAPI.bkid, pIesLocal->WAPI.bkid_count * 4); |