From abcf112ae0e040f94953e20bec86574f18d5a508 Mon Sep 17 00:00:00 2001 From: henrika Date: Thu, 13 Jul 2017 04:42:50 -0700 Subject: [PATCH] Adds sanity check for sample rate on iOS BUG=b/62909493 Review-Url: https://codereview.webrtc.org/2978913002 Cr-Commit-Position: refs/heads/master@{#19000} --- webrtc/modules/audio_device/ios/audio_device_ios.mm | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/webrtc/modules/audio_device/ios/audio_device_ios.mm b/webrtc/modules/audio_device/ios/audio_device_ios.mm index 3dcd48f2dc..1d59aff467 100644 --- a/webrtc/modules/audio_device/ios/audio_device_ios.mm +++ b/webrtc/modules/audio_device/ios/audio_device_ios.mm @@ -623,6 +623,12 @@ void AudioDeviceIOS::HandleSampleRateChange(float sample_rate) { return; } + // Extra sanity check to ensure that the new sample rate is valid. + if (session_sample_rate <= 0.0) { + RTCLogError(@"Sample rate is invalid: %f", session_sample_rate); + return; + } + // We need to adjust our format and buffer sizes. // The stream format is about to be changed and it requires that we first // stop and uninitialize the audio unit to deallocate its resources.