summaryrefslogtreecommitdiff
path: root/src/internal
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2011-08-23 09:37:39 -0400
committerRich Felker <dalias@aerifal.cx>2011-08-23 09:37:39 -0400
commitdf0b5a49406763aa4719dfad561a5de8924ecd59 (patch)
tree0d5dc42698f2b710dd27156554b10230ba21256b /src/internal
parentc0f344160d22d889460573d003cf349626a38184 (diff)
downloadmusl-df0b5a49406763aa4719dfad561a5de8924ecd59.tar.gz
musl-df0b5a49406763aa4719dfad561a5de8924ecd59.tar.bz2
musl-df0b5a49406763aa4719dfad561a5de8924ecd59.tar.xz
musl-df0b5a49406763aa4719dfad561a5de8924ecd59.zip
security hardening: ensure suid programs have valid stdin/out/err
this behavior (opening fds 0-2 for a suid program) is explicitly allowed (but not required) by POSIX to protect badly-written suid programs from clobbering files they later open. this commit does add some cost in startup code, but the availability of auxv and the security flag will be useful elsewhere in the future. in particular auxv is needed for static-linked vdso support, which is still waiting to be committed (sorry nik!)
Diffstat (limited to 'src/internal')
-rw-r--r--src/internal/libc.h6
1 files changed, 4 insertions, 2 deletions
diff --git a/src/internal/libc.h b/src/internal/libc.h
index 115cd865..d985902c 100644
--- a/src/internal/libc.h
+++ b/src/internal/libc.h
@@ -7,13 +7,15 @@
struct __libc {
void *main_thread;
int threaded;
- int canceldisable;
+ int secure;
+ size_t *auxv;
int (*atexit)(void (*)(void));
void (*fini)(void);
void (*ldso_fini)(void);
volatile int threads_minus_1;
- int ofl_lock;
+ int canceldisable;
FILE *ofl_head;
+ int ofl_lock;
};