Fix crash when calling absPath with empty input

Signed-off-by: Zhilei Ren <[email protected]>
gazebosim · Aug 1, 2024 · 7c47da2 · 7c47da2
1 parent ad1d5ad
commit 7c47da2
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/Filesystem.cc b/src/Filesystem.cc
@@ -135,7 +135,14 @@ std::string common::copyToUnixPath(const std::string &_path)
 /////////////////////////////////////////////////
 std::string common::absPath(const std::string &_path)
 {
-  return fs::absolute(_path).string();
+  std::error_code ec;
+  auto path = fs::absolute(_path, ec);
+  if (!fsWarn("absPath", ec))
+  {
+    path = "";
+  }
+
+  return path.string();
 }
 
 /////////////////////////////////////////////////

diff --git a/src/Filesystem_TEST.cc b/src/Filesystem_TEST.cc
@@ -636,3 +636,9 @@ TEST_F(FilesystemTest, separator)
   EXPECT_EQ("\\", gz::common::separator(""));
 #endif
 }
+
+/////////////////////////////////////////////////
+TEST_F(FilesystemTest, empty)
+{
+  EXPECT_EQ(common::absPath(""), "");
+}