diff options
author | Evgeny Vereshchagin <evvers@ya.ru> | 2016-11-20 19:38:58 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-11-20 19:38:58 +0300 |
commit | 270f419316758162bfead87608cbaeeb4fd6987e (patch) | |
tree | 47b8efe91c3326345d861f888ad59a38a44d502e /src/test | |
parent | 3f59367e6fae0ba991d1e287a0f784ecdcda79af (diff) | |
parent | cf5f9bb8402906d37cdd55e8b06da9c7b5ac06a5 (diff) |
Merge pull request #4691 from keszybz/test-suite-fixes
Test suite fixes
Diffstat (limited to 'src/test')
-rw-r--r-- | src/test/test-ipcrm.c | 8 | ||||
-rw-r--r-- | src/test/test-process-util.c | 9 |
2 files changed, 13 insertions, 4 deletions
diff --git a/src/test/test-ipcrm.c b/src/test/test-ipcrm.c index 551eba7215..463e135e2b 100644 --- a/src/test/test-ipcrm.c +++ b/src/test/test-ipcrm.c @@ -28,9 +28,11 @@ int main(int argc, char *argv[]) { r = get_user_creds(&name, &uid, NULL, NULL, NULL); if (r < 0) { - log_error_errno(r, "Failed to resolve \"%s\": %m", name); - return EXIT_FAILURE; + log_full_errno(r == -ESRCH ? LOG_NOTICE : LOG_ERR, + r, "Failed to resolve \"%s\": %m", name); + return r == -ESRCH ? EXIT_TEST_SKIP : EXIT_FAILURE; } - return clean_ipc_by_uid(uid) < 0 ? EXIT_FAILURE : EXIT_SUCCESS; + r = clean_ipc_by_uid(uid); + return r < 0 ? EXIT_FAILURE : EXIT_SUCCESS; } diff --git a/src/test/test-process-util.c b/src/test/test-process-util.c index 9ada46b1e9..7242b2c8b5 100644 --- a/src/test/test-process-util.c +++ b/src/test/test-process-util.c @@ -193,7 +193,14 @@ static void test_get_process_cmdline_harder(void) { fd = mkostemp(path, O_CLOEXEC); assert_se(fd >= 0); - assert_se(mount(path, "/proc/self/cmdline", "bind", MS_BIND, NULL) >= 0); + + if (mount(path, "/proc/self/cmdline", "bind", MS_BIND, NULL) < 0) { + /* This happens under selinux… Abort the test in this case. */ + log_warning_errno(errno, "mount(..., \"/proc/self/cmdline\", \"bind\", ...) failed: %m"); + assert(errno == EACCES); + return; + } + assert_se(unlink(path) >= 0); assert_se(prctl(PR_SET_NAME, "testa") >= 0); |