summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Michael Wachenschwanz <mwachens@google.com> 2021-07-29 11:30:37 -0700
committer Michael Wachenschwanz <mwachens@google.com> 2021-07-29 11:30:37 -0700
commit84fcc8963ed5f514dceb02272e5f577069d0b682 (patch)
tree52f943b38e7c98537ad025ea0fe3742b38e3b371
parent782b4e9e080bd89badcdfabd37e53740f2f0f2dd (diff)
Log and skip noteEvent calls with null name
Bug: 194733136 Test: builds + boots Change-Id: I909a4656d3d3aa502466d96ffa804573de258587
-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();