summaryrefslogtreecommitdiff
path: root/ipc
diff options
context:
space:
mode:
authorAlexey Gladkov <legion@kernel.org>2022-10-05 19:07:30 +0200
committerEric W. Biederman <ebiederm@xmission.com>2022-10-05 16:08:16 -0500
commitb895e6689ebf57f9d00956d1a9eb2e75d8ab25ad (patch)
treeeb2f095f141162c0eb816f8105f43f8ec7cea00c /ipc
parenta799be9bafe3ab638660bd3122469bddbc18fc38 (diff)
downloadlinux-next-b895e6689ebf57f9d00956d1a9eb2e75d8ab25ad.tar.gz
sysctl: Fix mq permission check
The kernel test robot found a typo that causes mq_permissions() to return a useless value. Reported-by: kernel test robot <yujie.liu@intel.com> Link: https://lore.kernel.org/r/202210052100.86dbb00a-yujie.liu@intel.com Fixes: 7608b6a72ed0 ("sysctl: Allow to change limits for posix messages queues") Signed-off-by: Alexey Gladkov <legion@kernel.org> Link: https://lkml.kernel.org/r/20221005170730.835761-1-legion@kernel.org Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Diffstat (limited to 'ipc')
-rw-r--r--ipc/mq_sysctl.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/ipc/mq_sysctl.c b/ipc/mq_sysctl.c
index ff1054fbbacc..ce03930aced5 100644
--- a/ipc/mq_sysctl.c
+++ b/ipc/mq_sysctl.c
@@ -102,7 +102,7 @@ static int mq_permissions(struct ctl_table_header *head, struct ctl_table *table
if (uid_eq(current_euid(), ns_root_uid))
mode >>= 6;
- if (in_egroup_p(ns_root_gid))
+ else if (in_egroup_p(ns_root_gid))
mode >>= 3;
mode &= 7;