add issetugid function to check for elevated privilege
authorBrent Cook <busterb@gmail.com>
Tue, 15 Jul 2014 16:30:07 +0000 (16:30 +0000)
committerRich Felker <dalias@aerifal.cx>
Sun, 20 Jul 2014 01:39:18 +0000 (21:39 -0400)
this function provides a way for third-party library code to use the
same logic that's used internally in libc for suppressing untrusted
input/state (e.g. the environment) when the application is running
with privleges elevated by the setuid or setgid bit or some other
mechanism. its semantics are intended to match the openbsd function by
the same name.

there was some question as to whether this function is necessary:
getauxval(AT_SECURE) was proposed as an alternative. however, this has
several drawbacks. the most obvious is that it asks programmers to be
aware of an implementation detail of ELF-based systems (the aux
vector) rather than simply the semantic predicate to be checked. and
trying to write a safe, reliable version of issetugid in terms of
getauxval is difficult. for example, early versions of the glibc
getauxval did not report ENOENT, which could lead to false negatives
if AT_SECURE was not present in the aux vector (this could probably
only happen when running on non-linux kernels under linux emulation,
since glibc does not support linux versions old enough to lack
AT_SECURE). as for musl, getauxval has always properly reported
errors, but prior to commit 7bece9c2095ee81f14b1088f6b0ba2f37fecb283,
the musl implementation did not emulate AT_SECURE if missing, which
would result in a false positive. since musl actually does partially
support kernels that lack AT_SECURE, this was problematic.

the intent is that library authors will use issetugid if its
availability is detected at build time, and only fall back to the
unreliable alternatives on systems that lack it.

patch by Brent Cook. commit message/rationale by Rich Felker.

include/unistd.h
src/misc/issetugid.c [new file with mode: 0644]

index bb19cd898310e7a8ac2bfcbc72189086d8f93a3e..ac6055a59fb5da99f24b5a4702c9f3459a9ea63a 100644 (file)
@@ -178,6 +178,7 @@ char *getusershell(void);
 int acct(const char *);
 long syscall(long, ...);
 int execvpe(const char *, char *const [], char *const []);
+int issetugid(void);
 #endif
 
 #ifdef _GNU_SOURCE
diff --git a/src/misc/issetugid.c b/src/misc/issetugid.c
new file mode 100644 (file)
index 0000000..6ffd930
--- /dev/null
@@ -0,0 +1,7 @@
+#include <unistd.h>
+#include "libc.h"
+
+int issetugid(void)
+{
+       return libc.secure;
+}