require Inode instead of Directory for list_dir

tsatke · Oct 27, 2023 · 5eba56d · 5eba56d
1 parent 2fc94a1
commit 5eba56d
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/ext2/src/dir.rs b/ext2/src/dir.rs
@@ -2,7 +2,7 @@ use crate::error::Error;
 use crate::superblock::RequiredFeatures;
 use crate::{
     bytefield, bytefield_field_read, bytefield_field_write, check_is_implemented, Directory,
-    Ext2Fs, Inode, InodeAddress,
+    Ext2Fs, Inode, InodeAddress, Type,
 };
 use alloc::vec;
 use alloc::vec::Vec;
@@ -15,7 +15,9 @@ where
     T: BlockDevice,
 {
     // TODO: make this return some Result<impl Iterator<Item=DirEntry>, Error>
-    pub fn list_dir(&self, dir: &Directory) -> Result<Vec<DirEntry>, Error> {
+    pub fn list_dir(&self, dir: &Inode) -> Result<Vec<DirEntry>, Error> {
+        debug_assert_eq!(dir.typ(), Type::Directory);
+
         let mut entries = Vec::new();
         let block_size = self.superblock.block_size() as usize;
         let dir_entries_have_type = self