commit aeb5d88dee9998a26fbe3f76790007a8f2fc98f7
author Alex Loiko <aleloi@webrtc.org> Tue Nov 07 13:14:20 2017 +0100
committer Commit Bot <commit-bot@chromium.org> Tue Nov 07 14:47:05 2017 +0000
tree 228cb9cb5bf8ec57dafd36dd428842d43b89a0ec
parent 1c9faeef5fdb28c1138d2ad5336741d845400fca

Replace left shift with equivalent multiplication.

Issue was found by the NetEq fuzzer.


TBR=kwiberg@webrtc.org

Bug: chromium:779380
Change-Id: Ib6a3db78fadc3c7ae11478ca4f695c6d19a2db19
Reviewed-on: https://webrtc-review.googlesource.com/20869
Reviewed-by: Alex Loiko <aleloi@webrtc.org>
Commit-Queue: Alex Loiko <aleloi@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#20586}

common_audio/vad/vad_filterbank.c

diff --git a/common_audio/vad/vad_filterbank.c b/common_audio/vad/vad_filterbank.c
index 5495b50..82cff25 100644
--- a/common_audio/vad/vad_filterbank.c
+++ b/common_audio/vad/vad_filterbank.c
@@ -91,7 +91,7 @@
   size_t i;
   int16_t tmp16 = 0;
   int32_t tmp32 = 0;
-  int32_t state32 = ((int32_t) (*filter_state) << 16);  // Q15
+  int32_t state32 = ((int32_t) (*filter_state) * (1 << 16));  // Q15
 
   for (i = 0; i < data_length; i++) {
     tmp32 = state32 + filter_coefficient * *data_in;