fix: Fix Windows is_absolute_path() crash on drive letter (#1338)

diff --git a/src/util/path.cpp b/src/util/path.cpp
index 29822ea3ed16e6f5d893c5313a2537b08002f784..3473cdd019956d84c38c3ab905ac532155592452 100644 (file)
--- a/src/util/path.cpp
+++ b/src/util/path.cpp
@@ -88,7 +88,7 @@ bool
 is_absolute_path(std::string_view path)
 {
 #ifdef _WIN32
-  if (path.length() >= 2 && path[1] == ':'
+  if (path.length() >= 3 && path[1] == ':'
       && (path[2] == '/' || path[2] == '\\')) {
     return true;
   }

diff --git a/unittest/test_util_path.cpp b/unittest/test_util_path.cpp
index 24d78558c4d0485cec725ba48305378045de93b4..1e6ff03c46d7e68750ea9ee655b11337f8f8b5ab 100644 (file)
--- a/unittest/test_util_path.cpp
+++ b/unittest/test_util_path.cpp
@@ -36,9 +36,11 @@ TEST_CASE("util::is_absolute_path")
 {
 #ifdef _WIN32
   CHECK(util::is_absolute_path("C:/"));
+  CHECK(util::is_absolute_path("C:\\"));
   CHECK(util::is_absolute_path("C:\\foo/fie"));
   CHECK(util::is_absolute_path("/C:\\foo/fie")); // MSYS/Cygwin path
   CHECK(!util::is_absolute_path(""));
+  CHECK(!util::is_absolute_path("C:"));
   CHECK(!util::is_absolute_path("foo\\fie/fum"));
   CHECK(!util::is_absolute_path("C:foo/fie"));
 #endif
@@ -48,7 +50,7 @@ TEST_CASE("util::is_absolute_path")
   CHECK(!util::is_absolute_path("foo/fie"));
 }
 
-TEST_CASE("util::is_absolute_path")
+TEST_CASE("util::is_full_path")
 {
   CHECK(!util::is_full_path(""));
   CHECK(!util::is_full_path("foo"));