diff --git a/base/include/string_ex.h b/base/include/string_ex.h
index bcb7d4b12c572a36cb4fbeac5f5bc8db47619224..41be2adcd377d03e36d21f2fb118e04c9bfc6eeb 100755
--- a/base/include/string_ex.h
+++ b/base/include/string_ex.h
@@ -118,13 +118,13 @@ bool IsAsciiString(const std::string& str);
 
 /**
  * The str16ToStr8 function convert string16 to string8.
- * If covert failed, return an empty string.
+ * If convert failed, return an empty string.
  */
 std::string Str16ToStr8(const std::u16string& str16);
 
 /**
  * The Str8ToStr16 function convert string8 to string16.
- * If covert failed, return an empty u16string.
+ * If convert failed, return an empty u16string.
  */
 std::u16string Str8ToStr16(const std::string& str);
 } // namespace OHOS
diff --git a/base/src/string_ex.cpp b/base/src/string_ex.cpp
index 666522311a6c6edfa198e2d151b7eaac352d3b62..65ce7d248c9c31e84665db25c3e08fe4f2df9a5e 100755
--- a/base/src/string_ex.cpp
+++ b/base/src/string_ex.cpp
@@ -216,14 +216,12 @@ string::size_type GetFirstSubStrBetween(const string& str, const string& left,
 void GetSubStrBetween(const string& str, const string& left, const string& right, vector<string>& sub)
 {
     string subString;
-    string::size_type pos = 0;
     string strTmp = str;
+    string::size_type pos = GetFirstSubStrBetween(strTmp, left, right, subString);
     while (pos != string::npos) {
+        sub.push_back(subString);
+        strTmp = strTmp.substr(pos);
         pos = GetFirstSubStrBetween(strTmp, left, right, subString);
-        if (pos != string::npos) {
-            sub.push_back(subString);
-            strTmp = strTmp.substr(pos);
-        }
     }
 }