From 6790ab170578594f26e8e84a3e57b80cb6094e21 Mon Sep 17 00:00:00 2001
From: Lukas Oesch <lukasoesch20@gmail.com>
Date: Tue, 28 Apr 2026 10:55:37 +0200
Subject: add is_symbolic_link method in ext2 inode

---
 .../include/kernel/filesystem/ext2/filesystem.hpp  |  1 +
 kernel/include/kernel/filesystem/ext2/inode.hpp    |  5 +++++
 kernel/include/kernel/filesystem/inode.hpp         |  6 ++++++
 kernel/src/filesystem/ext2/inode.cpp               |  5 +++++
 kernel/src/filesystem/ext2/inode.tests.cpp         | 24 ++++++++++++++++++++++
 kernel/src/filesystem/inode.cpp                    |  5 +++++
 6 files changed, 46 insertions(+)

(limited to 'kernel')

diff --git a/kernel/include/kernel/filesystem/ext2/filesystem.hpp b/kernel/include/kernel/filesystem/ext2/filesystem.hpp
index 516e71d..d22433f 100644
--- a/kernel/include/kernel/filesystem/ext2/filesystem.hpp
+++ b/kernel/include/kernel/filesystem/ext2/filesystem.hpp
@@ -35,6 +35,7 @@ namespace kernel::filesystem::ext2
     constexpr uint16_t inline mode_mask = 0xF000;
     constexpr uint16_t inline mode_regular = 0x8000;
     constexpr uint16_t inline mode_directory = 0x4000;
+    constexpr uint16_t inline mode_symbolic_link = 0xA000;
   }  // namespace constants
 
   /**
diff --git a/kernel/include/kernel/filesystem/ext2/inode.hpp b/kernel/include/kernel/filesystem/ext2/inode.hpp
index 64cdb06..688a1d8 100644
--- a/kernel/include/kernel/filesystem/ext2/inode.hpp
+++ b/kernel/include/kernel/filesystem/ext2/inode.hpp
@@ -81,6 +81,11 @@ namespace kernel::filesystem::ext2
      */
     [[nodiscard]] auto is_regular() const -> bool override;
 
+    /**
+    // TODO BA-FS26
+     */
+    [[nodiscard]] auto is_symbolic_link() const -> bool override;
+
   private:
     filesystem * m_filesystem;
     inode_data m_data{};
diff --git a/kernel/include/kernel/filesystem/inode.hpp b/kernel/include/kernel/filesystem/inode.hpp
index d071a6b..b34b921 100644
--- a/kernel/include/kernel/filesystem/inode.hpp
+++ b/kernel/include/kernel/filesystem/inode.hpp
@@ -57,6 +57,12 @@ namespace kernel::filesystem
     @return true if the inode is a device, false otherwise.
     */
     [[nodiscard]] virtual auto is_device() const -> bool;
+
+    /**
+    @brief Returns whether the inode is a symbolic link.
+    @return true if the inode is a symbolic link, false otherwise.
+    */
+    [[nodiscard]] virtual auto is_symbolic_link() const -> bool;
   };
 }  // namespace kernel::filesystem
 
diff --git a/kernel/src/filesystem/ext2/inode.cpp b/kernel/src/filesystem/ext2/inode.cpp
index 279c84f..5b6bcb3 100644
--- a/kernel/src/filesystem/ext2/inode.cpp
+++ b/kernel/src/filesystem/ext2/inode.cpp
@@ -72,4 +72,9 @@ namespace kernel::filesystem::ext2
   {
     return (m_data.mode & constants::mode_mask) == constants::mode_directory;
   }
+
+  auto inode::is_symbolic_link() const -> bool
+  {
+    return (m_data.mode & constants::mode_mask) == constants::mode_symbolic_link;
+  }
 }  // namespace kernel::filesystem::ext2
diff --git a/kernel/src/filesystem/ext2/inode.tests.cpp b/kernel/src/filesystem/ext2/inode.tests.cpp
index e68352f..f0f4aaf 100644
--- a/kernel/src/filesystem/ext2/inode.tests.cpp
+++ b/kernel/src/filesystem/ext2/inode.tests.cpp
@@ -33,6 +33,7 @@ SCENARIO("Ext2 inode initialization and properties", "[filesystem][ext2][inode]"
       REQUIRE(inode.is_regular());
       REQUIRE(!inode.is_directory());
       REQUIRE(!inode.is_device());
+      REQUIRE(!inode.is_symbolic_link());
     }
 
     THEN("the inode is initialized with directory mode in data and has the kind directory")
@@ -43,6 +44,29 @@ SCENARIO("Ext2 inode initialization and properties", "[filesystem][ext2][inode]"
       REQUIRE(!inode.is_regular());
       REQUIRE(inode.is_directory());
       REQUIRE(!inode.is_device());
+      REQUIRE(!inode.is_symbolic_link());
+    }
+
+    THEN("the inode is initialized with symbolic link mode in data and has the kind symbolic link")
+    {
+      data.mode = kernel::filesystem::ext2::constants::mode_symbolic_link;
+      auto inode = kernel::filesystem::ext2::inode(&fs, data);
+
+      REQUIRE(!inode.is_regular());
+      REQUIRE(!inode.is_directory());
+      REQUIRE(!inode.is_device());
+      REQUIRE(inode.is_symbolic_link());
+    }
+
+    THEN("the inode is initialized with zero mode in data and has no specific kind")
+    {
+      data.mode = 0;
+      auto inode = kernel::filesystem::ext2::inode(&fs, data);
+
+      REQUIRE(!inode.is_regular());
+      REQUIRE(!inode.is_directory());
+      REQUIRE(!inode.is_device());
+      REQUIRE(!inode.is_symbolic_link());
     }
   }
 
diff --git a/kernel/src/filesystem/inode.cpp b/kernel/src/filesystem/inode.cpp
index f2a7741..c188917 100644
--- a/kernel/src/filesystem/inode.cpp
+++ b/kernel/src/filesystem/inode.cpp
@@ -16,4 +16,9 @@ namespace kernel::filesystem
   {
     return false;
   }
+
+  auto inode::is_symbolic_link() const -> bool
+  {
+    return false;
+  }
 }  // namespace kernel::filesystem
\ No newline at end of file
-- 
cgit v1.2.3