fix: Acquire lock when stopping modules

include/modules/meta.hpp

@@ -210,13 +210,16 @@ namespace modules {
     }
 
     void stop() {
-      if (!enabled())
+      std::lock_guard<threading_util::spin_lock> lck(this->update_lock);
-        return;
+      {
-      m_log.trace("%s: Stop", name());
+        if (!enabled())
-      enable(false);
+          return;
-      wakeup();
+        m_log.trace("%s: Stop", name());
-      if (!on_stop.empty())
+        enable(false);
-        on_stop.emit(name());
+        wakeup();
+        if (!on_stop.empty())
+          on_stop.emit(name());
+      }
     }
 
     void refresh() {

include/modules/network.hpp

@@ -17,14 +17,6 @@ namespace modules {
    public:
     using timer_module::timer_module;
 
-    ~network_module() {
-      std::lock_guard<threading_util::spin_lock> lck(this->update_lock);
-      {
-        m_wireless.reset();
-        m_wired.reset();
-      }
-    }
-
     void setup() {
       // Load configuration values
       REQ_CONFIG_VALUE(name(), m_interface, "interface");

include/modules/volume.hpp

@@ -108,19 +108,6 @@ namespace modules {
       // }}}
     }
 
-    void stop() {
-      // Deconstruct all mixers before putting the module in its stopped state {{{
-
-      std::lock_guard<threading_util::spin_lock> lck(this->update_lock);
-      m_mixers[mixer::MASTER].reset();
-      m_mixers[mixer::SPEAKER].reset();
-      m_mixers[mixer::HEADPHONE].reset();
-      m_controls[control::HEADPHONE].reset();
-      event_module::stop();
-
-      // }}}
-    }
-
     bool has_event() {
       // Poll for mixer and control events {{{