Question

Could I close tcp::socket in different thread from the sync-reading thread?
It looks like:

boost::asio::ip::tcp::socket* tcp_socket;  //blocking mode

thread1:

while(true){
   try{
       std::vector<char> read_buffer(10);
       tcp_socket->read_some( boost::asio::buffer( read_buffer ) );
  }
  catch(boost::system::system_error& e){
  //TODO
  break;
  }
}

thread2:

tcp_socket->shutdown(boost::asio::ip::tcp::socket::shutdown_both);
tcp_socket->close();

I saw the document of tcp::socket. They say the object is thread-unsafty.But the demo code seems working well.
So Is it safe? What about tcp::acceptor? Could I invoke close and accept in multithreading
on same tcp::acceptor?

Answer 1

The documentation states that tcp::socket is not thread safe for shared objects.

Don’t count on it seemingly working as a guarantee that it will always work.

Moreover, closing a socket from another thread, at the socket layer, is not a portable way of getting the blocking thread to unblock.

Here’s what I would suggest:

• Use the asynchronous APIs for asio.
• Protect the socket with a mutex to prevent concurrent access or use an asio strand to serialize the access to it.

Quoting the author on a similar question:

Actually…

In practice, it will probably work on the platforms asio currently
supports**. However, I have deliberately specified the interface such
that it is not thread safe. This is to permit implementations to
store additional state in the socket object without needing explicit
synchronisation.

If you want to run multiple concurrent operations on the same socket,
the safe, portable way is to use the asynchronous operations.

**Unless you make the socket non-blocking.

Cheers, Chris