mirror of
https://github.com/systemd/systemd.git
synced 2024-12-25 01:34:28 +03:00
oomd: fix unreachable test case in test-oomd-util
This conditional with !empty_or_root(ctx->path) always returns false because the most recent oomd_cgroup_context_acquire() call was with the root cgroup. Make sure this test case can be reached by checking cgroup instead of ctx->path. While here, use an unused uid (61183) instead of the nobody uid so the test case does not fail in unprivileged LXD containers.
This commit is contained in:
parent
8918609348
commit
f05bcc1894
@ -475,9 +475,9 @@ static void test_oomd_fetch_cgroup_oom_preference(void) {
|
||||
|
||||
/* Assert that avoid/omit are not set if the cgroup and prefix are not
|
||||
* owned by the same user.*/
|
||||
if (test_xattrs && !empty_or_root(ctx->path)) {
|
||||
if (test_xattrs && !empty_or_root(cgroup)) {
|
||||
ctx = oomd_cgroup_context_free(ctx);
|
||||
assert_se(cg_set_access(SYSTEMD_CGROUP_CONTROLLER, cgroup, 65534, 0) >= 0);
|
||||
assert_se(cg_set_access(SYSTEMD_CGROUP_CONTROLLER, cgroup, 61183, 0) >= 0);
|
||||
assert_se(oomd_cgroup_context_acquire(cgroup, &ctx) == 0);
|
||||
|
||||
assert_se(oomd_fetch_cgroup_oom_preference(ctx, NULL) == 0);
|
||||
|
Loading…
Reference in New Issue
Block a user