In case we return false here, we should still unlock the

mutex first
2016-10-23 05:53:47 +02:00 · 2016-10-23 05:53:47 +02:00 · 1cbc847133
parent e1ce5209a0
commit 1cbc847133
1 changed files with 5 additions and 0 deletions
--- a/runloop.c
+++ b/runloop.c
@ -501,7 +501,12 @@ bool runloop_ctl(enum runloop_ctl_state state, void *data)
         {
            const char **ret = (const char**)data;
            if (!ret)
+            {
+#ifdef HAVE_THREADS
+               slock_unlock(_runloop_msg_queue_lock);
+#endif
               return false;
+            }
            *ret = msg_queue_pull(runloop_msg_queue);
         }
 #ifdef HAVE_THREADS