diff options
author | jilles <jilles@FreeBSD.org> | 2013-08-18 13:27:04 +0000 |
---|---|---|
committer | jilles <jilles@FreeBSD.org> | 2013-08-18 13:27:04 +0000 |
commit | 1d1d0428d27d178332c36d3cf6b9d05f4aa88bef (patch) | |
tree | 90285db54b39e41060e8aea8c1b932627366b2ad /sys/kern | |
parent | 836cb97bd119a53395314164575376d6b0791093 (diff) | |
download | FreeBSD-src-1d1d0428d27d178332c36d3cf6b9d05f4aa88bef.zip FreeBSD-src-1d1d0428d27d178332c36d3cf6b9d05f4aa88bef.tar.gz |
Disallow opening a POSIX message queue for execute.
O_EXEC was formerly ignored, so equivalent to O_RDONLY.
Reject O_EXEC with [EINVAL] like the invalid mode 3.
Diffstat (limited to 'sys/kern')
-rw-r--r-- | sys/kern/uipc_mqueue.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/kern/uipc_mqueue.c b/sys/kern/uipc_mqueue.c index 5a1a414..62c54d3 100644 --- a/sys/kern/uipc_mqueue.c +++ b/sys/kern/uipc_mqueue.c @@ -2046,7 +2046,7 @@ sys_kmq_open(struct thread *td, struct kmq_open_args *uap) struct mq_attr attr; int flags, error; - if ((uap->flags & O_ACCMODE) == O_ACCMODE) + if ((uap->flags & O_ACCMODE) == O_ACCMODE || uap->flags & O_EXEC) return (EINVAL); flags = FFLAGS(uap->flags); if ((flags & O_CREAT) != 0 && uap->attr != NULL) { @@ -2682,7 +2682,7 @@ freebsd32_kmq_open(struct thread *td, struct freebsd32_kmq_open_args *uap) struct mq_attr32 attr32; int flags, error; - if ((uap->flags & O_ACCMODE) == O_ACCMODE) + if ((uap->flags & O_ACCMODE) == O_ACCMODE || uap->flags & O_EXEC) return (EINVAL); flags = FFLAGS(uap->flags); if ((flags & O_CREAT) != 0 && uap->attr != NULL) { |