[LU-10788] __vfs_setxattr() should pass flags through

Related — Mon, 9 Apr 2018 20:55:10 +0000

In the compat definition of __vfs_setxattr() we should pass the value of flags that we received rather than XATTR_CREATE:

#ifdef HAVE_XATTR_HANDLER_FLAGS
static inline int
__vfs_setxattr(struct dentry *dentry, struct inode *inode, const char *name,
               const void *value, size_t size, int flags)
{
        const struct xattr_handler *handler;
        int rc;

        handler = get_xattr_type(name);
        if (!handler)
                return -ENXIO;

#if defined(HAVE_XATTR_HANDLER_INODE_PARAM)
        rc = handler->set(handler, dentry, inode, name, value, size,
                          XATTR_CREATE);
#elif defined(HAVE_XATTR_HANDLER_SIMPLIFIED)
        rc = handler->set(handler, dentry, name, value, size, XATTR_CREATE);
#else
        rc = handler->set(dentry, name, value, size, XATTR_CREATE,
                          handler->flags);
#endif /* !HAVE_XATTR_HANDLER_INODE_PARAM */
        return rc;
}

This is minor now since we only pass XATTR_CREATE to __vfs_setxattr().

Whamcloud Community JIRA

[LU-10788] __vfs_setxattr() should pass flags through