Avi Drissman | e4622aa | 2022-09-08 20:36:06 | [diff] [blame] | 1 | // Copyright 2013 The Chromium Authors |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 5 | #include "base/sync_socket.h" |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 6 | |
Jan Keitel | 351a86c6 | 2024-08-03 08:25:31 | [diff] [blame] | 7 | #include "base/containers/span.h" |
Keishi Hattori | 0e45c02 | 2021-11-27 09:25:52 | [diff] [blame] | 8 | #include "base/memory/raw_ptr.h" |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 9 | #include "base/synchronization/waitable_event.h" |
| 10 | #include "base/threading/platform_thread.h" |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 11 | #include "base/threading/simple_thread.h" |
| 12 | #include "base/time/time.h" |
Kevin McNee | ea3573ff | 2024-08-21 16:38:22 | [diff] [blame] | 13 | #include "build/build_config.h" |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 14 | #include "testing/gtest/include/gtest/gtest.h" |
| 15 | |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 16 | namespace base { |
| 17 | |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 18 | namespace { |
| 19 | |
Peter Kasting | e5a38ed | 2021-10-02 03:06:35 | [diff] [blame] | 20 | constexpr TimeDelta kReceiveTimeout = base::Milliseconds(750); |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 21 | |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 22 | class HangingReceiveThread : public DelegateSimpleThread::Delegate { |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 23 | public: |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 24 | explicit HangingReceiveThread(SyncSocket* socket, bool with_timeout) |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 25 | : socket_(socket), |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 26 | thread_(this, "HangingReceiveThread"), |
| 27 | with_timeout_(with_timeout), |
| 28 | started_event_(WaitableEvent::ResetPolicy::MANUAL, |
| 29 | WaitableEvent::InitialState::NOT_SIGNALED), |
| 30 | done_event_(WaitableEvent::ResetPolicy::MANUAL, |
| 31 | WaitableEvent::InitialState::NOT_SIGNALED) { |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 32 | thread_.Start(); |
| 33 | } |
| 34 | |
David Bienvenu | 5f4d4f0 | 2020-09-27 16:55:03 | [diff] [blame] | 35 | HangingReceiveThread(const HangingReceiveThread&) = delete; |
| 36 | HangingReceiveThread& operator=(const HangingReceiveThread&) = delete; |
Chris Watkins | bb7211c | 2017-11-29 07:16:38 | [diff] [blame] | 37 | ~HangingReceiveThread() override = default; |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 38 | |
dcheng | 5648818 | 2014-10-21 10:54:51 | [diff] [blame] | 39 | void Run() override { |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 40 | int data = 0; |
| 41 | ASSERT_EQ(socket_->Peek(), 0u); |
| 42 | |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 43 | started_event_.Signal(); |
| 44 | |
| 45 | if (with_timeout_) { |
Jan Keitel | 351a86c6 | 2024-08-03 08:25:31 | [diff] [blame] | 46 | ASSERT_EQ(0u, socket_->ReceiveWithTimeout(byte_span_from_ref(data), |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 47 | kReceiveTimeout)); |
| 48 | } else { |
Jan Keitel | 351a86c6 | 2024-08-03 08:25:31 | [diff] [blame] | 49 | ASSERT_EQ(0u, socket_->Receive(byte_span_from_ref(data))); |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 50 | } |
| 51 | |
| 52 | done_event_.Signal(); |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 53 | } |
| 54 | |
Peter Kasting | 134ef9af | 2024-12-28 02:30:09 | [diff] [blame] | 55 | void Stop() { thread_.Join(); } |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 56 | |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 57 | WaitableEvent* started_event() { return &started_event_; } |
| 58 | WaitableEvent* done_event() { return &done_event_; } |
| 59 | |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 60 | private: |
Keishi Hattori | 0e45c02 | 2021-11-27 09:25:52 | [diff] [blame] | 61 | raw_ptr<SyncSocket> socket_; |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 62 | DelegateSimpleThread thread_; |
| 63 | bool with_timeout_; |
| 64 | WaitableEvent started_event_; |
| 65 | WaitableEvent done_event_; |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 66 | }; |
| 67 | |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 68 | // Tests sending data between two SyncSockets. Uses ASSERT() and thus will exit |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 69 | // early upon failure. Callers should use ASSERT_NO_FATAL_FAILURE() if testing |
| 70 | // continues after return. |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 71 | void SendReceivePeek(SyncSocket* socket_a, SyncSocket* socket_b) { |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 72 | int received = 0; |
| 73 | const int kSending = 123; |
avi | 4ec0dff | 2015-11-24 14:26:24 | [diff] [blame] | 74 | static_assert(sizeof(kSending) == sizeof(received), "invalid data size"); |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 75 | |
| 76 | ASSERT_EQ(0u, socket_a->Peek()); |
| 77 | ASSERT_EQ(0u, socket_b->Peek()); |
| 78 | |
| 79 | // Verify |socket_a| can send to |socket_a| and |socket_a| can Receive from |
| 80 | // |socket_a|. |
Jan Keitel | 351a86c6 | 2024-08-03 08:25:31 | [diff] [blame] | 81 | ASSERT_EQ(sizeof(kSending), socket_a->Send(byte_span_from_ref(kSending))); |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 82 | ASSERT_EQ(sizeof(kSending), socket_b->Peek()); |
Jan Keitel | 351a86c6 | 2024-08-03 08:25:31 | [diff] [blame] | 83 | ASSERT_EQ(sizeof(kSending), socket_b->Receive(byte_span_from_ref(received))); |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 84 | ASSERT_EQ(kSending, received); |
| 85 | |
| 86 | ASSERT_EQ(0u, socket_a->Peek()); |
| 87 | ASSERT_EQ(0u, socket_b->Peek()); |
| 88 | |
| 89 | // Now verify the reverse. |
| 90 | received = 0; |
Jan Keitel | 351a86c6 | 2024-08-03 08:25:31 | [diff] [blame] | 91 | ASSERT_EQ(sizeof(kSending), socket_b->Send(byte_span_from_ref(kSending))); |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 92 | ASSERT_EQ(sizeof(kSending), socket_a->Peek()); |
Jan Keitel | 351a86c6 | 2024-08-03 08:25:31 | [diff] [blame] | 93 | ASSERT_EQ(sizeof(kSending), socket_a->Receive(byte_span_from_ref(received))); |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 94 | ASSERT_EQ(kSending, received); |
| 95 | |
| 96 | ASSERT_EQ(0u, socket_a->Peek()); |
| 97 | ASSERT_EQ(0u, socket_b->Peek()); |
| 98 | |
Robert Sesek | 6ab73b02 | 2020-02-13 16:42:39 | [diff] [blame] | 99 | socket_a->Close(); |
| 100 | socket_b->Close(); |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 101 | } |
| 102 | |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 103 | } // namespace |
| 104 | |
| 105 | class SyncSocketTest : public testing::Test { |
| 106 | public: |
| 107 | void SetUp() override { |
| 108 | ASSERT_TRUE(SyncSocket::CreatePair(&socket_a_, &socket_b_)); |
| 109 | } |
| 110 | |
| 111 | protected: |
| 112 | SyncSocket socket_a_; |
| 113 | SyncSocket socket_b_; |
| 114 | }; |
| 115 | |
| 116 | TEST_F(SyncSocketTest, NormalSendReceivePeek) { |
| 117 | SendReceivePeek(&socket_a_, &socket_b_); |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 118 | } |
| 119 | |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 120 | TEST_F(SyncSocketTest, ClonedSendReceivePeek) { |
| 121 | SyncSocket socket_c(socket_a_.Release()); |
| 122 | SyncSocket socket_d(socket_b_.Release()); |
| 123 | SendReceivePeek(&socket_c, &socket_d); |
Nico Weber | ca5f959 | 2019-01-31 14:35:41 | [diff] [blame] | 124 | } |
brucedawson | d350943 | 2015-09-18 18:28:13 | [diff] [blame] | 125 | |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 126 | class CancelableSyncSocketTest : public testing::Test { |
| 127 | public: |
| 128 | void SetUp() override { |
| 129 | ASSERT_TRUE(CancelableSyncSocket::CreatePair(&socket_a_, &socket_b_)); |
| 130 | } |
| 131 | |
| 132 | protected: |
| 133 | CancelableSyncSocket socket_a_; |
| 134 | CancelableSyncSocket socket_b_; |
| 135 | }; |
| 136 | |
| 137 | TEST_F(CancelableSyncSocketTest, NormalSendReceivePeek) { |
| 138 | SendReceivePeek(&socket_a_, &socket_b_); |
| 139 | } |
| 140 | |
| 141 | TEST_F(CancelableSyncSocketTest, ClonedSendReceivePeek) { |
| 142 | CancelableSyncSocket socket_c(socket_a_.Release()); |
| 143 | CancelableSyncSocket socket_d(socket_b_.Release()); |
brucedawson | d350943 | 2015-09-18 18:28:13 | [diff] [blame] | 144 | SendReceivePeek(&socket_c, &socket_d); |
| 145 | } |
| 146 | |
Kevin McNee | ea3573ff | 2024-08-21 16:38:22 | [diff] [blame] | 147 | // TODO(https://crbug.com/361250560): Flaky on mac. |
| 148 | #if BUILDFLAG(IS_MAC) |
| 149 | #define MAYBE_ShutdownCancelsReceive DISABLED_ShutdownCancelsReceive |
| 150 | #else |
| 151 | #define MAYBE_ShutdownCancelsReceive ShutdownCancelsReceive |
| 152 | #endif |
| 153 | TEST_F(CancelableSyncSocketTest, MAYBE_ShutdownCancelsReceive) { |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 154 | HangingReceiveThread thread(&socket_b_, /* with_timeout = */ false); |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 155 | |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 156 | // Wait for the thread to be started. Note that this doesn't guarantee that |
| 157 | // Receive() is called before Shutdown(). |
| 158 | thread.started_event()->Wait(); |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 159 | |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 160 | EXPECT_TRUE(socket_b_.Shutdown()); |
| 161 | EXPECT_TRUE(thread.done_event()->TimedWait(kReceiveTimeout)); |
brucedawson | d350943 | 2015-09-18 18:28:13 | [diff] [blame] | 162 | |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 163 | thread.Stop(); |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 164 | } |
| 165 | |
| 166 | TEST_F(CancelableSyncSocketTest, ShutdownCancelsReceiveWithTimeout) { |
| 167 | HangingReceiveThread thread(&socket_b_, /* with_timeout = */ true); |
| 168 | |
| 169 | // Wait for the thread to be started. Note that this doesn't guarantee that |
| 170 | // Receive() is called before Shutdown(). |
| 171 | thread.started_event()->Wait(); |
| 172 | |
| 173 | EXPECT_TRUE(socket_b_.Shutdown()); |
| 174 | EXPECT_TRUE(thread.done_event()->TimedWait(kReceiveTimeout)); |
| 175 | |
| 176 | thread.Stop(); |
| 177 | } |
| 178 | |
| 179 | TEST_F(CancelableSyncSocketTest, ReceiveAfterShutdown) { |
| 180 | socket_a_.Shutdown(); |
| 181 | int data = 0; |
Jan Keitel | 351a86c6 | 2024-08-03 08:25:31 | [diff] [blame] | 182 | EXPECT_EQ(0u, socket_a_.Receive(byte_span_from_ref(data))); |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 183 | } |
| 184 | |
| 185 | TEST_F(CancelableSyncSocketTest, ReceiveWithTimeoutAfterShutdown) { |
| 186 | socket_a_.Shutdown(); |
| 187 | TimeTicks start = TimeTicks::Now(); |
| 188 | int data = 0; |
Jan Keitel | 351a86c6 | 2024-08-03 08:25:31 | [diff] [blame] | 189 | EXPECT_EQ(0u, socket_a_.ReceiveWithTimeout(byte_span_from_ref(data), |
| 190 | kReceiveTimeout)); |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 191 | |
| 192 | // Ensure the receive didn't just timeout. |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 193 | EXPECT_LT(TimeTicks::Now() - start, kReceiveTimeout); |
[email protected] | 62558f1 | 2013-10-19 22:13:19 | [diff] [blame] | 194 | } |
Sergey Ulanov | e963175 | 2017-08-03 19:24:29 | [diff] [blame] | 195 | |
| 196 | } // namespace base |