commit d399bed72c51c063525f7656e6506efe7743b2cf
author Yunlian Jiang <yunlian@google.com> Thu Dec 05 13:34:51 2013 -0800
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Fri Dec 06 00:52:31 2013 +0000
tree dbf8034f1cf39f7c6e3496cc803315b4ed14021b
parent 33a1baef7a63fd5cc7c639d72188abc4cd8e2ef9

metrics: fix global-buffer-overflow

This fix the problem that the subscript of a global
array overflows in some cases.

BUG=chromium:326284
TEST=Unittest and suite:smoke passes.

Change-Id: I6a19d3bb18a886ee8a29b9a4df5f2b136ce57c5c
Reviewed-on: https://chromium-review.googlesource.com/178973
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Yunlian Jiang <yunlian@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>

metrics/metrics_daemon.cc

diff --git a/metrics/metrics_daemon.cc b/metrics/metrics_daemon.cc
index 1161f6c..1655fae 100644
--- a/metrics/metrics_daemon.cc
+++ b/metrics/metrics_daemon.cc
@@ -1048,7 +1048,7 @@
     // Enough active time has passed.  Do the work, and (if we succeed) see if
     // we need to do more.
     if (MemuseCallbackWork() &&
-        memuse_interval_index_ < arraysize(kMemuseIntervals)) {
+        memuse_interval_index_ < arraysize(kMemuseIntervals) - 1) {
       memuse_interval_index_++;
       ScheduleMemuseCallback(true, 0);
     }