From: Pan Bian <bianpan2...@163.com> commit 512c15ef05d73a04f1aef18a3bc61a8bb516f323 upstream.
The inode update should be stopped before returing the error code. Signed-off-by: Pan Bian <bianpan2...@163.com> Link: https://lore.kernel.org/r/20210117085732.93788-1-bianpan2...@163.com Fixes: 8016e29f4362 ("ext4: fast commit recovery path") Cc: sta...@kernel.org Reviewed-by: Harshad Shirwadkar <harshadshirwad...@gmail.com> Signed-off-by: Theodore Ts'o <ty...@mit.edu> Signed-off-by: Greg Kroah-Hartman <gre...@linuxfoundation.org> --- fs/ext4/inode.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) --- a/fs/ext4/inode.c +++ b/fs/ext4/inode.c @@ -5389,8 +5389,10 @@ int ext4_setattr(struct dentry *dentry, inode->i_gid = attr->ia_gid; error = ext4_mark_inode_dirty(handle, inode); ext4_journal_stop(handle); - if (unlikely(error)) + if (unlikely(error)) { + ext4_fc_stop_update(inode); return error; + } } if (attr->ia_valid & ATTR_SIZE) {
