summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Christopher Tate <ctate@google.com> 2013-09-25 01:20:38 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2013-09-25 01:20:38 +0000
commit044fd573ce58142309f9fc503cf5d52af447d082 (patch)
tree404ccf1cb9c68875b5d501e0ec6aa40ad4c2f324
parent8488f9fa2e38e75e7ecbaac7d776900f8f1e5917 (diff)
parent6578ad17cab67767f0770b88990145d57f705d5f (diff)
Merge "Don't crash when no alarms are scheduled" into klp-dev
-rw-r--r--services/java/com/android/server/AlarmManagerService.java5
1 files changed, 2 insertions, 3 deletions
diff --git a/services/java/com/android/server/AlarmManagerService.java b/services/java/com/android/server/AlarmManagerService.java
index bb6c4e6ba3a5..04d6332d34c9 100644
--- a/services/java/com/android/server/AlarmManagerService.java
+++ b/services/java/com/android/server/AlarmManagerService.java
@@ -1016,12 +1016,11 @@ class AlarmManagerService extends IAlarmManager.Stub {
private native int setKernelTimezone(int fd, int minuteswest);
private void triggerAlarmsLocked(ArrayList<Alarm> triggerList, long nowELAPSED, long nowRTC) {
- Batch batch;
-
// batches are temporally sorted, so we need only pull from the
// start of the list until we either empty it or hit a batch
// that is not yet deliverable
- while ((batch = mAlarmBatches.get(0)) != null) {
+ while (mAlarmBatches.size() > 0) {
+ Batch batch = mAlarmBatches.get(0);
if (batch.start > nowELAPSED) {
// Everything else is scheduled for the future
break;