commit 7db41f38d2f25668e162a8fc8b5d151e5d07cdd9
author Deepthi Gowri <c_gowri@qti.qualcomm.com> Mon Oct 13 17:02:29 2014 +0530
committer Satyanarayana Dash <c_sadash@qti.qualcomm.com> Wed Oct 15 16:55:59 2014 +0530
tree f653ba8d4782b6b61026b96b1b663004d6dc7c35
parent 6db9d533171c2796955bf4e4a902f2069a3c8af5

wlan: Fix for issue reported by Static Analysis tool

In WLANSAP_SetChannelRange(), type of operatingBand variable is
changed to eSapOperatingBand enum type,  but the enums used in
switch case are of 'eRfSubBand' enum type. Avoid this inconsistent
labelling by using proper enums.

Change-Id: Ic542a3dc2317665dc7e1296eedb37496f1f30240
CRs-Fixed: 738042

CORE/SAP/src/sapModule.c

diff --git a/CORE/SAP/src/sapModule.c b/CORE/SAP/src/sapModule.c
index 1f9c35e..445b604 100644
--- a/CORE/SAP/src/sapModule.c
+++ b/CORE/SAP/src/sapModule.c
@@ -1364,17 +1364,17 @@
     }
     switch(operatingBand)
     {
-       case RF_SUBBAND_2_4_GHZ:
+       case eSAP_RF_SUBBAND_2_4_GHZ:
           bandStartChannel = RF_CHAN_1;
           bandEndChannel = RF_CHAN_14;
           break;
 
-       case RF_SUBBAND_5_LOW_GHZ:
+       case eSAP_RF_SUBBAND_5_LOW_GHZ:
           bandStartChannel = RF_CHAN_36;
           bandEndChannel = RF_CHAN_64;
           break;
 
-       case RF_SUBBAND_5_MID_GHZ:
+       case eSAP_RF_SUBBAND_5_MID_GHZ:
           bandStartChannel = RF_CHAN_100;
 #ifndef FEATURE_WLAN_CH144
           bandEndChannel = RF_CHAN_140;
@@ -1383,11 +1383,16 @@
 #endif /* FEATURE_WLAN_CH144 */
           break;
 
-       case RF_SUBBAND_5_HIGH_GHZ:
+       case eSAP_RF_SUBBAND_5_HIGH_GHZ:
           bandStartChannel = RF_CHAN_149;
           bandEndChannel = RF_CHAN_165;
           break;
 
+       case eSAP_RF_SUBBAND_5_ALL_GHZ:
+          bandStartChannel = RF_CHAN_36;
+          bandEndChannel = RF_CHAN_165;
+          break;
+
        default:
           VOS_TRACE( VOS_MODULE_ID_SAP, VOS_TRACE_LEVEL_ERROR,
                    "Invalid operatingBand value on WLANSAP_SetChannelRange");