src: do not read string out of bounds

zcbenz · targos · commit 36e42aa570dc · 2024-02-15T12:00:56.000+01:00
PR-URL: #51358 Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io> Reviewed-By: Keyhan Vakil <kvakil@sylph.kvakil.me> Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
diff --git a/src/path.cc b/src/path.cc
@@ -24,15 +24,14 @@ std::string NormalizeString(const std::string_view path,
   int lastSegmentLength = 0;
   int lastSlash = -1;
   int dots = 0;
-  char code;
-  const auto pathLen = path.size();
-  for (uint8_t i = 0; i <= pathLen; ++i) {
-    if (i < pathLen) {
+  char code = 0;
+  for (size_t i = 0; i <= path.size(); ++i) {
+    if (i < path.size()) {
       code = path[i];
-    } else if (IsPathSeparator(path[i])) {
+    } else if (IsPathSeparator(code)) {
       break;
     } else {
-      code = node::kPathSeparator;
+      code = '/';
     }
 
     if (IsPathSeparator(code)) {
