commit ca7fe7ea0257ae5e4318549abbe7c0f79063865e
author Peter Boström <pbos@webrtc.org> Thu Jul 14 07:11:36 2016
committer Peter Boström <pbos@webrtc.org> Thu Jul 14 07:11:47 2016
tree 1045d08a3ce18d0d6935c36566cdc6201a0f0008
parent 921d16afda7ee9ac2b2389b7c5e5aa909984f399

Check for out-of-bounds access on |kIntrpCoef|.

BUG=chromium:600953

Review-Url: https://codereview.webrtc.org/2025493002
Cr-Commit-Position: refs/heads/master@{#13039}
(cherry picked from commit 7bf939c72061b48789304315c857c65a6ad8a1ae)

BUG=chromium:600953
R=tommi@webrtc.org

Review URL: https://codereview.webrtc.org/2144733007 .

Cr-Commit-Position: refs/branch-heads/52@{#10}
Cr-Branched-From: a376e70cf9d0df3c35d53533b454da542661775b-refs/heads/master@{#12798}

webrtc/modules/audio_coding/codecs/isac/fix/source/pitch_filter.c

diff --git a/webrtc/modules/audio_coding/codecs/isac/fix/source/pitch_filter.c b/webrtc/modules/audio_coding/codecs/isac/fix/source/pitch_filter.c
index 13c3899..a8c491c 100644
--- a/webrtc/modules/audio_coding/codecs/isac/fix/source/pitch_filter.c
+++ b/webrtc/modules/audio_coding/codecs/isac/fix/source/pitch_filter.c
@@ -118,7 +118,7 @@
       }
       frcQQ = ((indW32 << 7) + 64 - curLagQ7) >> 4;
 
-      if (frcQQ == PITCH_FRACS) {
+      if (frcQQ >= PITCH_FRACS) {
         frcQQ = 0;
       }
       fracoeffQQ = kIntrpCoef[frcQQ];
@@ -191,7 +191,7 @@
       indW16 = CalcLrIntQ(curLagQ7, 7);
       frcQQ = ((indW16 << 7) + 64 - curLagQ7) >> 4;
 
-      if (frcQQ == PITCH_FRACS) {
+      if (frcQQ >= PITCH_FRACS) {
         frcQQ = 0;
       }
       fracoeffQQ = kIntrpCoef[frcQQ];