Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1440183 > unrolled thread
| Started by | "Tiezhu Yang" <kernelpatch@126.com> |
|---|---|
| First post | 2016-07-11 01:30 +0200 |
| Last post | 2016-07-13 03:10 +0200 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] f2fs: return proper error code "Tiezhu Yang" <kernelpatch@126.com> - 2016-07-11 01:30 +0200
Re: [f2fs-dev] [PATCH] f2fs: return proper error code Chao Yu <yuchao0@huawei.com> - 2016-07-12 03:50 +0200
Re:Re: [f2fs-dev] [PATCH] f2fs: return proper error code "Tiezhu Yang" <kernelpatch@126.com> - 2016-07-13 03:10 +0200
| From | "Tiezhu Yang" <kernelpatch@126.com> |
|---|---|
| Date | 2016-07-11 01:30 +0200 |
| Subject | [PATCH] f2fs: return proper error code |
| Message-ID | <rTwkF-7sR-1@gated-at.bofh.it> |
When the length of file name is more than F2FS_NAME_LEN, it should return -ENAMETOOLONG. Signed-off-by: Tiezhu Yang <kernelpatch@126.com> --- fs/f2fs/xattr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/f2fs/xattr.c b/fs/f2fs/xattr.c index 28a5023..b225062 100644 --- a/fs/f2fs/xattr.c +++ b/fs/f2fs/xattr.c @@ -364,7 +364,7 @@ int f2fs_getxattr(struct inode *inode, int index, const char *name, len = strlen(name); if (len > F2FS_NAME_LEN) - return -ERANGE; + return -ENAMETOOLONG; base_addr = read_all_xattrs(inode, ipage); if (!base_addr) @@ -458,7 +458,7 @@ static int __f2fs_setxattr(struct inode *inode, int index, len = strlen(name); if (len > F2FS_NAME_LEN) - return -ERANGE; + return -ENAMETOOLONG; if (size > MAX_VALUE_LEN(inode)) return -E2BIG; -- 1.8.3.1
[toc] | [next] | [standalone]
| From | Chao Yu <yuchao0@huawei.com> |
|---|---|
| Date | 2016-07-12 03:50 +0200 |
| Subject | Re: [f2fs-dev] [PATCH] f2fs: return proper error code |
| Message-ID | <rTUZH-6B4-1@gated-at.bofh.it> |
| In reply to | #1440183 |
On 2016/7/11 7:20, Tiezhu Yang wrote: > When the length of file name is more than F2FS_NAME_LEN, Seem @name indicates a xattr/key name, not a file name. Thanks, > it should return -ENAMETOOLONG. > > Signed-off-by: Tiezhu Yang <kernelpatch@126.com> > --- > fs/f2fs/xattr.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/fs/f2fs/xattr.c b/fs/f2fs/xattr.c > index 28a5023..b225062 100644 > --- a/fs/f2fs/xattr.c > +++ b/fs/f2fs/xattr.c > @@ -364,7 +364,7 @@ int f2fs_getxattr(struct inode *inode, int index, const char *name, > > len = strlen(name); > if (len > F2FS_NAME_LEN) > - return -ERANGE; > + return -ENAMETOOLONG; > > base_addr = read_all_xattrs(inode, ipage); > if (!base_addr) > @@ -458,7 +458,7 @@ static int __f2fs_setxattr(struct inode *inode, int index, > len = strlen(name); > > if (len > F2FS_NAME_LEN) > - return -ERANGE; > + return -ENAMETOOLONG; > > if (size > MAX_VALUE_LEN(inode)) > return -E2BIG; >
[toc] | [prev] | [next] | [standalone]
| From | "Tiezhu Yang" <kernelpatch@126.com> |
|---|---|
| Date | 2016-07-13 03:10 +0200 |
| Subject | Re:Re: [f2fs-dev] [PATCH] f2fs: return proper error code |
| Message-ID | <rUgQx-4nw-1@gated-at.bofh.it> |
| In reply to | #1441015 |
At 2016-07-12 09:45:43, "Chao Yu" <yuchao0@huawei.com> wrote: >On 2016/7/11 7:20, Tiezhu Yang wrote: >> When the length of file name is more than F2FS_NAME_LEN, > >Seem @name indicates a xattr/key name, not a file name. Yes, you are right. Sorry for the noise. Thanks,
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web