From 221f4f232ae79b8123e7ce28d26a873e1ba9f9dc Mon Sep 17 00:00:00 2001 From: Andreas Sandberg Date: Wed, 9 Apr 2014 16:01:43 +0200 Subject: dev: Protect PollEvent processing when running in parallel mode The calling thread is undefined when the PollQueue services events. This implies that PollEvents need to handle the case where they are processed from a different thread than the thread that created the event. This changeset adds temporary event queue migrations to the VNC server, the ethernet tap device, and the terminal to protect them from inter-thread calls. --- src/base/vnc/vncserver.cc | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'src/base') diff --git a/src/base/vnc/vncserver.cc b/src/base/vnc/vncserver.cc index debf82014..9ca575915 100644 --- a/src/base/vnc/vncserver.cc +++ b/src/base/vnc/vncserver.cc @@ -174,6 +174,11 @@ VncServer::listen(int port) void VncServer::accept() { + // As a consequence of being called from the PollQueue, we might + // have been called from a different thread. Migrate to "our" + // thread. + EventQueue::ScopedMigration migrate(eventQueue()); + if (!listener.islistening()) panic("%s: cannot accept a connection if not listening!", name()); -- cgit v1.2.3