Files
Jamie LiuandgVisor bot 3197ed5d53 sentry/kernel: check for pending signals after Task.endInternalStopLocked()
This affects GNU make in particular, which can otherwise receive SIGCHLD during
vfork(), fail to deliver it, and subsequently miss all future SIGCHLDs due to
signal deduplication.

PiperOrigin-RevId: 852342893
2026-01-05 09:58:20 -08:00

263 lines
7.8 KiB
C++

// Copyright 2018 The gVisor Authors.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
#include <errno.h>
#include <signal.h>
#include <stdint.h>
#include <sys/types.h>
#include <sys/wait.h>
#include <unistd.h>
#include <atomic>
#include <string>
#include <utility>
#include "gmock/gmock.h"
#include "gtest/gtest.h"
#include "absl/flags/flag.h"
#include "absl/time/time.h"
#include "test/util/logging.h"
#include "test/util/multiprocess_util.h"
#include "test/util/signal_util.h"
#include "test/util/test_util.h"
#include "test/util/time_util.h"
ABSL_FLAG(bool, vfork_test_child, false,
"If true, run the VforkTest child workload.");
namespace gvisor {
namespace testing {
namespace {
// We don't test with raw CLONE_VFORK to avoid interacting with glibc's use of
// TLS.
//
// Even with vfork(2), we must be careful to do little more in the child than
// call execve(2). We use the simplest sleep function possible, though this is
// still precarious, as we're officially only allowed to call execve(2) and
// _exit(2).
constexpr absl::Duration kChildDelay = absl::Seconds(10);
// Exit code for successful child subprocesses. We don't want to use 0 since
// it's too common, and an execve(2) failure causes the child to exit with the
// errno, so kChildExitCode is chosen to be an unlikely errno:
constexpr int kChildExitCode = 118; // ENOTNAM: Not a XENIX named type file
int64_t MonotonicNow() {
struct timespec now;
TEST_PCHECK(clock_gettime(CLOCK_MONOTONIC, &now) == 0);
return now.tv_sec * 1000000000ll + now.tv_nsec;
}
std::atomic<uint64_t> sigchld_count = 0;
void SigchldCountingHandler(int signo, siginfo_t* siginfo, void* uc) {
if (signo == SIGCHLD) {
sigchld_count.fetch_add(1, std::memory_order_relaxed);
}
}
TEST(VforkTest, ParentStopsUntilChildExits) {
const auto test = [] {
// N.B. Run the test in a single-threaded subprocess because
// vfork is not safe in a multi-threaded process.
const int64_t start = MonotonicNow();
pid_t pid = vfork();
if (pid == 0) {
SleepSafe(kChildDelay);
_exit(kChildExitCode);
}
TEST_PCHECK_MSG(pid > 0, "vfork failed");
MaybeSave();
const int64_t end = MonotonicNow();
absl::Duration dur = absl::Nanoseconds(end - start);
TEST_CHECK(dur >= kChildDelay);
int status = 0;
TEST_PCHECK(RetryEINTR(waitpid)(pid, &status, 0));
TEST_CHECK(WIFEXITED(status));
TEST_CHECK(WEXITSTATUS(status) == kChildExitCode);
};
EXPECT_THAT(InForkedProcess(test), IsPosixErrorOkAndHolds(0));
}
TEST(VforkTest, ParentStopsUntilChildExecves) {
ExecveArray const owned_child_argv = {"/proc/self/exe", "--vfork_test_child"};
char* const* const child_argv = owned_child_argv.get();
const auto test = [&] {
const int64_t start = MonotonicNow();
pid_t pid = vfork();
if (pid == 0) {
SleepSafe(kChildDelay);
execve(child_argv[0], child_argv, /* envp = */ nullptr);
_exit(errno);
}
// Don't attempt save/restore until after recording end_time,
// since the test expects an upper bound on the time spent
// stopped.
int saved_errno = errno;
const int64_t end = MonotonicNow();
errno = saved_errno;
TEST_PCHECK_MSG(pid > 0, "vfork failed");
MaybeSave();
absl::Duration dur = absl::Nanoseconds(end - start);
// The parent should resume execution after execve, but before
// the post-execve test child exits.
TEST_CHECK(dur >= kChildDelay);
TEST_CHECK(dur <= 2 * kChildDelay);
int status = 0;
TEST_PCHECK(RetryEINTR(waitpid)(pid, &status, 0));
TEST_CHECK(WIFEXITED(status));
TEST_CHECK(WEXITSTATUS(status) == kChildExitCode);
};
EXPECT_THAT(InForkedProcess(test), IsPosixErrorOkAndHolds(0));
}
// A vfork child does not unstop the parent a second time when it exits after
// exec.
TEST(VforkTest, ExecedChildExitDoesntUnstopParent) {
ExecveArray const owned_child_argv = {"/proc/self/exe", "--vfork_test_child"};
char* const* const child_argv = owned_child_argv.get();
const auto test = [&] {
pid_t pid1 = vfork();
if (pid1 == 0) {
execve(child_argv[0], child_argv, /* envp = */ nullptr);
_exit(errno);
}
TEST_PCHECK_MSG(pid1 > 0, "vfork failed");
MaybeSave();
// pid1 exec'd and is now sleeping.
SleepSafe(kChildDelay / 2);
const int64_t start = MonotonicNow();
pid_t pid2 = vfork();
if (pid2 == 0) {
SleepSafe(kChildDelay);
_exit(kChildExitCode);
}
TEST_PCHECK_MSG(pid2 > 0, "vfork failed");
MaybeSave();
const int64_t end = MonotonicNow();
absl::Duration dur = absl::Nanoseconds(end - start);
// The parent should resume execution only after pid2 exits, not
// when pid1 exits.
TEST_CHECK(dur >= kChildDelay);
int status = 0;
TEST_PCHECK(RetryEINTR(waitpid)(pid1, &status, 0));
TEST_CHECK(WIFEXITED(status));
TEST_CHECK(WEXITSTATUS(status) == kChildExitCode);
TEST_PCHECK(RetryEINTR(waitpid)(pid2, &status, 0));
TEST_CHECK(WIFEXITED(status));
TEST_CHECK(WEXITSTATUS(status) == kChildExitCode);
};
EXPECT_THAT(InForkedProcess(test), IsPosixErrorOkAndHolds(0));
}
TEST(VforkTest, SigchldReceivedDuringVforkIsDelivered) {
const auto test = [&] {
struct sigaction sa = {};
sa.sa_sigaction = SigchldCountingHandler;
sigfillset(&sa.sa_mask);
sa.sa_flags = SA_SIGINFO;
const auto cleanup_sigaction =
TEST_CHECK_NO_ERRNO_AND_VALUE(ScopedSigaction(SIGCHLD, sa));
const auto cleanup_sigmask =
TEST_CHECK_NO_ERRNO_AND_VALUE(ScopedSignalMask(SIG_UNBLOCK, SIGCHLD));
const auto sigchld_count_before =
sigchld_count.load(std::memory_order_relaxed);
std::atomic_signal_fence(std::memory_order_seq_cst);
// Create a non-vfork child that will exit after kChildDelay/2, sending
// SIGCHLD.
const pid_t pid1 = fork();
if (pid1 == 0) {
SleepSafe(kChildDelay / 2);
_exit(kChildExitCode);
}
TEST_PCHECK_MSG(pid1 > 0, "fork failed");
MaybeSave();
// Create a vfork child that will exit after kChildDelay, so that SIGCHLD
// is sent for the first child while the parent is stopped for vfork.
const pid_t pid2 = vfork();
if (pid2 == 0) {
SleepSafe(kChildDelay);
_exit(kChildExitCode);
}
TEST_PCHECK_MSG(pid2 > 0, "vfork failed");
MaybeSave();
int status = 0;
TEST_PCHECK(RetryEINTR(waitpid)(pid1, &status, 0));
TEST_CHECK(WIFEXITED(status));
TEST_CHECK(WEXITSTATUS(status) == kChildExitCode);
TEST_PCHECK(RetryEINTR(waitpid)(pid2, &status, 0));
TEST_CHECK(WIFEXITED(status));
TEST_CHECK(WEXITSTATUS(status) == kChildExitCode);
// Check that at least one SIGCHLD was delivered (two child processes died,
// but one SIGCHLD may have been deduplicated.)
std::atomic_signal_fence(std::memory_order_seq_cst);
const auto sigchld_count_after =
sigchld_count.load(std::memory_order_relaxed);
TEST_CHECK(sigchld_count_before < sigchld_count_after);
};
EXPECT_THAT(InForkedProcess(test), IsPosixErrorOkAndHolds(0));
}
int RunChild() {
SleepSafe(kChildDelay);
return kChildExitCode;
}
} // namespace
} // namespace testing
} // namespace gvisor
int main(int argc, char** argv) {
gvisor::testing::TestInit(&argc, &argv);
if (absl::GetFlag(FLAGS_vfork_test_child)) {
return gvisor::testing::RunChild();
}
return gvisor::testing::RunAllTests();
}