Changes in / [ff4f073:5c96b2a] in mainline

Location:

uspace

Files:

• app/ext2info/ext2info.c (modified) (2 diffs)
• lib/ext2/libext2_directory.c (modified) (7 diffs)
• lib/ext2/libext2_filesystem.c (modified) (2 diffs)
• lib/ext2/libext2_superblock.c (modified) (1 diff)
• srv/fs/ext2fs/ext2fs_ops.c (modified) (3 diffs)

uspace/app/ext2info/ext2info.c ¶

@@ -586 +586 @@
         ext2_directory_iterator_t it;
         size_t name_size;
-        uint32_t inode;
         
         printf("  Directory contents:\n");
@@ -601 +600 @@
                 printf("    ");
                 print_data(&it.current->name, name_size);
-                
-                inode = ext2_directory_entry_ll_get_inode(it.current);
-                printf(" --> %u\n", inode);
+                printf(" --> %u\n", it.current->inode);
                 
                 rc = ext2_directory_iterator_next(&it);

uspace/lib/ext2/libext2_directory.c ¶

@@ -40 +40 @@
 #include <assert.h>
 
-static int ext2_directory_iterator_set(ext2_directory_iterator_t *it,
-    uint32_t block_size);
-
 /**
  * Get inode number for the directory entry
@@ -93 +90 @@
         int rc;
         uint32_t block_id;
-        uint32_t block_size;
-        
         it->inode_ref = inode_ref;
         it->fs = fs;
@@ -110 +105 @@
         }
         
-        block_size = ext2_superblock_get_block_size(fs->superblock);
-        
-        it->current_offset = 0; 
-        return ext2_directory_iterator_set(it, block_size);
+        it->current = it->current_block->data;
+        it->current_offset = 0;
+        
+        return EOK;
 }
 
@@ -131 +126 @@
         uint32_t next_block_phys_idx;
         uint32_t block_size;
+        uint32_t offset_in_block;
         
         assert(it->current != NULL);
@@ -179 +175 @@
         }
         
-        it->current_offset += skip;
-        return ext2_directory_iterator_set(it, block_size);
-}
-
-static int ext2_directory_iterator_set(ext2_directory_iterator_t *it,
-    uint32_t block_size)
-{
-        uint32_t offset_in_block = it->current_offset % block_size;
-        
-        it->current = NULL;
+        offset_in_block = (it->current_offset + skip) % block_size;
         
         /* Ensure proper alignment */
         if ((offset_in_block % 4) != 0) {
+                it->current = NULL;
                 return EIO;
         }
@@ -197 +185 @@
         /* Ensure that the core of the entry does not overflow the block */
         if (offset_in_block > block_size - 8) {
-                return EIO;
-        }
-        
-        ext2_directory_entry_ll_t *entry = it->current_block->data + offset_in_block;
+                it->current = NULL;
+                return EIO;
+        }
+                
+        it->current = it->current_block->data + offset_in_block;
+        it->current_offset += skip;
         
         /* Ensure that the whole entry does not overflow the block */
-        uint16_t length = ext2_directory_entry_ll_get_entry_length(entry);
-        if (offset_in_block + length > block_size) {
+        skip = ext2_directory_entry_ll_get_entry_length(it->current);
+        if (offset_in_block + skip > block_size) {
+                it->current = NULL;
                 return EIO;
         }
@@ -210 +201 @@
         /* Ensure the name length is not too large */
         if (ext2_directory_entry_ll_get_name_length(it->fs->superblock, 
-            entry) > length-8) {
-                return EIO;
-        }
-        
-        it->current = entry;
+            it->current) > skip-8) {
+                it->current = NULL;
+                return EIO;
+        }
+        
         return EOK;
 }

uspace/lib/ext2/libext2_filesystem.c ¶

@@ -42 +42 @@
 #include <malloc.h>
 #include <assert.h>
-#include <byteorder.h>
 
 /**
@@ -370 +369 @@
                 
                 assert(offset_in_block < block_ids_per_block);
-                current_block = uint32_t_le2host(((uint32_t*)block->data)[offset_in_block]);
+                current_block = ((uint32_t*)block->data)[offset_in_block];
                 
                 rc = block_put(block);

uspace/lib/ext2/libext2_superblock.c ¶

@@ -184 +184 @@
                 return EXT2_REV0_INODE_SIZE;
         }
-        return uint16_t_le2host(sb->inode_size);
+        return uint32_t_le2host(sb->inode_size);
 }
 

uspace/srv/fs/ext2fs/ext2fs_ops.c ¶

@@ -232 +232 @@
         size_t component_size;
         bool found = false;
-        uint32_t inode;
         
         fs = eparent->instance->filesystem;
@@ -255 +254 @@
         
         while (it.current != NULL) {
-                inode = ext2_directory_entry_ll_get_inode(it.current);
-                
                 /* ignore empty directory entries */
-                if (inode != 0) {
+                if (it.current->inode != 0) {
                         name_size = ext2_directory_entry_ll_get_name_length(fs->superblock,
                                 it.current);
@@ -265 +262 @@
                                     name_size) == 0) {
                                 rc = ext2fs_node_get_core(rfn, eparent->instance,
-                                        inode);
+                                        it.current->inode);
                                 if (rc != EOK) {
                                         ext2_directory_iterator_fini(&it);