common/str_map.cc: do boundry checks before access

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>

diff --git a/src/common/str_map.cc b/src/common/str_map.cc
index c3f6a2b6519e3c5b2597fd0ed7082de8c044d67e..6e952ab8419d0ec07da0a09880d7f78c81af7105 100644 (file)
--- 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) {