diff options
author | Mike Gilbert <floppym@gentoo.org> | 2018-05-30 23:06:33 -0400 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2018-06-02 09:48:19 +0200 |
commit | 96f64eb5741b157f26a9089816fdd992e959792e (patch) | |
tree | d1f07b373b14f63a574fb19ea93340bafa13693f | |
parent | e0bd32c5cd1d5e04fca9628babe2d87703264ca0 (diff) | |
download | systemd-96f64eb5741b157f26a9089816fdd992e959792e.tar.gz |
basic: set errno in raw_clone() on sparc
sparc sets the carry bit when a syscall fails. Use this information to
set errno and return -1 as appropriate.
The added test case calls raw_clone() with flags known to be invalid
according to the clone(2) manpage.
-rw-r--r-- | src/basic/raw-clone.h | 18 | ||||
-rw-r--r-- | src/test/test-util.c | 4 |
2 files changed, 16 insertions, 6 deletions
diff --git a/src/basic/raw-clone.h b/src/basic/raw-clone.h index d35540903a..6505e5a3cd 100644 --- a/src/basic/raw-clone.h +++ b/src/basic/raw-clone.h @@ -8,6 +8,7 @@ Copyright 2016 Michael Karcher ***/ +#include <errno.h> #include <sched.h> #include <sys/syscall.h> @@ -47,23 +48,28 @@ static inline pid_t raw_clone(unsigned long flags) { * %o1. Inline assembly is needed to get the flag returned * in %o1. */ - int in_child, child_pid; + int in_child, child_pid, error; - asm volatile("mov %2, %%g1\n\t" - "mov %3, %%o0\n\t" + asm volatile("mov %3, %%g1\n\t" + "mov %4, %%o0\n\t" "mov 0 , %%o1\n\t" #if defined(__arch64__) "t 0x6d\n\t" #else "t 0x10\n\t" #endif + "addx %%g0, 0, %2\n\t" "mov %%o1, %0\n\t" "mov %%o0, %1" : - "=r"(in_child), "=r"(child_pid) : + "=r"(in_child), "=r"(child_pid), "=r"(error) : "i"(__NR_clone), "r"(flags) : - "%o1", "%o0", "%g1" ); + "%o1", "%o0", "%g1" "cc" ); - ret = in_child ? 0 : child_pid; + if (error) { + errno = child_pid; + ret = -1; + } else + ret = in_child ? 0 : child_pid; } #else ret = (pid_t) syscall(__NR_clone, flags, NULL); diff --git a/src/test/test-util.c b/src/test/test-util.c index 92d2c77690..af0ada7d2a 100644 --- a/src/test/test-util.c +++ b/src/test/test-util.c @@ -214,6 +214,10 @@ static void test_raw_clone(void) { waitpid(pid, &status, __WCLONE); assert_se(WIFEXITED(status) && WEXITSTATUS(status) == EXIT_SUCCESS); } + + errno = 0; + assert_se(raw_clone(CLONE_FS|CLONE_NEWNS) == -1); + assert_se(errno == EINVAL); } static void test_physical_memory(void) { |