Whamcloud - gitweb
e2fsck: require i_size == fscrypt_symlink_data.len + 2
[tools/e2fsprogs.git] / e2fsck / super.c
index ce824f4..5501c9e 100644 (file)
@@ -22,6 +22,7 @@
 
 #define MIN_CHECK 1
 #define MAX_CHECK 2
+#define LOG2_CHECK 4
 
 static void check_super_value(e2fsck_t ctx, const char *descr,
                              unsigned long value, int flags,
@@ -29,8 +30,26 @@ static void check_super_value(e2fsck_t ctx, const char *descr,
 {
        struct          problem_context pctx;
 
-       if (((flags & MIN_CHECK) && (value < min_val)) ||
-           ((flags & MAX_CHECK) && (value > max_val))) {
+       if ((flags & MIN_CHECK && value < min_val) ||
+           (flags & MAX_CHECK && value > max_val) ||
+           (flags & LOG2_CHECK && (value & (value - 1)) != 0)) {
+               clear_problem_context(&pctx);
+               pctx.num = value;
+               pctx.str = descr;
+               fix_problem(ctx, PR_0_MISC_CORRUPT_SUPER, &pctx);
+               ctx->flags |= E2F_FLAG_ABORT; /* never get here! */
+       }
+}
+
+static void check_super_value64(e2fsck_t ctx, const char *descr,
+                               __u64 value, int flags,
+                               __u64 min_val, __u64 max_val)
+{
+       struct          problem_context pctx;
+
+       if ((flags & MIN_CHECK && value < min_val) ||
+           (flags & MAX_CHECK && value > max_val) ||
+           (flags & LOG2_CHECK && (value & (value - 1)) != 0)) {
                clear_problem_context(&pctx);
                pctx.num = value;
                pctx.str = descr;
@@ -74,7 +93,7 @@ static int release_inode_block(ext2_filsys fs,
        pctx->blk = blk;
        pctx->blkcount = blockcnt;
 
-       if (HOLE_BLKADDR(blk))
+       if (blk == 0)
                return 0;
 
        if ((blk < fs->super->s_first_data_block) ||
@@ -185,7 +204,7 @@ static int release_inode_blocks(e2fsck_t ctx, ext2_ino_t ino,
                                      block_buf, release_inode_block, &pb);
        if (retval) {
                com_err("release_inode_blocks", retval,
-                       _("while calling ext2fs_block_iterate for inode %d"),
+                       _("while calling ext2fs_block_iterate for inode %u"),
                        ino);
                return 1;
        }
@@ -199,16 +218,16 @@ static int release_inode_blocks(e2fsck_t ctx, ext2_ino_t ino,
                ext2fs_iblk_sub_blocks(fs, inode, pb.truncated_blocks);
 
        if (ext2fs_file_acl_block(fs, inode)) {
-               retval = ext2fs_adjust_ea_refcount2(fs,
-                                       ext2fs_file_acl_block(fs, inode),
-                                       block_buf, -1, &count);
+               retval = ext2fs_adjust_ea_refcount3(fs,
+                               ext2fs_file_acl_block(fs, inode),
+                               block_buf, -1, &count, ino);
                if (retval == EXT2_ET_BAD_EA_BLOCK_NUM) {
                        retval = 0;
                        count = 1;
                }
                if (retval) {
                        com_err("release_inode_blocks", retval,
-               _("while calling ext2fs_adjust_ea_refcount2 for inode %d"),
+               _("while calling ext2fs_adjust_ea_refcount2 for inode %u"),
                                ino);
                        return 1;
                }
@@ -328,8 +347,7 @@ void check_resize_inode(e2fsck_t ctx)
         * If the resize inode feature isn't set, then
         * s_reserved_gdt_blocks must be zero.
         */
-       if (!(fs->super->s_feature_compat &
-             EXT2_FEATURE_COMPAT_RESIZE_INODE)) {
+       if (!ext2fs_has_feature_resize_inode(fs->super)) {
                if (fs->super->s_reserved_gdt_blocks) {
                        pctx.num = fs->super->s_reserved_gdt_blocks;
                        if (fix_problem(ctx, PR_0_NONZERO_RESERVED_GDT_BLOCKS,
@@ -344,8 +362,7 @@ void check_resize_inode(e2fsck_t ctx)
        pctx.ino = EXT2_RESIZE_INO;
        retval = ext2fs_read_inode(fs, EXT2_RESIZE_INO, &inode);
        if (retval) {
-               if (fs->super->s_feature_compat &
-                   EXT2_FEATURE_COMPAT_RESIZE_INODE)
+               if (ext2fs_has_feature_resize_inode(fs->super))
                        ctx->flags |= E2F_FLAG_RESIZE_INODE;
                return;
        }
@@ -354,8 +371,7 @@ void check_resize_inode(e2fsck_t ctx)
         * If the resize inode feature isn't set, check to make sure
         * the resize inode is cleared; then we're done.
         */
-       if (!(fs->super->s_feature_compat &
-             EXT2_FEATURE_COMPAT_RESIZE_INODE)) {
+       if (!ext2fs_has_feature_resize_inode(fs->super)) {
                for (i=0; i < EXT2_N_BLOCKS; i++) {
                        if (inode.i_block[i])
                                break;
@@ -419,7 +435,7 @@ void check_resize_inode(e2fsck_t ctx)
                for (j = 1; j < fs->group_desc_count; j++) {
                        if (!ext2fs_bg_has_super(fs, j))
                                continue;
-                       expect = pblk + (j * fs->super->s_blocks_per_group);
+                       expect = pblk + EXT2_GROUPS_TO_BLOCKS(fs->super, j);
                        if (ind_buf[ind_off] != expect)
                                goto resize_inode_invalid;
                        ind_off++;
@@ -442,7 +458,7 @@ static void e2fsck_fix_dirhash_hint(e2fsck_t ctx)
        char    c;
 
        if ((ctx->options & E2F_OPT_READONLY) ||
-           !(sb->s_feature_compat & EXT2_FEATURE_COMPAT_DIR_INDEX) ||
+           !ext2fs_has_feature_dir_index(sb) ||
            (sb->s_flags & (EXT2_FLAGS_SIGNED_HASH|EXT2_FLAGS_UNSIGNED_HASH)))
                return;
 
@@ -469,6 +485,7 @@ void check_super_block(e2fsck_t ctx)
        problem_t       problem;
        blk64_t blocks_per_group = fs->super->s_blocks_per_group;
        __u32   bpg_max, cpg_max;
+       __u64   blks_max;
        int     inodes_per_block;
        int     inode_size;
        int     accept_time_fudge;
@@ -498,6 +515,15 @@ void check_super_block(e2fsck_t ctx)
        ctx->invalid_inode_table_flag = (int *) e2fsck_allocate_memory(ctx,
                sizeof(int) * fs->group_desc_count, "invalid_inode_table");
 
+       blks_max = (1ULL << 32) * EXT2_MAX_BLOCKS_PER_GROUP(fs->super);
+       if (ext2fs_has_feature_64bit(fs->super)) {
+               if (blks_max > ((1ULL << 48) - 1))
+                       blks_max = (1ULL << 48) - 1;
+       } else {
+               if (blks_max > ((1ULL << 32) - 1))
+                       blks_max = (1ULL << 32) - 1;
+       }
+
        clear_problem_context(&pctx);
 
        /*
@@ -505,8 +531,8 @@ void check_super_block(e2fsck_t ctx)
         */
        check_super_value(ctx, "inodes_count", sb->s_inodes_count,
                          MIN_CHECK, 1, 0);
-       check_super_value(ctx, "blocks_count", ext2fs_blocks_count(sb),
-                         MIN_CHECK, 1, 0);
+       check_super_value64(ctx, "blocks_count", ext2fs_blocks_count(sb),
+                           MIN_CHECK | MAX_CHECK, 1, blks_max);
        check_super_value(ctx, "first_data_block", sb->s_first_data_block,
                          MAX_CHECK, 0, ext2fs_blocks_count(sb));
        check_super_value(ctx, "log_block_size", sb->s_log_block_size,
@@ -527,14 +553,17 @@ void check_super_block(e2fsck_t ctx)
                          MAX_CHECK, 0, ext2fs_blocks_count(sb) / 2);
        check_super_value(ctx, "reserved_gdt_blocks",
                          sb->s_reserved_gdt_blocks, MAX_CHECK, 0,
-                         fs->blocksize/4);
+                         fs->blocksize / sizeof(__u32));
+       check_super_value(ctx, "desc_size",
+                         sb->s_desc_size, MAX_CHECK | LOG2_CHECK, 0,
+                         EXT2_MAX_DESC_SIZE);
        if (sb->s_rev_level > EXT2_GOOD_OLD_REV)
                check_super_value(ctx, "first_ino", sb->s_first_ino,
                                  MIN_CHECK | MAX_CHECK,
                                  EXT2_GOOD_OLD_FIRST_INO, sb->s_inodes_count);
        inode_size = EXT2_INODE_SIZE(sb);
        check_super_value(ctx, "inode_size",
-                         inode_size, MIN_CHECK | MAX_CHECK,
+                         inode_size, MIN_CHECK | MAX_CHECK | LOG2_CHECK,
                          EXT2_GOOD_OLD_INODE_SIZE, fs->blocksize);
        if (sb->s_blocks_per_group != (sb->s_clusters_per_group *
                                       EXT2FS_CLUSTER_RATIO(fs))) {
@@ -544,13 +573,6 @@ void check_super_block(e2fsck_t ctx)
                ctx->flags |= E2F_FLAG_ABORT; /* never get here! */
                return;
        }
-       if (inode_size & (inode_size - 1)) {
-               pctx.num = inode_size;
-               pctx.str = "inode_size";
-               fix_problem(ctx, PR_0_MISC_CORRUPT_SUPER, &pctx);
-               ctx->flags |= E2F_FLAG_ABORT; /* never get here! */
-               return;
-       }
 
        if ((ctx->flags & E2F_FLAG_GOT_DEVSIZE) &&
            (ctx->num_blocks < ext2fs_blocks_count(sb))) {
@@ -572,7 +594,9 @@ void check_super_block(e2fsck_t ctx)
                return;
        }
 
-       should_be = sb->s_inodes_per_group * fs->group_desc_count;
+       should_be = (blk64_t)sb->s_inodes_per_group * fs->group_desc_count;
+       if (should_be > UINT_MAX)
+               should_be = UINT_MAX;
        if (sb->s_inodes_count != should_be) {
                pctx.ino = sb->s_inodes_count;
                pctx.ino2 = should_be;
@@ -581,31 +605,90 @@ void check_super_block(e2fsck_t ctx)
                        ext2fs_mark_super_dirty(fs);
                }
        }
+       if (EXT2_INODE_SIZE(sb) > EXT2_GOOD_OLD_INODE_SIZE) {
+               unsigned min =
+                       sizeof(((struct ext2_inode_large *) 0)->i_extra_isize) +
+                       sizeof(((struct ext2_inode_large *) 0)->i_checksum_hi);
+               unsigned max = EXT2_INODE_SIZE(sb) - EXT2_GOOD_OLD_INODE_SIZE;
+               pctx.num = sb->s_min_extra_isize;
+               if (sb->s_min_extra_isize &&
+                   (sb->s_min_extra_isize < min ||
+                    sb->s_min_extra_isize > max ||
+                    sb->s_min_extra_isize & 3) &&
+                   fix_problem(ctx, PR_0_BAD_MIN_EXTRA_ISIZE, &pctx)) {
+                       sb->s_min_extra_isize =
+                               (sizeof(struct ext2_inode_large) -
+                                EXT2_GOOD_OLD_INODE_SIZE);
+                       ext2fs_mark_super_dirty(fs);
+               }
+               pctx.num = sb->s_want_extra_isize;
+               if (sb->s_want_extra_isize &&
+                   (sb->s_want_extra_isize < min ||
+                    sb->s_want_extra_isize > max ||
+                    sb->s_want_extra_isize & 3) &&
+                   fix_problem(ctx, PR_0_BAD_WANT_EXTRA_ISIZE, &pctx)) {
+                       sb->s_want_extra_isize =
+                               (sizeof(struct ext2_inode_large) -
+                                EXT2_GOOD_OLD_INODE_SIZE);
+                       ext2fs_mark_super_dirty(fs);
+               }
+       }
+                   
+       /* Are metadata_csum and uninit_bg both set? */
+       if (ext2fs_has_feature_metadata_csum(fs->super) &&
+           ext2fs_has_feature_gdt_csum(fs->super) &&
+           fix_problem(ctx, PR_0_META_AND_GDT_CSUM_SET, &pctx)) {
+               ext2fs_clear_feature_gdt_csum(fs->super);
+               ext2fs_mark_super_dirty(fs);
+               for (i = 0; i < fs->group_desc_count; i++)
+                       ext2fs_group_desc_csum_set(fs, i);
+       }
+
+       /* We can't have ^metadata_csum,metadata_csum_seed */
+       if (!ext2fs_has_feature_metadata_csum(fs->super) &&
+           ext2fs_has_feature_csum_seed(fs->super) &&
+           fix_problem(ctx, PR_0_CSUM_SEED_WITHOUT_META_CSUM, &pctx)) {
+               ext2fs_clear_feature_csum_seed(fs->super);
+               fs->super->s_checksum_seed = 0;
+               ext2fs_mark_super_dirty(fs);
+       }
 
        /* Is 64bit set and extents unset? */
-       if (EXT2_HAS_INCOMPAT_FEATURE(fs->super,
-                                     EXT4_FEATURE_INCOMPAT_64BIT) &&
-           !EXT2_HAS_INCOMPAT_FEATURE(fs->super,
-                                      EXT3_FEATURE_INCOMPAT_EXTENTS) &&
+       if (ext2fs_has_feature_64bit(fs->super) &&
+           !ext2fs_has_feature_extents(fs->super) &&
            fix_problem(ctx, PR_0_64BIT_WITHOUT_EXTENTS, &pctx)) {
-               fs->super->s_feature_incompat |=
-                       EXT3_FEATURE_INCOMPAT_EXTENTS;
+               ext2fs_set_feature_extents(fs->super);
                ext2fs_mark_super_dirty(fs);
        }
 
+       /* Did user ask us to convert files to extents? */
+       if (ctx->options & E2F_OPT_CONVERT_BMAP) {
+               ext2fs_set_feature_extents(fs->super);
+               ext2fs_mark_super_dirty(fs);
+       }
+
+       if (ext2fs_has_feature_meta_bg(fs->super) &&
+           (fs->super->s_first_meta_bg > fs->desc_blocks)) {
+               pctx.group = fs->desc_blocks;
+               pctx.num = fs->super->s_first_meta_bg;
+               if (fix_problem(ctx, PR_0_FIRST_META_BG_TOO_BIG, &pctx)) {
+                       ext2fs_clear_feature_meta_bg(fs->super);
+                       fs->super->s_first_meta_bg = 0;
+                       ext2fs_mark_super_dirty(fs);
+               }
+       }
+
        /*
         * Verify the group descriptors....
         */
        first_block = sb->s_first_data_block;
        last_block = ext2fs_blocks_count(sb)-1;
 
-       csum_flag = EXT2_HAS_RO_COMPAT_FEATURE(fs->super,
-                                              EXT4_FEATURE_RO_COMPAT_GDT_CSUM);
+       csum_flag = ext2fs_has_group_desc_csum(fs);
        for (i = 0; i < fs->group_desc_count; i++) {
                pctx.group = i;
 
-               if (!EXT2_HAS_INCOMPAT_FEATURE(fs->super,
-                                              EXT4_FEATURE_INCOMPAT_FLEX_BG)) {
+               if (!ext2fs_has_feature_flex_bg(fs->super)) {
                        first_block = ext2fs_group_first_block2(fs, i);
                        last_block = ext2fs_group_last_block2(fs, i);
                }
@@ -724,10 +807,15 @@ void check_super_block(e2fsck_t ctx)
 #ifndef EXT2_SKIP_UUID
        /*
         * If the UUID field isn't assigned, assign it.
+        * Skip if checksums are enabled and the filesystem is mounted,
+        * if the id changes under the kernel remounting rw may fail.
         */
-       if (!(ctx->options & E2F_OPT_READONLY) && uuid_is_null(sb->s_uuid)) {
+       if (!(ctx->options & E2F_OPT_READONLY) && uuid_is_null(sb->s_uuid) &&
+           !ext2fs_has_feature_metadata_csum(ctx->fs->super) &&
+           (!csum_flag || !(ctx->mount_flags & EXT2_MF_MOUNTED))) {
                if (fix_problem(ctx, PR_0_ADD_UUID, &pctx)) {
                        uuid_generate(sb->s_uuid);
+                       ext2fs_init_csum_seed(fs);
                        fs->flags |= EXT2_FLAG_DIRTY;
                        fs->flags &= ~EXT2_FLAG_MASTER_SB_ONLY;
                }
@@ -757,11 +845,9 @@ void check_super_block(e2fsck_t ctx)
         */
        if (!(ctx->options & E2F_OPT_READONLY) &&
            fs->super->s_creator_os == EXT2_OS_HURD &&
-           (fs->super->s_feature_incompat &
-            EXT2_FEATURE_INCOMPAT_FILETYPE)) {
+           ext2fs_has_feature_filetype(fs->super)) {
                if (fix_problem(ctx, PR_0_HURD_CLEAR_FILETYPE, &pctx)) {
-                       fs->super->s_feature_incompat &=
-                               ~EXT2_FEATURE_INCOMPAT_FILETYPE;
+                       ext2fs_clear_feature_filetype(fs->super);
                        ext2fs_mark_super_dirty(fs);
                        fs->flags &= ~EXT2_FLAG_MASTER_SB_ONLY;
                }
@@ -851,6 +937,8 @@ void check_super_block(e2fsck_t ctx)
                }
        }
 
+       e2fsck_validate_quota_inodes(ctx);
+
        /*
         * Move the ext3 journal file, if necessary.
         */
@@ -956,6 +1044,7 @@ int check_backup_super_block(e2fsck_t ctx)
                    SUPER_INCOMPAT_DIFFERENT(s_feature_incompat) ||
                    SUPER_RO_COMPAT_DIFFERENT(s_feature_ro_compat) ||
                    SUPER_DIFFERENT(s_blocks_count) ||
+                   SUPER_DIFFERENT(s_blocks_count_hi) ||
                    SUPER_DIFFERENT(s_inodes_count) ||
                    memcmp(fs->super->s_uuid, backup_sb->s_uuid,
                           sizeof(fs->super->s_uuid)))