[f2fs-dev] [PATCH] f2fs: check upper bound of ino value in f2fs_nfs_get_inode

2014-03-12 Thread Chao Yu
Upper bound checking of ino should be added to f2fs_nfs_get_inode, so unneeded
process before do_read_inode in f2fs_iget could be avoided when ino is invalid.

Signed-off-by: Chao Yu 
---
 fs/f2fs/super.c |2 ++
 1 file changed, 2 insertions(+)

diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
index 6e4851c..3a51d7a 100644
--- a/fs/f2fs/super.c
+++ b/fs/f2fs/super.c
@@ -644,6 +644,8 @@ static struct inode *f2fs_nfs_get_inode(struct super_block 
*sb,
 
if (unlikely(ino < F2FS_ROOT_INO(sbi)))
return ERR_PTR(-ESTALE);
+   if (unlikely(ino >= NM_I(sbi)->max_nid))
+   return ERR_PTR(-ESTALE);
 
/*
 * f2fs_iget isn't quite right if the inode is currently unallocated!
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[f2fs-dev] [PATCH] f2fs: check upper bound of ino value in f2fs_nfs_get_inode

2014-03-12 Thread Chao Yu
Upper bound checking of ino should be added to f2fs_nfs_get_inode, so unneeded
process before do_read_inode in f2fs_iget could be avoided when ino is invalid.

Signed-off-by: Chao Yu chao2...@samsung.com
---
 fs/f2fs/super.c |2 ++
 1 file changed, 2 insertions(+)

diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
index 6e4851c..3a51d7a 100644
--- a/fs/f2fs/super.c
+++ b/fs/f2fs/super.c
@@ -644,6 +644,8 @@ static struct inode *f2fs_nfs_get_inode(struct super_block 
*sb,
 
if (unlikely(ino  F2FS_ROOT_INO(sbi)))
return ERR_PTR(-ESTALE);
+   if (unlikely(ino = NM_I(sbi)-max_nid))
+   return ERR_PTR(-ESTALE);
 
/*
 * f2fs_iget isn't quite right if the inode is currently unallocated!
-- 
1.7.9.5

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/