Userland: Rely on a single authoritative source for the default `PATH`

2 files changed, 6 insertions, 2 deletions

diff --git a/Userland/Libraries/LibCore/File.h b/Userland/Libraries/LibCore/File.h
index 6037ff4c99..c331deaf66 100644
--- a/Userland/Libraries/LibCore/File.h
+++ b/Userland/Libraries/LibCore/File.h
@@ -11,6 +11,10 @@
 #include <LibCore/IODevice.h>
 #include <sys/stat.h>
 
+// FIXME: Make this a bit prettier.
+#define DEFAULT_PATH "/usr/local/sbin:/usr/local/bin:/usr/bin:/bin"
+#define DEFAULT_PATH_SV "/usr/local/sbin:/usr/local/bin:/usr/bin:/bin"sv
+
 namespace Core {
 
 class File final : public IODevice {
diff --git a/Userland/Libraries/LibCore/System.cpp b/Userland/Libraries/LibCore/System.cpp
index e2f277cdcd..8ffcd93870 100644
--- a/Userland/Libraries/LibCore/System.cpp
+++ b/Userland/Libraries/LibCore/System.cpp
@@ -958,7 +958,7 @@ ErrorOr<String> find_file_in_path(StringView filename)
     auto const* path_ptr = getenv("PATH");
     StringView path { path_ptr, strlen(path_ptr) };
     if (path.is_empty())
-        path = "/bin:/usr/bin"sv;
+        path = DEFAULT_PATH_SV;
     auto parts = path.split_view(':');
     for (auto& part : parts) {
         auto candidate = String::formatted("{}/{}", part, filename);
@@ -1043,7 +1043,7 @@ ErrorOr<void> exec(StringView filename, Span<StringView> arguments, SearchInPath
             ScopedValueRollback errno_rollback(errno);
             String path = getenv("PATH");
             if (path.is_empty())
-                path = "/bin:/usr/bin";
+                path = DEFAULT_PATH;
             auto parts = path.split(':');
             for (auto& part : parts) {
                 auto candidate = String::formatted("{}/{}", part, filename);