summaryrefslogtreecommitdiff
path: root/ipc/msg.c
diff options
context:
space:
mode:
authorAl Viro <viro@zeniv.linux.org.uk>2017-07-09 17:10:32 +0300
committerAl Viro <viro@zeniv.linux.org.uk>2017-07-16 03:46:45 +0300
commit28327fae62b011216026b66299882c53b95b4500 (patch)
treea5d0dc678591875f06f45c88cdbbb98c9a0cf8ba /ipc/msg.c
parentc0ebccb6fa1e2c9c3377fa8136e6d8bc006fca64 (diff)
downloadlinux-28327fae62b011216026b66299882c53b95b4500.tar.xz
ipc: make use of compat ipc_perm helpers
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'ipc/msg.c')
-rw-r--r--ipc/msg.c28
1 files changed, 4 insertions, 24 deletions
diff --git a/ipc/msg.c b/ipc/msg.c
index 3400012e1ce8..94690fb53f66 100644
--- a/ipc/msg.c
+++ b/ipc/msg.c
@@ -591,22 +591,14 @@ static int copy_compat_msqid_from_user(struct msqid64_ds *out, void __user *buf,
memset(out, 0, sizeof(*out));
if (version == IPC_64) {
struct compat_msqid64_ds *p = buf;
- struct compat_ipc64_perm v;
- if (copy_from_user(&v, &p->msg_perm, sizeof(v)))
+ if (get_compat_ipc64_perm(&out->msg_perm, &p->msg_perm))
return -EFAULT;
- out->msg_perm.uid = v.uid;
- out->msg_perm.gid = v.gid;
- out->msg_perm.mode = v.mode;
if (get_user(out->msg_qbytes, &p->msg_qbytes))
return -EFAULT;
} else {
struct compat_msqid_ds *p = buf;
- struct compat_ipc_perm v;
- if (copy_from_user(&v, &p->msg_perm, sizeof(v)))
+ if (get_compat_ipc_perm(&out->msg_perm, &p->msg_perm))
return -EFAULT;
- out->msg_perm.uid = v.uid;
- out->msg_perm.gid = v.gid;
- out->msg_perm.mode = v.mode;
if (get_user(out->msg_qbytes, &p->msg_qbytes))
return -EFAULT;
}
@@ -619,13 +611,7 @@ static int copy_compat_msqid_to_user(void __user *buf, struct msqid64_ds *in,
if (version == IPC_64) {
struct compat_msqid64_ds v;
memset(&v, 0, sizeof(v));
- v.msg_perm.key = in->msg_perm.key;
- v.msg_perm.uid = in->msg_perm.uid;
- v.msg_perm.gid = in->msg_perm.gid;
- v.msg_perm.cuid = in->msg_perm.cuid;
- v.msg_perm.cgid = in->msg_perm.cgid;
- v.msg_perm.mode = in->msg_perm.mode;
- v.msg_perm.seq = in->msg_perm.seq;
+ to_compat_ipc64_perm(&v.msg_perm, &in->msg_perm);
v.msg_stime = in->msg_stime;
v.msg_rtime = in->msg_rtime;
v.msg_ctime = in->msg_ctime;
@@ -638,13 +624,7 @@ static int copy_compat_msqid_to_user(void __user *buf, struct msqid64_ds *in,
} else {
struct compat_msqid_ds v;
memset(&v, 0, sizeof(v));
- v.msg_perm.key = in->msg_perm.key;
- SET_UID(v.msg_perm.uid, in->msg_perm.uid);
- SET_GID(v.msg_perm.gid, in->msg_perm.gid);
- SET_UID(v.msg_perm.cuid, in->msg_perm.cuid);
- SET_GID(v.msg_perm.cgid, in->msg_perm.cgid);
- v.msg_perm.mode = in->msg_perm.mode;
- v.msg_perm.seq = in->msg_perm.seq;
+ to_compat_ipc_perm(&v.msg_perm, &in->msg_perm);
v.msg_stime = in->msg_stime;
v.msg_rtime = in->msg_rtime;
v.msg_ctime = in->msg_ctime;