commit 8bdfac4c43643f7ef2fd6a2a70632b0cc388dcc3
author Praveen Kumar Sirisilla <c_pksiri@qca.qualcomm.com> Thu Oct 10 17:20:48 2013 -0700
committer Madan Mohan Koyyalamudi <mkoyyala@qca.qualcomm.com> Mon Oct 21 23:41:15 2013 -0700
tree b6c8722a7e708b67002e02d56317f3c38f01358b
parent 4da2c3a5d087730804284ff59562f9666be70b52

SS Code changes to pass the code review inspection [UMAC} changes

The patch modifies UMAC code to pass the SS code review inspection

Change-Id: Id527265f0e3ed33a03ab242a3e0afec069487269
CRs-Fixed: 555476

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

diff --git a/CORE/MAC/src/pe/lim/limAssocUtils.c b/CORE/MAC/src/pe/lim/limAssocUtils.c
index 7af69ef..5283e57 100644
--- a/CORE/MAC/src/pe/lim/limAssocUtils.c
+++ b/CORE/MAC/src/pe/lim/limAssocUtils.c
@@ -1825,7 +1825,7 @@
     isArate = 0;
 
     /* copy operational rate set from psessionEntry */
-    if ( psessionEntry->rateSet.numRates < SIR_MAC_RATESET_EID_MAX )
+    if ( psessionEntry->rateSet.numRates <= SIR_MAC_RATESET_EID_MAX )
     {
         palCopyMemory(pMac->hHdd,(tANI_U8 *)tempRateSet.rate,(tANI_U8*)(psessionEntry->rateSet.rate), psessionEntry->rateSet.numRates);
         tempRateSet.numRates = psessionEntry->rateSet.numRates;
@@ -1838,7 +1838,7 @@
     if (psessionEntry->dot11mode == WNI_CFG_DOT11_MODE_11G)
     {
 
-        if (psessionEntry->extRateSet.numRates < SIR_MAC_RATESET_EID_MAX)
+        if (psessionEntry->extRateSet.numRates <= SIR_MAC_RATESET_EID_MAX)
         {
             palCopyMemory(pMac->hHdd,(tANI_U8 *)tempRateSet2.rate, (tANI_U8*)(psessionEntry->extRateSet.rate), psessionEntry->extRateSet.numRates);
             tempRateSet2.numRates = psessionEntry->extRateSet.numRates;
@@ -1850,7 +1850,7 @@
     }
     else
         tempRateSet2.numRates = 0;
-    if ((tempRateSet.numRates + tempRateSet2.numRates) > 12)
+    if ((tempRateSet.numRates + tempRateSet2.numRates) > SIR_MAC_RATESET_EID_MAX)
     {
         //we are in big trouble
         limLog(pMac, LOGP, FL("more than 12 rates in CFG"));