diff options
| author | Ricky Zhou <rickyz@chromium.org> | 2016-10-13 10:34:08 -0700 |
|---|---|---|
| committer | Kees Cook <keescook@chromium.org> | 2016-11-01 08:58:10 -0700 |
| commit | 1ff120504f8c322a03fbce035d99e29e741da725 (patch) | |
| tree | 08c14543552cc15c4bd918ea0bd062483b6cfeda /samples | |
| parent | 0af04ba5640dd6318f117a9fcbfde886516a0220 (diff) | |
| download | op-kernel-dev-1ff120504f8c322a03fbce035d99e29e741da725.zip op-kernel-dev-1ff120504f8c322a03fbce035d99e29e741da725.tar.gz | |
samples/seccomp: Enable PR_SET_NO_NEW_PRIVS in dropper
Either CAP_SYS_ADMIN or PR_SET_NO_NEW_PRIVS is required to enable
seccomp. This allows samples/seccomp/dropper to be run without
CAP_SYS_ADMIN.
Signed-off-by: Ricky Zhou <rickyz@chromium.org>
Signed-off-by: Kees Cook <keescook@chromium.org>
Diffstat (limited to 'samples')
| -rw-r--r-- | samples/seccomp/dropper.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/samples/seccomp/dropper.c b/samples/seccomp/dropper.c index c69c347..68325ca 100644 --- a/samples/seccomp/dropper.c +++ b/samples/seccomp/dropper.c @@ -11,7 +11,6 @@ * When run, returns the specified errno for the specified * system call number against the given architecture. * - * Run this one as root as PR_SET_NO_NEW_PRIVS is not called. */ #include <errno.h> @@ -42,8 +41,12 @@ static int install_filter(int nr, int arch, int error) .len = (unsigned short)(sizeof(filter)/sizeof(filter[0])), .filter = filter, }; + if (prctl(PR_SET_NO_NEW_PRIVS, 1, 0, 0, 0)) { + perror("prctl(NO_NEW_PRIVS)"); + return 1; + } if (prctl(PR_SET_SECCOMP, 2, &prog)) { - perror("prctl"); + perror("prctl(PR_SET_SECCOMP)"); return 1; } return 0; |
