Changeset fde8a27 in mainline for uspace/srv/fs/minixfs/mfs_read.c

Timestamp:

2011-03-28T19:17:31Z (13 years ago)

Author:

Maurizio Lombardi <m.lombardi85@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

1f1cc9d

Parents:

ef76d72

Message:

Fix bug when reading inodes that causes on-disk structures corruption if the
endianness is not native.

File:

• uspace/srv/fs/minixfs/mfs_read.c (modified) (4 diffs)

uspace/srv/fs/minixfs/mfs_read.c

@@ -72 +72 @@
 static int read_map_ondisk(uint32_t *b, const struct mfs_node *mnode, int rblock)
 {
-        block_t *bi1, *bi2;
         int r, nr_direct;
         int ptrs_per_block;
+        uint32_t *bi1 = NULL;
+        uint32_t *bi2 = NULL;
 
         assert(mnode);
@@ -110 +111 @@
                 }
 
-                r = read_ind_block(bi2, mnode->instance, ino_i->i_izone[0]);
+                bi1 = (uint32_t *) malloc(sbi->block_size);
+                r = read_ind_block(bi1, mnode->instance, ino_i->i_izone[0],
+                                        fs_version);
 
-                if (r != EOK)
-                        goto out;
+                if (fs_version == MFS_VERSION_V1)
+                        *b = ((uint16_t *) bi1)[rblock];
+                else
+                        *b = bi1[rblock];
 
-                *b = fs_version == MFS_VERSION_V1 ? 
-                                ((uint16_t *) bi1->data)[rblock] :
-                                ((uint32_t *) bi1->data)[rblock];
-                goto out_block;
+                goto out;
         }
 
@@ -130 +132 @@
                 goto out;
         }
-        r = read_ind_block(bi1, mnode->instance, ino_i->i_izone[1]);
+
+        bi1 = (uint32_t *) malloc(sbi->block_size);
+        bi2 = (uint32_t *) malloc(sbi->block_size);
+
+        r = read_ind_block(bi1, mnode->instance, ino_i->i_izone[1],
+                                fs_version);
 
         if (r != EOK)
@@ -144 +151 @@
         if (fs_version == MFS_VERSION_V1) {
                 r = read_ind_block(bi2, mnode->instance,
-                        ((uint16_t *) bi1->data)[di_block]);
+                        ((uint16_t *) bi1)[di_block], fs_version);
 
                 if (r != EOK)
-                        goto out_block;
+                        goto out;
 
-                *b = ((uint16_t *) bi2->data)[rblock % ptrs_per_block];
+                *b = ((uint16_t *) bi2)[rblock % ptrs_per_block];
         } else {
                 r = read_ind_block(bi2, mnode->instance,
-                        ((uint32_t *) bi1->data)[di_block]);
+                        ((uint32_t *) bi1)[di_block], fs_version);
 
                 if (r != EOK)
-                        goto out_block;
+                        goto out;
 
-                *b = ((uint32_t *) bi2->data)[rblock % ptrs_per_block];
+                *b = bi2[rblock % ptrs_per_block];
         }
         r = EOK;
-        block_put(bi2);
 
-out_block:
-        block_put(bi1);
 out:
+        if (bi1)
+                free(bi1);
+        if (bi2)
+                free(bi2);
         return r;
 }