apps/ls.cpp

@@ -8,8 +8,8 @@
 int main(int argc, char** argv)
 {
     StringView pathname;
-    bool show_all { false };
+    bool show_all;
-    bool show_almost_all { false };
+    bool show_almost_all;
 
     ArgumentParser parser;
     parser.add_positional_argument(pathname, "directory"_sv, "/"_sv);

kernel/src/sys/open.cpp

@@ -44,8 +44,6 @@ Result<u64> sys_open(Registers*, SyscallArgs args)
         if ((flags & O_WRONLY) && (inode->mode() & S_IWUSR) == 0) return err(EACCES);
     }
 
-    if(inode->type() != VFS::InodeType::Directory && (flags & O_DIRECTORY)) return err(ENOTDIR);
-
     if ((flags & O_WRONLY) && (flags & O_TRUNC)) inode->truncate(0);
 
     int fd = TRY(current->allocate_fd(0));

libc/include/bits/open-flags.h

@@ -12,7 +12,6 @@
 #define O_TRUNC 32
 #define O_NONBLOCK 64
 #define O_CLOEXEC 128
-#define O_DIRECTORY 256
 
 #define O_ACCMODE O_RDWR
 

libc/src/dirent.cpp

@@ -13,7 +13,8 @@ extern "C"
         DIR* dp = (DIR*)malloc(sizeof(DIR));
         if (!dp) { return nullptr; }
 
-        int fd = open(path, O_RDONLY | O_DIRECTORY);
+        // FIXME: Use O_DIRECTORY (validate that path is actually a directory)
+        int fd = open(path, O_RDONLY);
         if (fd < 0)
         {
             free(dp);