Whamcloud - gitweb
LU-16067 misc: cleanup compiler warnings
[tools/e2fsprogs.git] / e2fsck / journal.c
index 922c252..01c3bed 100644 (file)
@@ -289,7 +289,7 @@ static int ext4_fc_replay_scan(journal_t *j, struct buffer_head *bh,
        struct ext4_fc_tail *tail;
        __u8 *start, *end;
        struct ext4_fc_head *head;
-       struct ext2fs_extent ext2fs_ex;
+       struct ext2fs_extent ext2fs_ex = {0};
 
        state = &ctx->fc_replay_state;
 
@@ -325,6 +325,7 @@ static int ext4_fc_replay_scan(journal_t *j, struct buffer_head *bh,
                                ret = JBD2_FC_REPLAY_STOP;
                        else
                                ret = JBD2_FC_REPLAY_CONTINUE;
+                       /* fallthrough */
                case EXT4_FC_TAG_DEL_RANGE:
                case EXT4_FC_TAG_LINK:
                case EXT4_FC_TAG_UNLINK:
@@ -444,7 +445,6 @@ static int ex_len_compar(const void *arg1, const void *arg2)
 
 static void ex_sort_and_merge(e2fsck_t ctx, struct extent_list *list)
 {
-       blk64_t ex_end;
        int i, j;
 
        if (list->count < 2)
@@ -492,7 +492,7 @@ static int ext4_modify_extent_list(e2fsck_t ctx, struct extent_list *list,
 {
        int ret;
        int i, offset;
-       struct ext2fs_extent add_ex = *ex, add_ex2;
+       struct ext2fs_extent add_ex = *ex;
 
        /* First let's create a hole from ex->e_lblk of length ex->e_len */
        for (i = 0; i < list->count; i++) {
@@ -612,42 +612,40 @@ struct dentry_info_args {
 };
 
 static inline void tl_to_darg(struct dentry_info_args *darg,
-                               struct  ext4_fc_tl *tl)
+                             struct ext4_fc_tl *tl)
 {
        struct ext4_fc_dentry_info *fcd;
-       int tag = le16_to_cpu(tl->fc_tag);
 
        fcd = (struct ext4_fc_dentry_info *)ext4_fc_tag_val(tl);
 
        darg->parent_ino = le32_to_cpu(fcd->fc_parent_ino);
        darg->ino = le32_to_cpu(fcd->fc_ino);
-       darg->dname = fcd->fc_dname;
+       darg->dname = (char *) fcd->fc_dname;
        darg->dname_len = ext4_fc_tag_len(tl) -
                        sizeof(struct ext4_fc_dentry_info);
        darg->dname = malloc(darg->dname_len + 1);
        memcpy(darg->dname, fcd->fc_dname, darg->dname_len);
        darg->dname[darg->dname_len] = 0;
        jbd_debug(1, "%s: %s, ino %d, parent %d\n",
-               tag == EXT4_FC_TAG_CREAT ? "create" :
-               (tag == EXT4_FC_TAG_LINK ? "link" :
-               (tag == EXT4_FC_TAG_UNLINK ? "unlink" : "error")),
-               darg->dname, darg->ino, darg->parent_ino);
+                 le16_to_cpu(tl->fc_tag) == EXT4_FC_TAG_CREAT ? "create" :
+                 (le16_to_cpu(tl->fc_tag) == EXT4_FC_TAG_LINK ? "link" :
+                  (le16_to_cpu(tl->fc_tag) == EXT4_FC_TAG_UNLINK ? "unlink" :
+                   "error")),
+                 darg->dname, darg->ino, darg->parent_ino);
 }
 
 static int ext4_fc_handle_unlink(e2fsck_t ctx, struct ext4_fc_tl *tl)
 {
-       struct ext2_inode inode;
        struct dentry_info_args darg;
-       ext2_filsys fs = ctx->fs;
        int ret;
 
        tl_to_darg(&darg, tl);
        ext4_fc_flush_extents(ctx, darg.ino);
-       ret = errcode_to_errno(
-                      ext2fs_unlink(ctx->fs, darg.parent_ino,
-                                    darg.dname, darg.ino, 0));
+       ret = errcode_to_errno(ext2fs_unlink(ctx->fs, darg.parent_ino,
+                                            darg.dname, darg.ino, 0));
        /* It's okay if the above call fails */
        free(darg.dname);
+
        return ret;
 }
 
@@ -706,19 +704,19 @@ out:
 static void ext4_fc_replay_fixup_iblocks(struct ext2_inode_large *ondisk_inode,
        struct ext2_inode_large *fc_inode)
 {
-       if (le32_to_cpu(ondisk_inode->i_flags) & EXT4_EXTENTS_FL) {
+       if (ondisk_inode->i_flags & EXT4_EXTENTS_FL) {
                struct ext3_extent_header *eh;
 
                eh = (struct ext3_extent_header *)(&ondisk_inode->i_block[0]);
-               if (eh->eh_magic != EXT3_EXT_MAGIC) {
+               if (le16_to_cpu(eh->eh_magic) != EXT3_EXT_MAGIC) {
                        memset(eh, 0, sizeof(*eh));
-                       eh->eh_magic = EXT3_EXT_MAGIC;
+                       eh->eh_magic = cpu_to_le16(EXT3_EXT_MAGIC);
                        eh->eh_max = cpu_to_le16(
                                (sizeof(ondisk_inode->i_block) -
                                        sizeof(struct ext3_extent_header)) /
-                                       sizeof(struct ext3_extent));
+                               sizeof(struct ext3_extent));
                }
-       } else if (le32_to_cpu(ondisk_inode->i_flags) & EXT4_INLINE_DATA_FL) {
+       } else if (ondisk_inode->i_flags & EXT4_INLINE_DATA_FL) {
                memcpy(ondisk_inode->i_block, fc_inode->i_block,
                        sizeof(fc_inode->i_block));
        }
@@ -726,36 +724,42 @@ static void ext4_fc_replay_fixup_iblocks(struct ext2_inode_large *ondisk_inode,
 
 static int ext4_fc_handle_inode(e2fsck_t ctx, struct ext4_fc_tl *tl)
 {
-       struct e2fsck_fc_replay_state *state = &ctx->fc_replay_state;
        int ino, inode_len = EXT2_GOOD_OLD_INODE_SIZE;
-       struct ext2_inode_large *inode = NULL;
-       struct ext4_fc_inode *fc_inode;
+       struct ext2_inode_large *inode = NULL, *fc_inode = NULL;
+       struct ext4_fc_inode *fc_inode_val;
        errcode_t err;
        blk64_t blks;
 
-       fc_inode = (struct ext4_fc_inode *)ext4_fc_tag_val(tl);
-       ino = le32_to_cpu(fc_inode->fc_ino);
+       fc_inode_val = (struct ext4_fc_inode *)ext4_fc_tag_val(tl);
+       ino = le32_to_cpu(fc_inode_val->fc_ino);
 
        if (EXT2_INODE_SIZE(ctx->fs->super) > EXT2_GOOD_OLD_INODE_SIZE)
                inode_len += ext2fs_le16_to_cpu(
-                       ((struct ext2_inode_large *)fc_inode->fc_raw_inode)
+                       ((struct ext2_inode_large *)fc_inode_val->fc_raw_inode)
                                ->i_extra_isize);
        err = ext2fs_get_mem(inode_len, &inode);
        if (err)
-               return errcode_to_errno(err);
+               goto out;
+       err = ext2fs_get_mem(inode_len, &fc_inode);
+       if (err)
+               goto out;
        ext4_fc_flush_extents(ctx, ino);
 
        err = ext2fs_read_inode_full(ctx->fs, ino, (struct ext2_inode *)inode,
                                        inode_len);
        if (err)
                goto out;
-       memcpy(inode, fc_inode->fc_raw_inode,
-               offsetof(struct ext2_inode_large, i_block));
-       memcpy(&inode->i_generation,
-               &((struct ext2_inode_large *)(fc_inode->fc_raw_inode))->i_generation,
+#ifdef WORDS_BIGENDIAN
+       ext2fs_swap_inode_full(ctx->fs, fc_inode,
+                              (struct ext2_inode_large *)fc_inode_val->fc_raw_inode,
+                              0, sizeof(*inode));
+#else
+       memcpy(fc_inode, fc_inode_val->fc_raw_inode, inode_len);
+#endif
+       memcpy(inode, fc_inode, offsetof(struct ext2_inode_large, i_block));
+       memcpy(&inode->i_generation, &fc_inode->i_generation,
                inode_len - offsetof(struct ext2_inode_large, i_generation));
-       ext4_fc_replay_fixup_iblocks(inode,
-               (struct ext2_inode_large *)fc_inode->fc_raw_inode);
+       ext4_fc_replay_fixup_iblocks(inode, fc_inode);
        err = ext2fs_count_blocks(ctx->fs, ino, EXT2_INODE(inode), &blks);
        if (err)
                goto out;
@@ -774,6 +778,7 @@ static int ext4_fc_handle_inode(e2fsck_t ctx, struct ext4_fc_tl *tl)
 
 out:
        ext2fs_free_mem(&inode);
+       ext2fs_free_mem(&fc_inode);
        return errcode_to_errno(err);
 }
 
@@ -784,7 +789,6 @@ static int ext4_fc_handle_add_extent(e2fsck_t ctx, struct ext4_fc_tl *tl)
 {
        struct ext2fs_extent extent;
        struct ext4_fc_add_range *add_range;
-       struct ext4_fc_del_range *del_range;
        int ret = 0, ino;
 
        add_range = (struct ext4_fc_add_range *)ext4_fc_tag_val(tl);
@@ -819,7 +823,7 @@ static int ext4_fc_handle_del_range(e2fsck_t ctx, struct ext4_fc_tl *tl)
 
        memset(&extent, 0, sizeof(extent));
        extent.e_lblk = ext2fs_le32_to_cpu(del_range->fc_lblk);
-       extent.e_len = ext2fs_le16_to_cpu(del_range->fc_len);
+       extent.e_len = ext2fs_le32_to_cpu(del_range->fc_len);
        ret = ext4_fc_read_extents(ctx, ino);
        if (ret)
                return ret;