Properly adjust the number of frames left to be processed by the resampler.

After calling the resampler, we would reduce the number of frames in the input buffer correctly by the number of frames used, but would always fully clear the input buffer after.
Resulting with a input_buffer and its frame counters to be potentially out of sync

1 files changed, 4 insertions, 2 deletions

diff --git a/src/cubeb_audiounit.cpp b/src/cubeb_audiounit.cpp
index 007e411..e020e5c 100644
--- a/src/cubeb_audiounit.cpp
+++ b/src/cubeb_audiounit.cpp
@@ -621,6 +621,7 @@ audiounit_output_callback(void * user_ptr,
     input_frames = stm->input_linear_buffer->length() / stm->input_desc.mChannelsPerFrame;
     // Number of input frames pushed inside resampler.
     input_frames_before_fill = input_frames;
+    assert(input_frames == stm->available_input_frames);
   }
 
   /* Call user callback through resampler. */
@@ -634,10 +635,11 @@ audiounit_output_callback(void * user_ptr,
                                         output_frames);
 
   if (input_buffer) {
+    // Pop from the buffer the frames used by the the resampler.
+    stm->input_linear_buffer->pop(input_frames * stm->input_desc.mChannelsPerFrame);
     // Decrease counter by the number of frames used by resampler
+    assert(stm->available_input_frames >= input_frames);
     stm->available_input_frames -= input_frames;
-    // Pop from the buffer the frames pushed to the resampler.
-    stm->input_linear_buffer->pop(input_frames_before_fill * stm->input_desc.mChannelsPerFrame);
   }
 
   if (outframes < 0 || outframes > output_frames) {