Kernel: Make FileSystem::initialize() return KResult

This forced me to also come up with error codes for a bunch of
situations where we'd previously just panic the kernel.

1 files changed, 9 insertions, 12 deletions

diff --git a/Kernel/FileSystem/ISO9660FileSystem.cpp b/Kernel/FileSystem/ISO9660FileSystem.cpp
index 3d2b852f0c..127efe9a47 100644
--- a/Kernel/FileSystem/ISO9660FileSystem.cpp
+++ b/Kernel/FileSystem/ISO9660FileSystem.cpp
@@ -202,18 +202,15 @@ ISO9660FS::~ISO9660FS()
 {
 }
 
-bool ISO9660FS::initialize()
-{
-    if (!BlockBasedFileSystem::initialize())
-        return false;
-
-    // FIXME: Fix the FileSystem::initialize contract to be able to return a
-    //        KResult.
-    if (parse_volume_set().is_error())
-        return false;
-    if (create_root_inode().is_error())
-        return false;
-    return true;
+KResult ISO9660FS::initialize()
+{
+    if (auto result = BlockBasedFileSystem::initialize(); result.is_error())
+        return result;
+    if (auto result = parse_volume_set(); result.is_error())
+        return result;
+    if (auto result = create_root_inode(); result.is_error())
+        return result;
+    return KSuccess;
 }
 
 Inode& ISO9660FS::root_inode()