linux-ext4 - Re: [PATCH v2 01/15] ext4: add a function which extends a group without checking parameters

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAGBYx2aT6wGc1T+ijycua3+tbJo4_pxE2tt2q4c+yDogqNnLhQ@mail.gmail.com>
Date:	Sun, 30 Oct 2011 21:48:43 +0800
From:	Yongqiang Yang <xiaoqiangnk@...il.com>
To:	tytso@....edu
Cc:	adilger@...ger.ca, linux-ext4@...r.kernel.org,
	Yongqiang Yang <xiaoqiangnk@...il.com>
Subject: Re: [PATCH v2 01/15] ext4: add a function which extends a group
 without checking parameters

ping?

On Tue, Aug 30, 2011 at 4:10 PM, Yongqiang Yang <xiaoqiangnk@...il.com> wrote:
> This patch added a function named __ext4_group_extend() whose code
> is copied from ext4_group_extend().  __ext4_group_extend() assumes
> the parameter is valid and has been checked by caller.
>
> __ext4_group_extend() will be used by new resize implementation. It
> can also be used by ext4_group_extend(), but this patch series does
> not do this.
>
> Signed-off-by: Yongqiang Yang <xiaoqiangnk@...il.com>
> ---
>  fs/ext4/resize.c |   53 +++++++++++++++++++++++++++++++++++++++++++++++++++++
>  1 files changed, 53 insertions(+), 0 deletions(-)
>
> diff --git a/fs/ext4/resize.c b/fs/ext4/resize.c
> index 707d3f1..5126aac 100644
> --- a/fs/ext4/resize.c
> +++ b/fs/ext4/resize.c
> @@ -969,6 +969,59 @@ exit_put:
>  } /* ext4_group_add */
>
>  /*
> + * extend a group without checking assuming that checking has been done.
> + */
> +static int __ext4_group_extend(struct super_block *sb,
> +                              ext4_fsblk_t o_blocks_count, ext4_grpblk_t add)
> +{
> +       struct ext4_super_block *es = EXT4_SB(sb)->s_es;
> +       handle_t *handle;
> +       int err = 0, err2;
> +
> +       /* We will update the superblock, one block bitmap, and
> +        * one group descriptor via ext4_group_add_blocks().
> +        */
> +       handle = ext4_journal_start_sb(sb, 3);
> +       if (IS_ERR(handle)) {
> +               err = PTR_ERR(handle);
> +               ext4_warning(sb, "error %d on journal start", err);
> +               goto out;
> +       }
> +
> +       err = ext4_journal_get_write_access(handle, EXT4_SB(sb)->s_sbh);
> +       if (err) {
> +               ext4_warning(sb, "error %d on journal write access", err);
> +               ext4_journal_stop(handle);
> +               goto out;
> +       }
> +
> +       ext4_blocks_count_set(es, o_blocks_count + add);
> +       ext4_debug("freeing blocks %llu through %llu\n", o_blocks_count,
> +                  o_blocks_count + add);
> +       /* We add the blocks to the bitmap and set the group need init bit */
> +       err = ext4_group_add_blocks(handle, sb, o_blocks_count, add);
> +       if (err)
> +               goto exit_journal;
> +       ext4_handle_dirty_super(handle, sb);
> +       ext4_debug("freed blocks %llu through %llu\n", o_blocks_count,
> +                  o_blocks_count + add);
> +exit_journal:
> +       err2 = ext4_journal_stop(handle);
> +       if (err2 && !err)
> +               err = err2;
> +
> +       if (!err) {
> +               if (test_opt(sb, DEBUG))
> +                       printk(KERN_DEBUG "EXT4-fs: extended group to %llu "
> +                              "blocks\n", ext4_blocks_count(es));
> +               update_backups(sb, EXT4_SB(sb)->s_sbh->b_blocknr, (char *)es,
> +                              sizeof(struct ext4_super_block));
> +       }
> +out:
> +       return err;
> +}
> +
> +/*
>  * Extend the filesystem to the new number of blocks specified.  This entry
>  * point is only used to extend the current filesystem to the end of the last
>  * existing group.  It can be accessed via ioctl, or by "remount,resize=<size>"
> --
> 1.7.5.1
>
>



-- 
Best Wishes
Yongqiang Yang
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html