summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorYehuda Sadeh <yehuda@inktank.com>2013-10-15 10:20:48 -0700
committerYehuda Sadeh <yehuda@inktank.com>2013-10-15 10:23:39 -0700
commitbebbd6cb7b71697b34b8f27652cabdc40c97a33b (patch)
treedcadfc5a4f30a043f2a3f0272c03ed1001ea2c65
parenta0ffba6de2e439300c7c181524694f9abf6b085d (diff)
downloadceph-wip-6553.tar.gz
rgw: fix authenticated users acl group checkwip-6553
Fixes: #6553 Backport: bobtail, cuttlefish, dumpling Authenticated users group acl bit was not working correctly. Check to test whether user is anonymous was wrong. Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
-rw-r--r--src/rgw/rgw_acl.cc2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/rgw/rgw_acl.cc b/src/rgw/rgw_acl.cc
index 3f99d72cd5b..02504524847 100644
--- a/src/rgw/rgw_acl.cc
+++ b/src/rgw/rgw_acl.cc
@@ -79,7 +79,7 @@ int RGWAccessControlPolicy::get_perm(string& id, int perm_mask) {
if ((perm & perm_mask) != perm_mask) {
perm |= acl.get_group_perm(ACL_GROUP_ALL_USERS, perm_mask);
- if (compare_group_name(id, ACL_GROUP_ALL_USERS) != 0) {
+ if (!compare_group_name(id, ACL_GROUP_ALL_USERS)) {
/* this is not the anonymous user */
perm |= acl.get_group_perm(ACL_GROUP_AUTHENTICATED_USERS, perm_mask);
}