summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author alexbianchi <alexbianchi@google.com> 2022-08-15 19:26:01 +0000
committer Alex Bianchi <alexbianchi@google.com> 2022-08-15 19:44:30 +0000
commit56196aadd6d666d93a7233aadc3316ad74741ea4 (patch)
tree97f064021b4e55c47923c47f26edaaea2be7acc0
parente2d779319a8c9b7b6156dbcc698ad082211c8d25 (diff)
Fix Flexibility Tracker Dump Function
Fixed out of bounds error caused by using the wrong size to loop through an ArraySet. Bug: 242181094 Test: Manual. Run `adb shell dumpsys jobscheduler` Test: atest frameworks/base/services/tests/mockingservicestests/src/com/android/server/job Test: atest frameworks/base/services/tests/servicestests/src/com/android/server/job Test: atest CtsJobSchedulerTestCases Change-Id: I7ca5340f8f22b369f4a9d0807c8184f2f7018e85
-rw-r--r--apex/jobscheduler/service/java/com/android/server/job/controllers/FlexibilityController.java2
1 files changed, 1 insertions, 1 deletions
diff --git a/apex/jobscheduler/service/java/com/android/server/job/controllers/FlexibilityController.java b/apex/jobscheduler/service/java/com/android/server/job/controllers/FlexibilityController.java
index e69cbedb955e..921ef5e58909 100644
--- a/apex/jobscheduler/service/java/com/android/server/job/controllers/FlexibilityController.java
+++ b/apex/jobscheduler/service/java/com/android/server/job/controllers/FlexibilityController.java
@@ -499,7 +499,7 @@ public final class FlexibilityController extends StateController {
public void dump(IndentingPrintWriter pw, Predicate<JobStatus> predicate) {
for (int i = 0; i < mTrackedJobs.size(); i++) {
ArraySet<JobStatus> jobs = mTrackedJobs.get(i);
- for (int j = 0; j < mTrackedJobs.size(); j++) {
+ for (int j = 0; j < jobs.size(); j++) {
final JobStatus js = jobs.valueAt(j);
if (!predicate.test(js)) {
continue;