diff options
author | gystemd <gystemd@gmail.com> | 2021-08-17 23:25:47 +0200 |
---|---|---|
committer | gystemd <gystemd@gmail.com> | 2021-08-19 01:05:42 +0200 |
commit | ce4e54186e7c4a59bcdd9a74d89c5d4c2a7174f9 (patch) | |
tree | 1763604ef1b4866efcaef1d8b27ade69ec4ef3df | |
parent | 482de52b60d9ac0bc40f264c82c8eb094401a119 (diff) |
Added a system call test for SetForegroundProcessGroup
-Added a test to check if the process in background is blocking SIGTTOU
-Some minor formatting fixes
-rw-r--r-- | test/syscalls/linux/pty.cc | 79 |
1 files changed, 53 insertions, 26 deletions
diff --git a/test/syscalls/linux/pty.cc b/test/syscalls/linux/pty.cc index 2e0dc6b3e..c008c89e7 100644 --- a/test/syscalls/linux/pty.cc +++ b/test/syscalls/linux/pty.cc @@ -1640,57 +1640,84 @@ TEST_F(JobControlTest, DISABLED_SetForegroundProcessGroup) { ASSERT_NO_ERRNO(res); } -// This test verify if a SIGTTOU signal is sent to the calling process'group +// This test verifies if a SIGTTOU signal is sent to the calling process's group // when tcsetpgrp is called by a background process TEST_F(JobControlTest, SetForegroundProcessGroupSIGTTOUBackground) { auto res = RunInChild([=]() { - setsid(); + TEST_PCHECK(setsid() >= 0); TEST_PCHECK(!ioctl(replica_.get(), TIOCSCTTY, 0)); pid_t grandchild = fork(); if (!grandchild) { - //assign a different pgid to the child so it will result as - //a background process - setpgid(grandchild, getpid()); - tcsetpgrp(replica_.get(), getpgid(0)); - // We should never reach this. - _exit(1); - } + // Assign a different pgid to the child so it will result as + // a background process. + TEST_PCHECK(!setpgid(grandchild, getpid())); + TEST_PCHECK(!tcsetpgrp(replica_.get(), getpgid(0))); + // We should never reach this. + _exit(1); + } int wstatus; TEST_PCHECK(waitpid(grandchild, &wstatus, WSTOPPED) == grandchild); - TEST_PCHECK(WSTOPSIG(wstatus)==SIGTTOU); + TEST_PCHECK(WSTOPSIG(wstatus) == SIGTTOU); }); ASSERT_NO_ERRNO(res); } -// This test verify that a SIGTTOU signal is not delivered to +// This test verifies that a SIGTTOU signal is not delivered to // a background process which calls tcsetpgrp and is ignoring SIGTTOU TEST_F(JobControlTest, SetForegroundProcessGroupSIGTTOUIgnored) { auto res = RunInChild([=]() { - setsid(); + TEST_PCHECK(setsid() >= 0); TEST_PCHECK(!ioctl(replica_.get(), TIOCSCTTY, 0)); pid_t grandchild = fork(); if (!grandchild) { - // ignore SIGTTOU so the child in background won't - // be stopped when it will call tcsetpgrp - struct sigaction sa = {}; - sa.sa_handler = SIG_IGN; - sa.sa_flags = 0; - sigemptyset(&sa.sa_mask); - sigaction(SIGTTOU, &sa, NULL); - //assign a different pgid to the child so it will result as - //a background process - setpgid(grandchild, getpid()); - tcsetpgrp(replica_.get(), getpgid(0)); - _exit(0); - } + // Ignore SIGTTOU so the child in background won't + // be stopped when it will call tcsetpgrp + struct sigaction sa = {}; + sa.sa_handler = SIG_IGN; + sa.sa_flags = 0; + sigemptyset(&sa.sa_mask); + sigaction(SIGTTOU, &sa, NULL); + // Assign a different pgid to the child so it will result as + // a background process. + TEST_PCHECK(!setpgid(grandchild, getpid())); + TEST_PCHECK(!tcsetpgrp(replica_.get(), getpgid(0))); + _exit(0); + } int wstatus; TEST_PCHECK(waitpid(grandchild, &wstatus, WSTOPPED) == grandchild); - TEST_PCHECK(WSTOPSIG(wstatus)!=SIGTTOU); + TEST_PCHECK(WSTOPSIG(wstatus) != SIGTTOU); TEST_PCHECK(WIFEXITED(wstatus)); }); ASSERT_NO_ERRNO(res); } +// This test verifies that a SIGTTOU signal is not delivered to +// a background process which calls tcsetpgrp and is blocking SIGTTOU +TEST_F(JobControlTest, SetForegroundProcessGroupSIGTTOUBlocked) { + auto res = RunInChild([=]() { + TEST_PCHECK(setsid() >= 0); + TEST_PCHECK(!ioctl(replica_.get(), TIOCSCTTY, 0)); + pid_t grandchild = fork(); + if (!grandchild) { + // Block SIGTTOU so the child in background won't + // be stopped when it will call tcsetpgrp + sigset_t signal_set; + sigemptyset(&signal_set); + sigaddset(&signal_set, SIGTTOU); + sigprocmask(SIG_BLOCK, &signal_set, NULL); + // Assign a different pgid to the child so it will result as + // a background process. + TEST_PCHECK(!setpgid(grandchild, getpid())); + TEST_PCHECK(!tcsetpgrp(replica_.get(), getpgid(0))); + _exit(0); + } + int wstatus; + TEST_PCHECK(waitpid(grandchild, &wstatus, WSTOPPED) == grandchild); + TEST_PCHECK(WSTOPSIG(wstatus) != SIGTTOU); + }); + ASSERT_NO_ERRNO(res); +} + TEST_F(JobControlTest, SetForegroundProcessGroupWrongTTY) { pid_t pid = getpid(); ASSERT_THAT(ioctl(replica_.get(), TIOCSPGRP, &pid), |