Browse Source

FileSystem: Add FIXME about resolve_path bug

Rok Povsic 5 years ago
parent
commit
eb9ccf1c0a
1 changed files with 3 additions and 0 deletions
  1. 3 0
      Kernel/FileSystem/VirtualFileSystem.cpp

+ 3 - 0
Kernel/FileSystem/VirtualFileSystem.cpp

@@ -637,6 +637,9 @@ Custody& VFS::root_custody()
 
 KResultOr<NonnullRefPtr<Custody>> VFS::resolve_path(StringView path, Custody& base, RefPtr<Custody>* parent_custody, int options)
 {
+    // FIXME: resolve_path currently doesn't deal with .. and . . If path is ../. and base is /home/anon, it returns
+    //        /home/anon/../. instead of /home .
+
     if (path.is_empty())
         return KResult(-EINVAL);