diff --git a/services/usage/java/com/android/server/usage/UsageStatsService.java b/services/usage/java/com/android/server/usage/UsageStatsService.java index f595c3de104e0..ea40100227c4b 100644 --- a/services/usage/java/com/android/server/usage/UsageStatsService.java +++ b/services/usage/java/com/android/server/usage/UsageStatsService.java @@ -221,6 +221,7 @@ public class UsageStatsService extends SystemService implements final SparseArray mVisibleActivities = new SparseArray(); @GuardedBy("mLock") private final SparseArray mLaunchTimeAlarmQueues = new SparseArray<>(); + @GuardedBy("mUsageEventListeners") // Don't hold the main lock when calling out private final ArraySet mUsageEventListeners = new ArraySet<>(); private final CopyOnWriteArraySet @@ -1168,9 +1169,11 @@ public class UsageStatsService extends SystemService implements service.reportEvent(event); } - final int size = mUsageEventListeners.size(); - for (int i = 0; i < size; ++i) { - mUsageEventListeners.valueAt(i).onUsageEvent(userId, event); + synchronized (mUsageEventListeners) { + final int size = mUsageEventListeners.size(); + for (int i = 0; i < size; ++i) { + mUsageEventListeners.valueAt(i).onUsageEvent(userId, event); + } } } @@ -1661,7 +1664,7 @@ public class UsageStatsService extends SystemService implements * Called via the local interface. */ private void registerListener(@NonNull UsageStatsManagerInternal.UsageEventListener listener) { - synchronized (mLock) { + synchronized (mUsageEventListeners) { mUsageEventListeners.add(listener); } } @@ -1671,7 +1674,7 @@ public class UsageStatsService extends SystemService implements */ private void unregisterListener( @NonNull UsageStatsManagerInternal.UsageEventListener listener) { - synchronized (mLock) { + synchronized (mUsageEventListeners) { mUsageEventListeners.remove(listener); } }