diff options
| author | Szabolcs Nagy <szabolcs.nagy@arm.com> | 2022-10-24 16:18:42 +0100 |
|---|---|---|
| committer | Szabolcs Nagy <szabolcs.nagy@arm.com> | 2022-10-27 14:46:52 +0100 |
| commit | c31913cc4121f13e1ef283bb8f82c1a9f21358f0 (patch) | |
| tree | 4cac1673a864a86329241a8480da62f3a937ad18 | |
| parent | 835ba1822cb031f526ca95538d8c48f71bd3a718 (diff) | |
| download | glibc-c31913cc4121f13e1ef283bb8f82c1a9f21358f0.tar glibc-c31913cc4121f13e1ef283bb8f82c1a9f21358f0.tar.gz glibc-c31913cc4121f13e1ef283bb8f82c1a9f21358f0.tar.bz2 glibc-c31913cc4121f13e1ef283bb8f82c1a9f21358f0.zip | |
cheri: nptl: Fix thread stack capability permissions
| -rw-r--r-- | nptl/allocatestack.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/nptl/allocatestack.c b/nptl/allocatestack.c index 219854f2cb..755ace5ec4 100644 --- a/nptl/allocatestack.c +++ b/nptl/allocatestack.c @@ -52,6 +52,13 @@ # define MAP_STACK 0 #endif +/* On CHERI targets ensure the mmap returned capability has RW permissions. */ +#ifdef PROT_MAX +# define PROT_MAX_RW PROT_MAX (PROT_READ | PROT_WRITE) +#else +# define PROT_MAX_RW 0 +#endif + /* Get a stack frame from the cache. We have to match by size since some blocks might be too small or far too large. */ static struct pthread * @@ -363,7 +370,8 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp, /* If a guard page is required, avoid committing memory by first allocate with PROT_NONE and then reserve with required permission excluding the guard page. */ - mem = __mmap (NULL, size, (guardsize == 0) ? prot : PROT_NONE, + mem = __mmap (NULL, size, + (guardsize == 0) ? prot : PROT_NONE | PROT_MAX_RW, MAP_PRIVATE | MAP_ANONYMOUS | MAP_STACK, -1, 0); if (__glibc_unlikely (mem == MAP_FAILED)) |