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

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.12.0
• Severity: 3
• Assignee: John Hammond
• Reporter: John Hammond

Description

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().

Issue Links

• is related to: LU-9679 Prepare lustre for adoption into the linux kernel