Checking indices before dereference (#2040)

open-telemetry · Mar 14, 2023 · 15ab9ed · 15ab9ed
1 parent b046764
commit 15ab9ed
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/api/include/opentelemetry/common/string_util.h b/api/include/opentelemetry/common/string_util.h
@@ -14,11 +14,11 @@ class StringUtil
 public:
   static nostd::string_view Trim(nostd::string_view str, size_t left, size_t right) noexcept
   {
-    while (str[static_cast<std::size_t>(left)] == ' ' && left <= right)
+    while (left <= right && str[static_cast<std::size_t>(left)] == ' ')
     {
       left++;
     }
-    while (str[static_cast<std::size_t>(right)] == ' ' && left <= right)
+    while (left <= right && str[static_cast<std::size_t>(right)] == ' ')
     {
       right--;
     }