fixed cspot crash when switching with LMS

components/spotify/cspot/bell/include/Queue.h

@@ -103,6 +103,8 @@ namespace bell
                 //delete m_queue.front();
                 m_queue.pop();
             }
+            lk.unlock();
+            m_cv.notify_one();
         }
         /// <summary> Check queue in forced exit state. </summary>
         bool isExit() const

components/spotify/cspot/include/MercuryManager.h

@@ -59,6 +59,7 @@ private:
   std::map<uint64_t, mercuryCallback> callbacks;
   std::mutex reconnectionMutex;
   std::mutex runningMutex;
+  std::mutex stopMutex;
   std::map<std::string, mercuryCallback> subscriptions;
   std::unique_ptr<Session> session;
   std::shared_ptr<LoginBlob> lastAuthBlob; 

components/spotify/cspot/src/AudioChunkManager.cpp

@@ -45,8 +45,8 @@ void AudioChunkManager::close() {
 }
 
 void AudioChunkManager::runTask() {
-    this->isRunning = true;
     std::scoped_lock lock(this->runningMutex);
+	this->isRunning = true;
     while (isRunning) {
         std::pair<std::vector<uint8_t>, bool> audioPair;
         if (this->audioChunkDataQueue.wtpop(audioPair, 100)) {

components/spotify/cspot/src/MercuryManager.cpp

@@ -29,6 +29,8 @@ MercuryManager::MercuryManager(std::unique_ptr<Session> session): bell::Task("me
 
 bool MercuryManager::timeoutHandler()
 {
+    if (!isRunning) return true;
+        
     auto currentTimestamp = timeProvider->getSyncedTimestamp();
 
     if (this->lastRequestTimestamp != -1 && currentTimestamp - this->lastRequestTimestamp > AUDIOCHUNK_TIMEOUT_MS)
@@ -161,7 +163,7 @@ void MercuryManager::runTask()
         }
         catch (const std::runtime_error& e)
         {
-			if (!isRunning) break;
+            if (!isRunning) break;
             // Reconnection required
             this->reconnect();
             this->reconnectedCallback();
@@ -189,10 +191,11 @@ void MercuryManager::runTask()
 }
 
 void MercuryManager::stop() {
+    std::scoped_lock stop(this->stopMutex); 
     CSPOT_LOG(debug, "Stopping mercury manager");
     isRunning = false;
     audioChunkManager->close();
-    std::scoped_lock lock(audioChunkManager->runningMutex, this->runningMutex);
+    std::scoped_lock lock(this->runningMutex);
     CSPOT_LOG(debug, "mercury stopped");
 }
 
@@ -274,6 +277,8 @@ void MercuryManager::handleQueue()
     {
         this->updateQueue();
     }
+    
+    std::scoped_lock lock(this->stopMutex);
 }
 
 uint64_t MercuryManager::execute(MercuryType method, std::string uri, mercuryCallback& callback, mercuryCallback& subscription, mercuryParts& payload)

components/spotify/cspot/src/SpircController.cpp

@@ -55,7 +55,8 @@ void SpircController::disconnect(void) {
     player->cancelCurrentTrack();
     state->setActive(false);
     notify();
-	sendEvent(CSpotEventType::DISC);
+	// Send the event at the end at it might be a last gasp
+	sendEvent(CSpotEventType::DISC);	
 }
 
 void SpircController::playToggle() {