Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(807)

Side by Side Diff: webrtc/common_audio/signal_processing/dot_product_with_scale.c

Issue 2717123004: Avoid overflow in WebRtcSpl_DotProductWithScale (Closed)
Patch Set: Remove wd4334 Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 /*
2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
3 *
4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree.
9 */
10
11 #include "webrtc/common_audio/signal_processing/include/signal_processing_librar y.h"
12
13 int32_t WebRtcSpl_DotProductWithScale(const int16_t* vector1,
14 const int16_t* vector2,
15 size_t length,
16 int scaling) {
17 int32_t sum = 0;
18 size_t i = 0;
19
20 /* Unroll the loop to improve performance. */
21 for (i = 0; i + 3 < length; i += 4) {
22 sum += (vector1[i + 0] * vector2[i + 0]) >> scaling;
23 sum += (vector1[i + 1] * vector2[i + 1]) >> scaling;
24 sum += (vector1[i + 2] * vector2[i + 2]) >> scaling;
25 sum += (vector1[i + 3] * vector2[i + 3]) >> scaling;
26 }
27 for (; i < length; i++) {
28 sum += (vector1[i] * vector2[i]) >> scaling;
29 }
30
31 return sum;
32 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698