Merged in MHR-18 (pull request #9)

MHR-18

1 files changed, 342 insertions, 261 deletions

diff --git a/Private/Source/FS/NewFS.cxx b/Private/Source/FS/NewFS.cxx
index a5dfe45b..57f61cd2 100644
--- a/Private/Source/FS/NewFS.cxx
+++ b/Private/Source/FS/NewFS.cxx
@@ -6,21 +6,16 @@
 
 #ifdef __FSKIT_NEWFS__
 
+#include <Builtins/AHCI/AHCI.hxx>
+#include <Builtins/ATA/ATA.hxx>
 #include <FSKit/NewFS.hxx>
 #include <KernelKit/HError.hpp>
 #include <NewKit/Crc32.hpp>
-#include <NewKit/Utils.hpp>
 #include <NewKit/String.hpp>
-#include <Builtins/ATA/ATA.hxx>
-#include <Builtins/AHCI/AHCI.hxx>
+#include <NewKit/Utils.hpp>
 
 using namespace NewOS;
 
-/// forward decl.
-
-STATIC Lba ke_find_free_fork(SizeT sz, Int32 drv, NewCatalog* catalog, Boolean isDataFork);
-STATIC Lba ke_find_free_catalog(SizeT kind, Int32 drv);
-
 STATIC MountpointInterface sMountpointInterface;
 
 /// @brief Creates a new fork inside the New filesystem partition.
@@ -30,67 +25,39 @@ STATIC MountpointInterface sMountpointInterface;
 _Output NewFork* NewFSParser::CreateFork(_Input NewCatalog* catalog,
                                          _Input NewFork& theFork) {
   if (catalog && theFork.Name[0] != 0 && theFork.DataSize > 0) {
-    Lba whereFork = 0;
-
-    theFork.DataOffset =
-        ke_find_free_fork(theFork.DataSize, this->fDriveIndex, catalog, theFork.Kind == kNewFSDataForkKind);
-    theFork.Flags |= kNewFSFlagCreated;
-    Lba lba = theFork.Kind == kNewFSDataForkKind ? catalog->DataFork : catalog->ResourceFork;
-
-    if (lba == 0) {
-      lba = whereFork;
-    } else {
-      if (lba == 0) {
-        theFork.PreviousSibling = lba;
-      }
-    }
+    theFork.Flags = kNewFSFlagCreated;
+    Lba lba = (theFork.Kind == kNewFSDataForkKind) ? catalog->DataFork
+                                                   : catalog->ResourceFork;
 
-    if (theFork.Kind == kNewFSDataForkKind) {
-        if (catalog->DataFork == 0) {
-          catalog->DataFork = whereFork;
-        } else {
-          theFork.PreviousSibling = catalog->DataFork;
-        }
-    } else {
-        if (catalog->ResourceFork == 0) {
-          catalog->ResourceFork = whereFork;
-        } else {
-          theFork.PreviousSibling = catalog->ResourceFork;
-        }
-    }
+    if (lba <= kNewFSCatalogStartAddress) return nullptr;
 
-    if (!sMountpointInterface.GetAddressOf(this->fDriveIndex))
-      return nullptr;
+    theFork.DataOffset = lba + sizeof(NewFork);
+
+    if (!sMountpointInterface.GetAddressOf(this->fDriveIndex)) return nullptr;
 
     auto drv = sMountpointInterface.GetAddressOf(this->fDriveIndex);
 
-    drv->fPacket.fLba = whereFork;
-    drv->fPacket.fPacketSize = theFork.DataSize;
-    drv->fPacket.fPacketContent = (VoidPtr)&theFork;
+    Char cpyFork[sizeof(NewFork)] = {0};
 
-    rt_copy_memory((VoidPtr) "fs/newfs-packet", drv->fPacket.fPacketMime, 16);
+    drv->fPacket.fLba = lba + sizeof(NewCatalog);
+    drv->fPacket.fPacketSize = sizeof(NewFork);
+    drv->fPacket.fPacketContent = (VoidPtr)cpyFork;
 
-    if (auto res =
-            fs_newfs_write(&sMountpointInterface, *drv, this->fDriveIndex);
-        res) {
-      switch (res) {
-        case 1:
-          DbgLastError() = kErrorDiskReadOnly;
-          break;
-        case 2:
-          DbgLastError() = kErrorDiskIsFull;
-          break;
-          DbgLastError() = kErrorNoSuchDisk;
-          break;
+    drv->fInput(&drv->fPacket);
 
-        default:
-          break;
-      }
-      return nullptr;
-    }
+    NewFork* cpyForkStruct = (NewFork*)cpyFork;
+
+    if (cpyForkStruct->Flags == kNewFSFlagCreated) return nullptr;
+
+    /// special treatment.
+    rt_copy_memory((VoidPtr) "fs/newfs-packet", drv->fPacket.fPacketMime,
+                   rt_string_len("fs/newfs-packet"));
+
+    drv->fPacket.fLba = lba;
+    drv->fPacket.fPacketSize = sizeof(NewFork);
+    drv->fPacket.fPacketContent = (VoidPtr)&theFork;
 
-    /// Also update catalog.
-    this->WriteCatalog(catalog, nullptr);
+    drv->fOutput(&drv->fPacket);
 
     return &theFork;
   }
@@ -103,7 +70,8 @@ _Output NewFork* NewFSParser::CreateFork(_Input NewCatalog* catalog,
 /// @param name the fork name.
 /// @return the fork.
 _Output NewFork* NewFSParser::FindFork(_Input NewCatalog* catalog,
-                                       _Input const Char* name, Boolean isDataFork) {
+                                       _Input const Char* name,
+                                       Boolean isDataFork) {
   auto drv = sMountpointInterface.GetAddressOf(this->fDriveIndex);
   NewFork* theFork = nullptr;
   Lba lba = isDataFork ? catalog->DataFork : catalog->ResourceFork;
@@ -160,6 +128,186 @@ _Output NewCatalog* NewFSParser::CreateCatalog(_Input const char* name) {
 _Output NewCatalog* NewFSParser::CreateCatalog(_Input const char* name,
                                                _Input const Int32& flags,
                                                _Input const Int32& kind) {
+  if (!sMountpointInterface.GetAddressOf(this->fDriveIndex)) return nullptr;
+
+  Lba outLba = 0UL;
+
+  if (kind == kNewFSCatalogKindDir &&
+      name[rt_string_len(name) - 1] != NewFilesystemHelper::Separator())
+    return nullptr;
+
+  /// as well as this, we don't want that.
+  if (kind != kNewFSCatalogKindDir &&
+      name[rt_string_len(name) - 1] == NewFilesystemHelper::Separator())
+    return nullptr;
+
+  NewCatalog* checkForCpy = this->FindCatalog(name, outLba);
+
+  if (checkForCpy) {
+    return checkForCpy;
+  }
+
+  char copyName[kNewFSNodeNameLen] = {0};
+
+  for (SizeT indexName = 0UL; indexName < rt_string_len(name); ++indexName) {
+    copyName[indexName] = name[indexName];
+  }
+
+  if (*copyName == 0) {
+    DbgLastError() = kErrorFileNotFound;
+    return nullptr;
+  }
+
+  for (SizeT indexFill = 0; indexFill < rt_string_len(name); ++indexFill) {
+    copyName[indexFill] = name[indexFill];
+  }
+
+  SizeT indexReverseCopy = rt_string_len(copyName);
+
+  // zero character.
+  copyName[--indexReverseCopy] = 0;
+
+  // mandatory / character.
+  copyName[--indexReverseCopy] = 0;
+
+  while (copyName[indexReverseCopy] != '/') {
+    copyName[indexReverseCopy] = 0;
+    --indexReverseCopy;
+  }
+
+  NewCatalog* catalog = this->FindCatalog(copyName, outLba);
+
+  if (!catalog) {
+    return nullptr;
+  }
+
+  if (catalog->Kind == kNewFSCatalogKindFile) {
+    delete catalog;
+    return nullptr;
+  }
+
+  constexpr SizeT cDefaultForkSize = 8192;
+
+  NewCatalog* catalogChild = new NewCatalog();
+
+  catalogChild->ResourceForkOverallSize = cDefaultForkSize;
+  catalogChild->DataForkSize = cDefaultForkSize;
+
+  catalogChild->NextSibling = 0UL;
+  catalogChild->PrevSibling = outLba;
+  catalogChild->Flags = flags;
+  catalogChild->Kind = kind;
+  catalogChild->Flags |= kNewFSFlagCreated;
+
+  rt_copy_memory((VoidPtr)name, (VoidPtr)catalogChild->Name,
+                 rt_string_len(name));
+
+  UInt16 sectorBuf[kNewFSMinimumSectorSz] = {0};
+
+  auto drive = sMountpointInterface.GetAddressOf(this->fDriveIndex);
+
+  Lba startFree = catalogChild->PrevSibling + catalog->NextSibling;
+
+  kcout << "Next-Catalog: " << hex_number(startFree) << endl;
+
+  drive->fPacket.fPacketContent = sectorBuf;
+  drive->fPacket.fPacketSize = kNewFSMinimumSectorSz;
+  drive->fPacket.fLba = startFree;
+
+  drive->fInput(&drive->fPacket);
+
+  while (drive->fPacket.fPacketGood) {
+    NewCatalog* nextSibling = (NewCatalog*)sectorBuf;
+
+    if (startFree <= kNewFSAddressAsLba) {
+      delete catalogChild;
+      delete catalog;
+
+      return nullptr;
+    }
+
+    /// allocation or reallocation or catalog...
+    if ((nextSibling->Name[0] == 0 &&
+         nextSibling->Flags != kNewFSFlagCreated)) {
+      catalogChild->DataFork = startFree + sizeof(NewCatalog);
+      catalogChild->ResourceFork =
+          startFree + sizeof(NewCatalog) + catalogChild->DataForkSize;
+
+      if ((nextSibling->Flags != kNewFSFlagCreated)) {
+        nextSibling->Flags = kNewFSFlagCreated;
+
+        drive->fPacket.fPacketContent = catalogChild;
+        drive->fPacket.fPacketSize = sizeof(NewCatalog);
+        drive->fPacket.fLba = startFree;
+
+        if (catalogChild->Kind == kNewFSCatalogKindFile) {
+          rt_copy_memory((VoidPtr) "x-kind/file", (VoidPtr)catalogChild->Mime,
+                         rt_string_len("x-kind/file"));
+        } else {
+          rt_copy_memory((VoidPtr) "x-kind/dir", (VoidPtr)catalogChild->Mime,
+                         rt_string_len("x-kind/dir"));
+        }
+
+        catalogChild->NextSibling = sizeof(NewCatalog) +
+                                    catalogChild->DataForkSize +
+                                    catalogChild->ResourceForkOverallSize;
+
+        drive->fOutput(&drive->fPacket);
+
+        kcout << "New OS: Create new catalog successfully!\r\n";
+
+        Char sectBuf[sizeof(NewCatalog)] = {0};
+
+        drive->fPacket.fPacketContent = sectBuf;
+        drive->fPacket.fPacketSize = sizeof(NewCatalog);
+        drive->fPacket.fLba = catalogChild->PrevSibling;
+
+        drive->fInput(&drive->fPacket);
+
+        NewCatalog* prevCatalog = (NewCatalog*)sectBuf;
+        prevCatalog->NextSibling = startFree;
+
+        drive->fOutput(&drive->fPacket);
+
+        kcout << "Edit-Catalog: " << prevCatalog->Name << endl;
+
+        Char sectorBufPartBlock[kNewFSMinimumSectorSz] = {0};
+
+        drive->fPacket.fPacketContent = sectorBufPartBlock;
+        drive->fPacket.fPacketSize = kNewFSMinimumSectorSz;
+        drive->fPacket.fLba = kNewFSAddressAsLba;
+
+        drive->fInput(&drive->fPacket);
+
+        NewPartitionBlock* partBlock = (NewPartitionBlock*)sectorBufPartBlock;
+
+        partBlock->SectorCount -= 1;
+        partBlock->CatalogCount += 1;
+        partBlock->FreeCatalog -= 1;
+
+        drive->fOutput(&drive->fPacket);
+
+        delete catalog;
+        return catalogChild;
+      } else {
+        delete catalog;
+        return nullptr;
+      }
+    }
+
+    //// @note that's how we find the next catalog in the partition block.
+    startFree += sizeof(NewFork) + sizeof(NewCatalog) +
+                 nextSibling->DataForkSize +
+                 nextSibling->ResourceForkOverallSize;
+
+    drive->fPacket.fPacketContent = sectorBuf;
+    drive->fPacket.fPacketSize = kNewFSMinimumSectorSz;
+    drive->fPacket.fLba = startFree;
+
+    drive->fInput(&drive->fPacket);
+  }
+
+  delete catalog;
   return nullptr;
 }
 
@@ -176,7 +324,7 @@ bool NewFSParser::Format(_Input _Output DriveTrait* drive) {
     return false;
   }
 
-  UInt16 sectorBuf[kNewFSMinimumSectorSz] = {0};
+  Char sectorBuf[kNewFSMinimumSectorSz] = {0};
 
   drive->fPacket.fPacketContent = sectorBuf;
   drive->fPacket.fPacketSize = kNewFSMinimumSectorSz;
@@ -193,18 +341,24 @@ bool NewFSParser::Format(_Input _Output DriveTrait* drive) {
         rt_string_cmp(partBlock->Ident, kNewFSIdent, kNewFSIdentLen)) {
       /// partition is free and valid.
 
+      partBlock->Version = kNewFSVersionInteger;
+
+      const auto cUntitledHD = "New OS HD\0";
+
       rt_copy_memory((VoidPtr)kNewFSIdent, (VoidPtr)partBlock->Ident,
                      kNewFSIdentLen);
-      rt_copy_memory((VoidPtr) "Untitled HD\0", (VoidPtr)partBlock->PartitionName,
-                     rt_string_len("Untitled HD\0"));
+
+      rt_copy_memory((VoidPtr)cUntitledHD, (VoidPtr)partBlock->PartitionName,
+                     rt_string_len(cUntitledHD));
 
       SizeT catalogCount = 0;
       SizeT sectorCount = drv_std_get_sector_count();
       SizeT diskSize = drv_std_get_drv_size();
 
       partBlock->Kind = kNewFSPartitionTypeStandard;
-      partBlock->StartCatalog = kNewFSCatalogStartAddress;;
-      partBlock->CatalogCount = catalogCount;
+      partBlock->StartCatalog = kNewFSCatalogStartAddress;
+      partBlock->Flags |= kNewFSPartitionTypeStandard;
+      partBlock->CatalogCount = sectorCount / sizeof(NewCatalog);
       partBlock->SectorCount = sectorCount;
       partBlock->DiskSize = diskSize;
       partBlock->FreeCatalog = partBlock->StartCatalog;
@@ -218,7 +372,7 @@ bool NewFSParser::Format(_Input _Output DriveTrait* drive) {
       return true;
     }
 
-    kcout << "New OS: Partition already exists.\r\n";
+    kcout << "New OS: PartitionBlock already exists.\r\n";
 
     /// return success as well, do not ignore that partition.
     return true;
@@ -227,92 +381,141 @@ bool NewFSParser::Format(_Input _Output DriveTrait* drive) {
   return false;
 }
 
-/// @brief
-/// @param catalog
-/// @param data
-/// @return
-bool NewFSParser::WriteCatalog(_Input _Output NewCatalog* catalog,
-                               voidPtr data) {
+/// @brief Writes the data fork into a specific catalog.
+/// @param catalog the catalog itself
+/// @param data the data.
+/// @return if the catalog w rote the contents successfully.
+bool NewFSParser::WriteCatalog(_Input _Output NewCatalog* catalog, voidPtr data,
+                               SizeT sizeOfData) {
+  if (sizeOfData > catalog->DataForkSize) return false;
+  if (!sMountpointInterface.GetAddressOf(this->fDriveIndex)) return false;
+
+  NewFork* forkData = new NewFork();
+
+  auto drive = sMountpointInterface.GetAddressOf(this->fDriveIndex);
+
+  auto startFork = catalog->DataFork;
+
+  /// sanity check of the fork position as the condition to run the loop.
+  if (startFork >= kNewFSCatalogStartAddress) {
+    drive->fPacket.fPacketContent = forkData;
+    drive->fPacket.fPacketSize = sizeof(NewFork);
+    drive->fPacket.fLba = startFork;
+
+    drive->fInput(&drive->fPacket);
+
+    /// check if fork even exists.
+    if (forkData->Flags != kNewFSFlagCreated) {
+      DbgLastError() = kErrorDiskIsCorrupted;
+
+      delete forkData;
+      return false;
+    }
+
+    /// sanity check the fork.
+    if (forkData->DataOffset <= kNewFSCatalogStartAddress) {
+      DbgLastError() = kErrorDiskIsCorrupted;
+
+      delete forkData;
+      return false;
+    }
+
+    drive->fPacket.fPacketContent = data;
+    drive->fPacket.fPacketSize = sizeOfData;
+    drive->fPacket.fLba = forkData->DataOffset;
+
+    drive->fOutput(&drive->fPacket);
+
+    return true;
+  }
+
   return false;
 }
 
 /// @brief
 /// @param catalogName
 /// @return
-_Output NewCatalog* NewFSParser::FindCatalog(_Input const char* catalogName) {
-    if (!sMountpointInterface.GetAddressOf(this->fDriveIndex)) return nullptr;
+_Output NewCatalog* NewFSParser::FindCatalog(_Input const char* catalogName,
+                                             Lba& outLba) {
+  if (!sMountpointInterface.GetAddressOf(this->fDriveIndex)) return nullptr;
 
-    Char* sectorBuf = new Char[sizeof(NewPartitionBlock)];
-    auto drive = sMountpointInterface.GetAddressOf(this->fDriveIndex);
+  Char* sectorBuf = new Char[sizeof(NewPartitionBlock)];
+  auto drive = sMountpointInterface.GetAddressOf(this->fDriveIndex);
 
-    drive->fPacket.fPacketContent = sectorBuf;
-    drive->fPacket.fPacketSize = sizeof(NewPartitionBlock);
-    drive->fPacket.fLba = kNewFSAddressAsLba;
+  drive->fPacket.fPacketContent = sectorBuf;
+  drive->fPacket.fPacketSize = sizeof(NewPartitionBlock);
+  drive->fPacket.fLba = kNewFSAddressAsLba;
 
-    drive->fInput(&drive->fPacket);
+  drive->fInput(&drive->fPacket);
 
-    NewPartitionBlock* part = (NewPartitionBlock*)sectorBuf;
+  NewPartitionBlock* part = (NewPartitionBlock*)sectorBuf;
 
-    kcout << "Drive-Kind: " << drive->fDriveKind() << endl;
+  kcout << "Drive-Kind: " << drive->fDriveKind() << endl;
 
-    kcout << "Partition-Name: " << part->PartitionName << endl;
-    kcout << "Start-Catalog: " << number(part->StartCatalog) << endl;
-    kcout << "Catalog-Count: " << number(part->CatalogCount) << endl;
-    kcout << "Free-Catalog: " << number(part->FreeCatalog) << endl;
-    kcout << "Free-Sectors: " << number(part->FreeSectors) << endl;
-    kcout << "Sector-Size: " << number(part->SectorSize) << endl;
+  kcout << "Partition-Name: " << part->PartitionName << endl;
+  kcout << "Start-Catalog: " << number(part->StartCatalog) << endl;
+  kcout << "Catalog-Count: " << number(part->CatalogCount) << endl;
+  kcout << "Free-Catalog: " << number(part->FreeCatalog) << endl;
+  kcout << "Free-Sectors: " << number(part->FreeSectors) << endl;
+  kcout << "Sector-Size: " << number(part->SectorSize) << endl;
 
-    auto start = part->StartCatalog;
+  auto start = part->StartCatalog;
 
-    drive->fPacket.fLba = start;
-    drive->fPacket.fPacketContent = sectorBuf;
-    drive->fPacket.fPacketSize = sizeof(NewCatalog);
+  drive->fPacket.fLba = start;
+  drive->fPacket.fPacketContent = sectorBuf;
+  drive->fPacket.fPacketSize = sizeof(NewCatalog);
 
-    drive->fInput(&drive->fPacket);
+  drive->fInput(&drive->fPacket);
 
-    while (drive->fPacket.fPacketGood) {
-        NewCatalog* catalog = (NewCatalog*)sectorBuf;
+  while (drive->fPacket.fPacketGood) {
+    NewCatalog* catalog = (NewCatalog*)sectorBuf;
 
-        if (StringBuilder::Equals(catalogName, catalog->Name)) {
-            NewCatalog* catalogPtr = new NewCatalog();
-            rt_copy_memory(catalog, catalogPtr, sizeof(NewCatalog));
+    if (StringBuilder::Equals(catalogName, catalog->Name)) {
+      NewCatalog* catalogPtr = new NewCatalog();
+      rt_copy_memory(catalog, catalogPtr, sizeof(NewCatalog));
 
-            delete[] sectorBuf;
-            return catalogPtr;
-        }
+      outLba = start;
+      delete[] sectorBuf;
+      return catalogPtr;
+    }
 
-        if (catalog->NextSibling == 0)
-            break;
+    start = catalog->NextSibling;
 
-        drive->fPacket.fLba = catalog->NextSibling;
-        drive->fPacket.fPacketSize = kNewFSMinimumSectorSz;
-        drive->fInput(&drive->fPacket);
-    }
+    if (start <= kNewFSAddressAsLba) break;
+
+    drive->fPacket.fLba = start;
+    drive->fPacket.fPacketContent = sectorBuf;
+    drive->fPacket.fPacketSize = sizeof(NewCatalog);
 
-    delete[] sectorBuf;
+    drive->fInput(&drive->fPacket);
+  }
 
-    return nullptr;
+  outLba = 0UL;
+  delete[] sectorBuf;
+
+  return nullptr;
 }
 
 /// @brief
 /// @param name
 /// @return
 _Output NewCatalog* NewFSParser::GetCatalog(_Input const char* name) {
-  return this->FindCatalog(name);
+  Lba unused = 0;
+  return this->FindCatalog(name, unused);
 }
 
 /// @brief
 /// @param catalog
 /// @return
 Boolean NewFSParser::CloseCatalog(_Input _Output NewCatalog* catalog) {
-    if (this->WriteCatalog(catalog, nullptr)) {
-        delete catalog;
-        catalog = nullptr;
+  if (this->WriteCatalog(catalog, nullptr, 0)) {
+    delete catalog;
+    catalog = nullptr;
 
-        return true;
-    }
+    return true;
+  }
 
-    return false;
+  return false;
 }
 
 /// @brief Mark catalog as removed.
@@ -320,12 +523,12 @@ Boolean NewFSParser::CloseCatalog(_Input _Output NewCatalog* catalog) {
 /// @return
 Boolean NewFSParser::RemoveCatalog(_Input _Output NewCatalog* catalog) {
   if (!catalog) {
-      DbgLastError() = kErrorFileNotFound;
-      return false;
+    DbgLastError() = kErrorFileNotFound;
+    return false;
   }
 
   catalog->Flags |= kNewFSFlagDeleted;
-  this->WriteCatalog(catalog, nullptr);
+  this->WriteCatalog(catalog, nullptr, 0);
 
   return false;
 }
@@ -340,12 +543,12 @@ Boolean NewFSParser::RemoveCatalog(_Input _Output NewCatalog* catalog) {
 /// @return
 VoidPtr NewFSParser::ReadCatalog(_Input _Output NewCatalog* catalog,
                                  SizeT dataSz) {
-    if (!catalog) {
-        DbgLastError() = kErrorFileNotFound;
-        return nullptr;
-    }
-
+  if (!catalog) {
+    DbgLastError() = kErrorFileNotFound;
     return nullptr;
+  }
+
+  return nullptr;
 }
 
 /// @brief Seek in the data fork.
@@ -353,143 +556,21 @@ VoidPtr NewFSParser::ReadCatalog(_Input _Output NewCatalog* catalog,
 /// @param off
 /// @return
 bool NewFSParser::Seek(_Input _Output NewCatalog* catalog, SizeT off) {
-    if (!catalog) {
-        DbgLastError() = kErrorFileNotFound;
-        return false;
-    }
-
+  if (!catalog) {
+    DbgLastError() = kErrorFileNotFound;
     return false;
+  }
+
+  return false;
 }
 
 /// @brief Tell where we are inside the data fork.
 /// @param catalog
 /// @return
 SizeT NewFSParser::Tell(_Input _Output NewCatalog* catalog) {
-    if (!catalog) {
-        DbgLastError() = kErrorFileNotFound;
-        return false;
-    }
-
-    return 0;
-}
-
-/// @brief Find a free fork inside the filesystem.
-/// @param sz the size of the fork to set.
-/// @return the valid lba.
-STATIC Lba ke_find_free_fork(SizeT sz, Int32 drv, NewCatalog* catalog, Boolean isDataFork) {
-
-  if (sMountpointInterface.GetAddressOf(drv)) {
-    auto drive = *sMountpointInterface.GetAddressOf(drv);
-
-    /// prepare packet.
-    bool done = false;
-    bool error = false;
-
-    Lba lba = isDataFork ? catalog->DataFork : catalog->ResourceFork;
-
-    while (!done) {
-      UInt16 sectorBuf[kNewFSMinimumSectorSz] = {0};
-
-      drive.fPacket.fPacketContent = sectorBuf;
-      drive.fPacket.fPacketSize = kNewFSMinimumSectorSz;
-      drive.fPacket.fLba = lba;
-
-      drive.fInput(&drive.fPacket);
-
-      if (!drive.fPacket.fPacketGood) {
-        ///! not a lot of choices, disk has become unreliable.
-        if (ke_calculate_crc32((Char*)sectorBuf, kNewFSMinimumSectorSz) !=
-            drive.fPacket.fPacketCRC32) {
-          DbgLastError() = kErrorDiskIsCorrupted;
-        }
-
-        error = true;
-        break;
-      }
-
-      NewFork* fork = (NewFork*)sectorBuf;
-
-      if (fork->DataSize == 0 && fork->Name[0] == 0 &&
-          (fork->Flags == kNewFSFlagDeleted ||
-           fork->Flags == kNewFSFlagUnallocated)) {
-        fork->DataSize = sz;
-        fork->Flags |= kNewFSFlagCreated;
-
-        drive.fOutput(&drive.fPacket);
-
-        /// here it's either a read-only filesystem or something bad happened.'
-        if (!drive.fPacket.fPacketGood) {
-          DbgLastError() = kErrorDiskReadOnly;
-
-          return 0;
-        }
-
-        return lba;
-      }
-
-      lba += sizeof(NewFork);
-    }
-
-    if (error) {
-      DbgLastError() = kErrorDisk;
-      return 0;
-    }
-  }
-
-  return 0;
-}
-
-/// @brief find a free catalog.
-/// @param kind the catalog kind.
-/// @return the valid lba.
-STATIC Lba ke_find_free_catalog(SizeT kind, Int32 drv) {
-
-  if (sMountpointInterface.GetAddressOf(drv)) {
-    auto drive = *sMountpointInterface.GetAddressOf(drv);
-    UInt16 sectorBuf[kNewFSMinimumSectorSz] = {0};
-
-    /// prepare packet.
-
-    drive.fPacket.fPacketContent = sectorBuf;
-    drive.fPacket.fPacketSize = kNewFSMinimumSectorSz;
-    drive.fPacket.fLba = kNewFSAddressAsLba;
-
-    drive.fInput(&drive.fPacket);
-
-    NewPartitionBlock* partBlock = (NewPartitionBlock*)sectorBuf;
-
-    /// check for a valid partition.
-    if (partBlock->PartitionName[0] != 0 &&
-        rt_string_cmp(partBlock->Ident, kNewFSIdent, kNewFSIdentLen) == 0) {
-      auto startLba = partBlock->FreeCatalog;
-
-      if (startLba == 0) {
-        DbgLastError() = kErrorDiskIsFull;
-        return 1;
-      } else {
-        while (startLba != 0) {
-          drive.fPacket.fPacketContent = sectorBuf;
-          drive.fPacket.fPacketSize = kNewFSMinimumSectorSz;
-          drive.fPacket.fLba = startLba;
-
-          drive.fInput(&drive.fPacket);
-
-          NewCatalog* catalog = (NewCatalog*)sectorBuf;
-
-          if (catalog->Flags == kNewFSFlagUnallocated ||
-              catalog->Flags == kNewFSFlagDeleted) {
-            catalog->Flags |= kNewFSFlagCreated;
-            catalog->Kind |= kind;
-
-            return startLba;
-          }
-
-          startLba = catalog->NextSibling;
-        }
-
-        return 0;
-      }
-    }
+  if (!catalog) {
+    DbgLastError() = kErrorFileNotFound;
+    return false;
   }
 
   return 0;
@@ -497,14 +578,14 @@ STATIC Lba ke_find_free_catalog(SizeT kind, Int32 drv) {
 
 namespace NewOS::Detail {
 Boolean fs_init_newfs(Void) noexcept {
-    sMountpointInterface.A() = construct_main_drive();
-    sMountpointInterface.B() = construct_drive();
-    sMountpointInterface.C() = construct_drive();
-    sMountpointInterface.D() = construct_drive();
+  sMountpointInterface.A() = construct_main_drive();
+  sMountpointInterface.B() = construct_drive();
+  sMountpointInterface.C() = construct_drive();
+  sMountpointInterface.D() = construct_drive();
 
-    sMountpointInterface.A().fVerify(&sMountpointInterface.A().fPacket);
+  sMountpointInterface.A().fVerify(&sMountpointInterface.A().fPacket);
 
-    return true;
+  return true;
 }
 }  // namespace NewOS::Detail