ubifs: Massage assert in ubifs_xattr_set() wrt. fscrypto
authorRichard Weinberger <richard@nod.at>
Thu, 29 Sep 2016 18:11:03 +0000 (20:11 +0200)
committerRichard Weinberger <richard@nod.at>
Mon, 12 Dec 2016 22:07:38 +0000 (23:07 +0100)
When we're creating a new inode in UBIFS the inode is not
yet exposed and fscrypto calls ubifs_xattr_set() without
holding the inode mutex. This is okay but ubifs_xattr_set()
has to know about this.

Signed-off-by: Richard Weinberger <richard@nod.at>
fs/ubifs/xattr.c

index 77ffc9788f45141e51d494b5199466ca08c0684c..da59ea3b50c443fb38f3605f8553b075268e5da7 100644 (file)
@@ -289,7 +289,13 @@ int ubifs_xattr_set(struct inode *host, const char *name, const void *value,
        union ubifs_key key;
        int err;
 
-       ubifs_assert(inode_is_locked(host));
+       /*
+        * Creating an encryption context is done unlocked since we
+        * operate on a new inode which is not visible to other users
+        * at this point.
+        */
+       if (strcmp(name, UBIFS_XATTR_NAME_ENCRYPTION_CONTEXT) != 0)
+               ubifs_assert(inode_is_locked(host));
 
        if (size > UBIFS_MAX_INO_DATA)
                return -ERANGE;