summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Michael Wachenschwanz <mwachens@google.com> 2021-07-30 00:49:10 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2021-07-30 00:49:10 +0000
commitb4db3d25056ccb33b2ff787a2020d98344bb2b79 (patch)
treea3040422bad258361f1feef20058c9129563eefc
parenta1089252cd2169f926ef429168727bee8a6318eb (diff)
parent0e1ba5a342d40b8436841f3d0319ee40be3ee3f5 (diff)
Merge "Log and skip noteEvent calls with null name" into sc-dev am: 0e1ba5a342
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15419007 Change-Id: Ie4baae18cc7495ae337142c99cede75f957bfdd4
-rw-r--r--services/core/java/com/android/server/am/BatteryStatsService.java6
1 files changed, 6 insertions, 0 deletions
diff --git a/services/core/java/com/android/server/am/BatteryStatsService.java b/services/core/java/com/android/server/am/BatteryStatsService.java
index 9f41c8ba9626..ae14ca7b66bd 100644
--- a/services/core/java/com/android/server/am/BatteryStatsService.java
+++ b/services/core/java/com/android/server/am/BatteryStatsService.java
@@ -841,6 +841,12 @@ public final class BatteryStatsService extends IBatteryStats.Stub
public void noteEvent(final int code, final String name, final int uid) {
enforceCallingPermission();
+ if (name == null) {
+ // TODO(b/194733136): Replace with an IllegalArgumentException throw.
+ Slog.wtfStack(TAG, "noteEvent called with null name. code = " + code);
+ return;
+ }
+
synchronized (mLock) {
final long elapsedRealtime = SystemClock.elapsedRealtime();
final long uptime = SystemClock.uptimeMillis();