fix: ShowItemUsingFileManager should escape path in Linux

Co-authored-by: hsfzxjy <hsfzxjy@gmail.com>
electron · Nov 20, 2023 · 1ee04e4 · 1ee04e4
1 parent ead3de1
commit 1ee04e4
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/shell/common/platform_util_linux.cc b/shell/common/platform_util_linux.cc
@@ -21,6 +21,7 @@
 #include "base/posix/eintr_wrapper.h"
 #include "base/process/kill.h"
 #include "base/process/launch.h"
+#include "base/strings/escape.h"
 #include "base/strings/string_util.h"
 #include "base/threading/thread_restrictions.h"
 #include "components/dbus/thread_linux/dbus_thread_linux.h"
@@ -216,8 +217,9 @@ class ShowItemHelper {
     dbus::MessageWriter writer(&show_items_call);
 
     writer.AppendArrayOfStrings(
-        {"file://" + full_path.value()});  // List of file(s) to highlight.
-    writer.AppendString({});               // startup-id
+        {"file://" + base::EscapePath(
+                         full_path.value())});  // List of file(s) to highlight.
+    writer.AppendString({});                    // startup-id
 
     ShowItemUsingBusCall(&show_items_call, full_path);
   }