common/str_map.cc: do boundry checks before access

Signed-off-by: Danny Al-Gaaf <[email protected]>
areed · Apr 13, 2018 · 84b8387 · 84b8387
1 parent 1de5d14
commit 84b8387
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/common/str_map.cc b/src/common/str_map.cc
@@ -59,10 +59,10 @@ int get_json_str_map(
 string trim(const string& str) {
   size_t start = 0;
   size_t end = str.size() - 1;
-  while (isspace(str[start]) != 0 && start <= end) {
+  while (start <= end && isspace(str[start]) != 0) {
     ++start;
   }
-  while (isspace(str[end]) != 0 && start <= end) {
+  while (start <= end && isspace(str[end]) != 0) {
     --end;
   }
   if (start <= end) {