ubifs: Call iput(xino) only once in ubifs_purge_xattrs()

An iput(xino) call was immediately used after a return value check
for a remove_xattr() call in this function implementation.
Thus call such a function only once instead directly before the check.

This issue was transformed by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Reviewed-by: Zhihao Cheng <chengzhihao1@huawei.com>
Signed-off-by: Richard Weinberger <richard@nod.at>

diff --git a/fs/ubifs/xattr.c b/fs/ubifs/xattr.c
index c673fd03d9b43b9bb1da853e478afff9a8398b7e..cf77fb2074dfca7f5295f32b018820e5df3bbf77 100644 (file)
--- a/fs/ubifs/xattr.c
+++ b/fs/ubifs/xattr.c
@@ -541,16 +541,14 @@ int ubifs_purge_xattrs(struct inode *host)
 
                clear_nlink(xino);
                err = remove_xattr(c, host, xino, &nm);
+               iput(xino);
                if (err) {
                        kfree(pxent);
                        kfree(xent);
-                       iput(xino);
                        ubifs_err(c, "cannot remove xattr, error %d", err);
                        goto out_err;
                }
 
-               iput(xino);
-
                kfree(pxent);
                pxent = xent;
                key_read(c, &xent->key, &key);