On Wed, 2016-02-17 at 19:47 +0530, arun at accosted.net wrote: > @@ -363,7 +367,8 @@ void pa_webrtc_ec_play(pa_echo_canceller *ec, const uint8_t *play) { > Â Â Â Â Â pa_assert(play_frame.samples_per_channel_ <= webrtc::AudioFrame::kMaxDataSizeSamples); > Â Â Â Â Â memcpy(play_frame.data_, play, ec->params.webrtc.blocksize * pa_frame_size(ss)); > Â > -Â Â Â Â apm->ProcessReverseStream(&play_frame); > +Â Â Â Â if (apm->ProcessReverseStream(&play_frame) != webrtc::AudioProcessing::kNoError) > +Â Â Â Â Â Â Â Â pa_log("Failed to process playback stream"); This should have been changed to an assertion too. I have it changed in my local tree. -- Tanu