Shell: Use lstat instead of access to check if glob target exists

Fixes #4905
Author: https://github.com/alimpfard Commit: https://github.com/SerenityOS/serenity/commit/a90905c54c8 Pull-request: https://github.com/SerenityOS/serenity/pull/4923 Issue: https://github.com/SerenityOS/serenity/issues/4905
2025-01-24 18:24:45 -05:00 · 2021-01-12 17:45:26 +03:30 · 2021-01-12 17:45:26 +03:30 · a90905c54c · 2024-07-18 23:53:23 +09:00
commit a90905c54c
parent b2b3ccb12d
1 changed files with 4 additions and 3 deletions
--- a/Userland/Shell/Shell.cpp
+++ b/Userland/Shell/Shell.cpp
@ -266,9 +266,10 @@ Vector<String> Shell::expand_globs(Vector<StringView> path_segments, const Strin
 {
    if (path_segments.is_empty()) {
        String base_str = base;
-        if (access(base_str.characters(), F_OK) == 0)
-            return { move(base_str) };
-        return {};
+        struct stat statbuf;
+        if (lstat(base_str.characters(), &statbuf) < 0)
+            return {};
+        return { move(base_str) };
    }

    auto first_segment = path_segments.take_first();