diff options
| author | Andreas Gruenbacher <agruenba@redhat.com> | 2016-09-29 17:48:30 +0200 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2016-10-06 22:17:38 -0400 |
| commit | 5d18cbf16cfb0ad65368fe2ead19237305f4b822 (patch) | |
| tree | c46b8e5375ccc764e78e054f066f68b413026468 /fs/xattr.c | |
| parent | 08895a8b6b06ed2323cd97a36ee40a116b3db8ed (diff) | |
| download | lwn-5d18cbf16cfb0ad65368fe2ead19237305f4b822.tar.gz lwn-5d18cbf16cfb0ad65368fe2ead19237305f4b822.zip | |
xattr: Remove unnecessary NULL attribute name check
When NULL is passed to one of the xattr system calls as the attribute
name, copying that name from user space already fails with -EFAULT;
xattr_resolve_name is never called with a NULL attribute name.
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/xattr.c')
| -rw-r--r-- | fs/xattr.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/fs/xattr.c b/fs/xattr.c index c243905835ab..1f72c9217398 100644 --- a/fs/xattr.c +++ b/fs/xattr.c @@ -675,9 +675,6 @@ xattr_resolve_name(const struct xattr_handler **handlers, const char **name) { const struct xattr_handler *handler; - if (!*name) - return ERR_PTR(-EINVAL); - for_each_xattr_handler(handlers, handler) { const char *n; |