From a7e54a29f8d3de1aa81282076695737fb71f434d Mon Sep 17 00:00:00 2001 From: Roger Zanoni Date: Tue, 7 Jul 2026 15:53:01 -0300 Subject: [PATCH] nplb: use SIGUSR2 in KillSendsSignalToSelf SIGUSR1 is intercepted by the Android runtime and never reaches the handler installed by the test. Switch the test to SIGUSR2. Bug: 532068409 --- starboard/nplb/posix_compliance/posix_signal_test.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/starboard/nplb/posix_compliance/posix_signal_test.cc b/starboard/nplb/posix_compliance/posix_signal_test.cc index 78d92bbdc123..d37267e374f9 100644 --- a/starboard/nplb/posix_compliance/posix_signal_test.cc +++ b/starboard/nplb/posix_compliance/posix_signal_test.cc @@ -236,13 +236,13 @@ TEST_F(PosixSignalTest, KillSendsSignalToSelf) { struct sigaction sa = {}; sa.sa_handler = PipeWritingSignalHandler; sigemptyset(&sa.sa_mask); - ASSERT_EQ(sigaction(SIGUSR1, &sa, nullptr), 0); + ASSERT_EQ(sigaction(SIGUSR2, &sa, nullptr), 0); - ASSERT_EQ(kill(getpid(), SIGUSR1), 0); + ASSERT_EQ(kill(getpid(), SIGUSR2), 0); int received_signal = 0; EXPECT_TRUE(WaitForSignalWithTimeout(&received_signal, 1000)); - EXPECT_EQ(received_signal, SIGUSR1); + EXPECT_EQ(received_signal, SIGUSR2); } TEST_F(PosixSignalTest, PauseIsInterruptedBySignal) {