Kernel: Delay moving accepted sockets to SetupState::Completed a bit

Make sure we don't move accepted sockets to the Completed setup state
until we've actually constructed a FileDescription for them.

This is important, since this state transition will trigger connect()
to unblock on the client side, and the client may try writing to the
socket right away.

This makes DNS lookups way more reliable since we don't just fail to
write() right after connect()ing to LookupServer sometimes. :^)

1 files changed, 0 insertions, 1 deletions

diff --git a/Kernel/Net/Socket.cpp b/Kernel/Net/Socket.cpp
index 91d5bc31cb..0749d0ee2e 100644
--- a/Kernel/Net/Socket.cpp
+++ b/Kernel/Net/Socket.cpp
@@ -65,7 +65,6 @@ RefPtr<Socket> Socket::accept()
     auto client = m_pending.take_first();
     ASSERT(!client->is_connected());
     client->m_acceptor_pid = m_origin_pid;
-    client->set_setup_state(SetupState::Completed);
     client->m_connected = true;
     client->m_role = Role::Accepted;
     return client;