Luis Henriques 899eb9
From: Chengguang Xu <cgxu519@gmx.com>
Luis Henriques 899eb9
Date: Mon, 9 Jul 2018 22:48:07 +0800
Luis Henriques 899eb9
Subject: ceph: return errors from posix_acl_equiv_mode() correctly
Luis Henriques 899eb9
Git-commit: 61ad36d47dd273b7b8c3d63fc8359d96f7976c79
Luis Henriques 899eb9
Patch-mainline: v4.19-rc1
Luis Henriques 899eb9
References: bsc#1107320
Luis Henriques 899eb9
Luis Henriques 899eb9
In order to return correct error code should replace variable ret
Luis Henriques 899eb9
using err in error case.
Luis Henriques 899eb9
Luis Henriques 899eb9
Signed-off-by: Chengguang Xu <cgxu519@gmx.com>
Luis Henriques 899eb9
Reviewed-by: "Yan, Zheng" <zyan@redhat.com>
Luis Henriques 899eb9
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Luis Henriques 899eb9
Acked-by: Luis Henriques <lhenriques@suse.com>
Luis Henriques 899eb9
---
Luis Henriques 899eb9
 fs/ceph/acl.c |    6 +++---
Luis Henriques 899eb9
 1 file changed, 3 insertions(+), 3 deletions(-)
Luis Henriques 899eb9
Luis Henriques 899eb9
--- a/fs/ceph/acl.c
Luis Henriques 899eb9
+++ b/fs/ceph/acl.c
Luis Henriques 899eb9
@@ -171,10 +171,10 @@ int ceph_pre_init_acls(struct inode *dir
Luis Henriques 899eb9
 		return err;
Luis Henriques 899eb9
 
Luis Henriques 899eb9
 	if (acl) {
Luis Henriques 899eb9
-		int ret = posix_acl_equiv_mode(acl, mode);
Luis Henriques 899eb9
-		if (ret < 0)
Luis Henriques 899eb9
+		err = posix_acl_equiv_mode(acl, mode);
Luis Henriques 899eb9
+		if (err < 0)
Luis Henriques 899eb9
 			goto out_err;
Luis Henriques 899eb9
-		if (ret == 0) {
Luis Henriques 899eb9
+		if (err == 0) {
Luis Henriques 899eb9
 			posix_acl_release(acl);
Luis Henriques 899eb9
 			acl = NULL;
Luis Henriques 899eb9
 		}
Luis Henriques 899eb9