lkml.org 
[lkml]   [2018]   [Jan]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] f2fs: add sanity check for quota sysfile ino
On 01/29, Chao Yu wrote:
> Add missing sanity check for quota sysfile ino.

We don't need to limit the specific inode numbers for quota files, since
we may be able to set any inode numbers later. How about checkint the
numbers are allocated as quota?

>
> Signed-off-by: Chao Yu <yuchao0@huawei.com>
> ---
> fs/f2fs/super.c | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
>
> diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
> index 368f63d7bad2..6011071688ca 100644
> --- a/fs/f2fs/super.c
> +++ b/fs/f2fs/super.c
> @@ -2116,6 +2116,25 @@ static int sanity_check_raw_super(struct f2fs_sb_info *sbi,
> return 1;
> }
>
> + /* check quota sysfile ino info */
> + if (raw_super->feature & cpu_to_le32(F2FS_FEATURE_QUOTA_INO)) {
> + if (le32_to_cpu(raw_super->qf_ino[0]) != 4 ||
> + le32_to_cpu(raw_super->qf_ino[1]) != 5) {
> + f2fs_msg(sb, KERN_INFO,
> + "Invalid Quota Ino: user_ino(%u), grp_ino(%u)",
> + le32_to_cpu(raw_super->qf_ino[0]),
> + le32_to_cpu(raw_super->qf_ino[1]));
> + return 1;
> + }
> + if (raw_super->feature & cpu_to_le32(F2FS_FEATURE_PRJQUOTA) &&
> + le32_to_cpu(raw_super->qf_ino[2]) != 6) {
> + f2fs_msg(sb, KERN_INFO,
> + "Invalid Quota Ino: prj_ino(%u)",
> + le32_to_cpu(raw_super->qf_ino[2]));
> + return 1;
> + }
> + }
> +
> if (le32_to_cpu(raw_super->segment_count) > F2FS_MAX_SEGMENT) {
> f2fs_msg(sb, KERN_INFO,
> "Invalid segment count (%u)",
> --
> 2.15.0.55.gc2ece9dc4de6

\
 
 \ /
  Last update: 2018-01-31 02:36    [W:0.048 / U:0.764 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site