diff options
| author | Chao Yu <chao2.yu@samsung.com> | 2014-03-22 07:59:45 +0100 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2014-04-01 11:54:24 +0200 |
| commit | 6e452d69d421e10e99446c6de16a19604149c40f (patch) | |
| tree | 78236f48ae6d645bb9a918181e06ce8f53ae8aa8 /fs | |
| parent | f2fs: avoid unnecessary bio submit when wait page writeback (diff) | |
| download | linux-6e452d69d421e10e99446c6de16a19604149c40f.tar.xz linux-6e452d69d421e10e99446c6de16a19604149c40f.zip | |
f2fs: avoid unneeded lookup when xattr name length is too long
In f2fs_setxattr we have limit this attribute name length, so we should also
check it in f2fs_getxattr to avoid useless lookup caused by invalid name length.
Signed-off-by: Chao Yu <chao2.yu@samsung.com>
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/f2fs/xattr.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/f2fs/xattr.c b/fs/f2fs/xattr.c index 0121e4595ccd..503c2451131e 100644 --- a/fs/f2fs/xattr.c +++ b/fs/f2fs/xattr.c @@ -407,6 +407,8 @@ int f2fs_getxattr(struct inode *inode, int name_index, const char *name, if (name == NULL) return -EINVAL; name_len = strlen(name); + if (name_len > F2FS_NAME_LEN) + return -ERANGE; base_addr = read_all_xattrs(inode, NULL); if (!base_addr) |