main/binding: do not allow concurrent threads in App
authorMarcus Fritzsch <marcus_fritzsch@mentor.com>
Thu, 31 Aug 2017 15:17:42 +0000 (17:17 +0200)
committerMarcus Fritzsch <marcus_fritzsch@mentor.com>
Mon, 4 Sep 2017 14:54:02 +0000 (16:54 +0200)
Signed-off-by: Marcus Fritzsch <marcus_fritzsch@mentor.com>
generate-binding-glue.py
src/main.cpp

index a489b63..82f71ae 100644 (file)
@@ -49,6 +49,7 @@ def emit_func_impl(api, f):
 
 def emit_func(api, f):
     p('void %(impl_name)s(afb_req req) noexcept {' % f)
+    p('   std::lock_guard<std::mutex> guard(binding_m);')
     p('   if (g_afb_instance == nullptr) {',
       '      afb_req_fail(req, "failed", "Binding not initialized, did the compositor die?");',
       '      return;',
index a768347..47e3335 100644 (file)
@@ -20,6 +20,8 @@
 #include "wayland.hpp"
 
 #include <algorithm>
+#include <mutex>
+
 #include <json.h>
 
 extern "C" {
@@ -28,6 +30,8 @@ extern "C" {
 }
 
 namespace {
+std::mutex binding_m;
+
 struct afb_instance {
    std::unique_ptr<wl::display> display;
    wm::App app;
@@ -49,6 +53,8 @@ int afb_instance::init() {
 
 int display_event_callback(sd_event_source *evs, int /*fd*/, uint32_t events,
                            void * /*data*/) {
+   std::lock_guard<std::mutex> guard(binding_m);
+
    if ((events & EPOLLHUP) != 0) {
       logerror("The compositor hung up, dying now.");
       delete g_afb_instance;