Fixed usage of running_ that did not include a lock.

Bryan Dallas
1 parent 91e3e06c
Showing 1 changed file with 5 additions and 3 deletions
include/redox/client.hpp
@@ -393,9 +393,11 @@ template &lt;class ReplyT&gt;
 Command<ReplyT> &Redox::createCommand(const std::vector<std::string> &cmd,
                                       const std::function<void(Command<ReplyT> &)> &callback,
                                       double repeat, double after, bool free_memory) {
-
-  if (!running_) {
-    throw std::runtime_error("[ERROR] Need to connect Redox before running commands!");
+  {
+    unique_lock<mutex> ul(running_lock_);
+    if (!running_) {
+      throw std::runtime_error("[ERROR] Need to connect Redox before running commands!");
+    }
   }
   commands_created_ += 1;