diff options
author | Mark Johnston <markj@FreeBSD.org> | 2018-11-06 21:57:03 +0000 |
---|---|---|
committer | Mark Johnston <markj@FreeBSD.org> | 2018-11-06 21:57:03 +0000 |
commit | 07702f72e51e2201645165c4d2c292868ade6b1c (patch) | |
tree | 6575163734e4ed104fc13e63248d7171a203f5b2 /sys/vm | |
parent | 2350a089e78bc7911117cd99a4d5bdc1c5e7eb5e (diff) | |
download | src-07702f72e51e2201645165c4d2c292868ade6b1c.tar.gz src-07702f72e51e2201645165c4d2c292868ade6b1c.zip |
Avoid specifying VM_PROT_EXECUTE in mappings from pipe_map and exec_map.
These submaps are used for mapping pipe buffers and execv() argument
strings respectively, so there's no need for such mappings to have
execute permissions.
Reported by: jhb
Reviewed by: alc, jhb, kib
MFC after: 2 weeks
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D17827
Notes
Notes:
svn path=/head/; revision=340205
Diffstat (limited to 'sys/vm')
-rw-r--r-- | sys/vm/vm_kern.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/vm/vm_kern.c b/sys/vm/vm_kern.c index 79108ce1ef64..3720ffbee161 100644 --- a/sys/vm/vm_kern.c +++ b/sys/vm/vm_kern.c @@ -650,8 +650,8 @@ kmap_alloc_wait(vm_map_t map, vm_size_t size) map->needs_wakeup = TRUE; vm_map_unlock_and_wait(map, 0); } - vm_map_insert(map, NULL, 0, addr, addr + size, VM_PROT_ALL, - VM_PROT_ALL, MAP_ACC_CHARGED); + vm_map_insert(map, NULL, 0, addr, addr + size, VM_PROT_RW, VM_PROT_RW, + MAP_ACC_CHARGED); vm_map_unlock(map); return (addr); } |