Fix buffer overflow bug

When 2 transmitters sending at the same time, a buffer overflow can happen. So, a check is added to flush the incoming samples in that case. Nice side effect is, that you can hear 2 transmitters when they are sending at the same time (distorted).

Author: FedericoBusero

lib/audio_output/src/OutputBuffer.h

@@ -53,13 +53,17 @@ class OutputBuffer
   void add_samples(const uint8_t *samples, int count)
   {
     xSemaphoreTake(m_semaphore, portMAX_DELAY);
-    // copy the samples into the buffer wrapping around as needed
-    for (int i = 0; i < count; i++)
+    // check if there is still room in the buffer
+    if (m_available_samples + count <= m_buffer_size)
     {
-      m_buffer[m_write_head] = samples[i];
-      m_write_head = (m_write_head + 1) % m_buffer_size;
+      // copy the samples into the buffer wrapping around as needed
+      for (int i = 0; i < count; i++)
+      {
+        m_buffer[m_write_head] = samples[i];
+        m_write_head = (m_write_head + 1) % m_buffer_size;
+      }
+      m_available_samples += count;
     }
-    m_available_samples += count;
     xSemaphoreGive(m_semaphore);
   }