f2fs: avoid unneeded lookup when xattr name length is too long
authorChao Yu <chao2.yu@samsung.com>
Sat, 22 Mar 2014 06:59:45 +0000 (14:59 +0800)
committerJaegeuk Kim <jaegeuk.kim@samsung.com>
Tue, 1 Apr 2014 09:54:24 +0000 (18:54 +0900)
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>
fs/f2fs/xattr.c

index 0121e4595ccde97e24dd8b4992bc58558d9e8d53..503c2451131e5ba78b26fcce09870b0ca1a3d9bb 100644 (file)
@@ -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)