commit ac145c256c60ab4fc13a83c7c89da9bc1455f9f3
author Anurag Chouhan <achouhan@qti.qualcomm.com> Tue Nov 22 16:51:47 2016 +0530
committer Anjaneedevi Kapparapu <akappa@codeaurora.org> Thu Dec 15 16:00:10 2016 +0530
tree 853273a5fd2a8d62f1b0e7b56ea3aa63febf237a
parent 83026000fed0ac84e94e99d0e1ee9a0d0c6f4f09

wlan: Modify hdd_string_to_u8_array to pass separator

Add separator as a parameter to the hdd_string_to_u8_array api.

Change-Id: Ic3a28260dc4cb9144230947f2f106016ccf4d70c
CRs-Fixed: 1093144

CORE/HDD/src/wlan_hdd_cfg.c

diff --git a/CORE/HDD/src/wlan_hdd_cfg.c b/CORE/HDD/src/wlan_hdd_cfg.c
index aedea63..29ee032 100644
--- a/CORE/HDD/src/wlan_hdd_cfg.c
+++ b/CORE/HDD/src/wlan_hdd_cfg.c
@@ -4909,7 +4909,8 @@
 }
 
 VOS_STATUS hdd_string_to_u8_array(char *str, tANI_U8 *intArray,
-				   tANI_U8 *len, tANI_U8 intArrayMaxLen)
+				   tANI_U8 *len, tANI_U8 intArrayMaxLen,
+				   char *seperator)
 {
    char *s = str;
 
@@ -4926,10 +4927,12 @@
       //Any other return value means error. Ignore it.
       if( sscanf(s, "%d", &val ) == 1 )
       {
+         if (val > 255 || val < 0)
+             return VOS_STATUS_E_FAILURE;
          intArray[*len] = (tANI_U8) val;
          *len += 1;
       }
-      s = strpbrk( s, "," );
+      s = strpbrk( s, seperator);
       if( s )
          s++;
    }
@@ -6252,7 +6255,7 @@
    hdd_string_to_u8_array( pConfig->neighborScanChanList,
                                         smeConfig->csrConfig.neighborRoamConfig.neighborScanChanList.channelList,
                                         &smeConfig->csrConfig.neighborRoamConfig.neighborScanChanList.numChannels,
-                                        WNI_CFG_VALID_CHANNEL_LIST_LEN );
+                                        WNI_CFG_VALID_CHANNEL_LIST_LEN, "," );
 #endif
 
    smeConfig->csrConfig.addTSWhenACMIsOff = pConfig->AddTSWhenACMIsOff;