What is the proper way of closing and cleaning up a Socket connection?

Question

I am a bit confused by the cornucopia of related methods on the Socket object that supposedly close and clean up a socket connection. Consider the following:

var socket = new Socket(AddressFamily.InterNetwork, SocketType.Stream, ProtocolType.Tcp); socket.Connect("192.168.1.22", 8333); socket.Send(Encoding.UTF8.GetBytes("hello")); // now for the closing fun socket.Shutdown(SocketShutdown.Both); socket.Disconnect(false); socket.Close(); socket.Dispose();

In addition, this guy says that to really cleanly close a connection, one must execute socket.Shutdown(SocketShudown.Send), then wait for the other side to respond.

What is the correct way to close, then clean up a socket connection?

That guy is half right. I think it depends on the protocol between the two. If you are expecting to still be able to receive for some time then yes, you should tell the otherside of your intent to shutdown and say that you will stop sending but can still receive. — uriDium
– uriDium, Commented Mar 12, 2013 at 7:20
"that guy" is mostly correct, as I have modeled my server code similarly. But I believe the half-close-then-wait approach is only necessary for asynchronous (non-blocking) sockets. I believe in the case of non-blocking sockets, you call "Send" followed immediately by Close, there are some edge conditions in which data loss can occur if there are packets that need to be retransmitted. — selbie
– selbie, Commented Mar 12, 2013 at 8:40
I used a hybrid approach with my socket server. The server will half-close the connection first and wait for recv() to return 0 or error to indicate the client is done. However, I have a timer that periodically goes off every 45 seconds to sweep and close any half-open connections that have been lingering around since the last sweep. — selbie
– selbie, Commented Mar 12, 2013 at 8:42

Kelly Elton · Accepted Answer · 2020-06-04 21:46:46Z

Closing socket closes the connection, and Close is a wrapper-method around Dispose, so generally

socket.Shutdown(SocketShutdown.Both); socket.Close();

should be enough. Some might argue, that Close implementation might change one day (so it no longer calls Dispose), and you should call Dispose manually after calling Close, but i doubt thats gonna happen, personally :)

Alternatively, consider using using (yeh):

using (var socket = new Socket(...)) { .... socket.Shutdown(SocketShutdown.Both); socket.Close(); }

Kelly Elton · Accepted Answer · 2020-06-04 21:56:53Z

According to MSDN, for connection-oriented sockets, if you call Connect, you should call Close. This applies to BeginConnect and ConnectAsync as well.

using (var socket = new Socket(...)) { socket.Connect(host); ... socket.Shutdown(SocketShutdown.Both); socket.Close(); }

Collectives™ on Stack Overflow

What is the proper way of closing and cleaning up a Socket connection?

2 Answers 2

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Linked

Related