There is a missing unlock if ext4_journal_start() fails. Fixes: 374431bae296 ('ext4 crypto: add ioctls to allow backup of encryption metadata') Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> diff --git a/fs/ext4/crypto_key.c b/fs/ext4/crypto_key.c index 6e6fbcb..21fdcea 100644 --- a/fs/ext4/crypto_key.c +++ b/fs/ext4/crypto_key.c @@ -372,8 +372,10 @@ int ext4_set_encryption_metadata(struct inode *inode, handle = ext4_journal_start(inode, EXT4_HT_MISC, ext4_jbd2_credits_xattr(inode)); - if (IS_ERR(handle)) - return PTR_ERR(handle); + if (IS_ERR(handle)) { + res = PTR_ERR(handle); + goto errout; + } res = ext4_xattr_set(inode, EXT4_XATTR_INDEX_ENCRYPTION, EXT4_XATTR_NAME_ENCRYPTION_CONTEXT, ctx, sizeof(struct ext4_encryption_context), 0); -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html