Protect _broadcast.socket from simultaneous access.

[dcpomatic.git] / src / lib / encode_server.cc

diff --git a/src/lib/encode_server.cc b/src/lib/encode_server.cc
index 9902c8e84c65bf8d792700f6b2ce5594ea216c4d..d31760c0d975686757df88acc4469baa897956a2 100644 (file)
--- a/src/lib/encode_server.cc
+++ b/src/lib/encode_server.cc
@@ -97,9 +97,15 @@ EncodeServer::~EncodeServer ()
                delete i;
        }
 
-       if (_broadcast.socket) {
-               _broadcast.socket->close ();
+       {
+               boost::mutex::scoped_lock lm (_broadcast.mutex);
+               if (_broadcast.socket) {
+                       _broadcast.socket->close ();
+                       delete _broadcast.socket;
+                       _broadcast.socket = 0;
+               }
        }
+
        _broadcast.io_service.stop ();
        if (_broadcast.thread) {
                /* Ideally this would be a DCPOMATIC_ASSERT(_broadcast.thread->joinable()) but we
@@ -306,6 +312,7 @@ EncodeServer::broadcast_received ()
                }
        }
 
+       boost::mutex::scoped_lock lm (_broadcast.mutex);
        _broadcast.socket->async_receive_from (
                boost::asio::buffer (_broadcast.buffer, sizeof (_broadcast.buffer)),
                _broadcast.send_endpoint, boost::bind (&EncodeServer::broadcast_received, this)