diff options
| author | Yehuda Sadeh <yehuda@inktank.com> | 2013-10-15 10:20:48 -0700 |
|---|---|---|
| committer | Sage Weil <sage@inktank.com> | 2013-10-15 15:05:17 -0700 |
| commit | fcf5f117a9111c2d88b8fa5d00c975a8e377df7e (patch) | |
| tree | 664b34c903ac94f6e7cbd3308af5ff7dda5347b0 | |
| parent | 991ed515480114c476cd3c4d761f256d1708fb39 (diff) | |
| download | ceph-fcf5f117a9111c2d88b8fa5d00c975a8e377df7e.tar.gz | |
rgw: fix authenticated users acl group check
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>
(cherry picked from commit bebbd6cb7b71697b34b8f27652cabdc40c97a33b)
| -rw-r--r-- | src/rgw/rgw_acl.cc | 2 |
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); } |