Kernel: Make Inode::lookup() return a KResultOr<NonnullRefPtr<Inode>>

This allows file systems to return arbitrary error codes instead of just
an Inode or not an Inode.

1 files changed, 7 insertions, 4 deletions

diff --git a/Kernel/FileSystem/DevPtsFS.cpp b/Kernel/FileSystem/DevPtsFS.cpp
index 80fc243211..ee5c2a4b46 100644
--- a/Kernel/FileSystem/DevPtsFS.cpp
+++ b/Kernel/FileSystem/DevPtsFS.cpp
@@ -136,21 +136,24 @@ KResult DevPtsFSInode::traverse_as_directory(Function<bool(FileSystem::Directory
     return KSuccess;
 }
 
-RefPtr<Inode> DevPtsFSInode::lookup(StringView name)
+KResultOr<NonnullRefPtr<Inode>> DevPtsFSInode::lookup(StringView name)
 {
     VERIFY(identifier().index() == 1);
 
     if (name == "." || name == "..")
-        return this;
+        return *this;
 
     auto& fs = static_cast<DevPtsFS&>(this->fs());
 
     auto pty_index = name.to_uint();
     if (pty_index.has_value() && s_ptys->contains(pty_index.value())) {
-        return fs.get_inode({ fsid(), pty_index_to_inode_index(pty_index.value()) });
+        auto inode = fs.get_inode({ fsid(), pty_index_to_inode_index(pty_index.value()) });
+        if (!inode)
+            return ENOENT;
+        return inode.release_nonnull();
     }
 
-    return {};
+    return ENOENT;
 }
 
 void DevPtsFSInode::flush_metadata()