diff options
| author | 2021-07-29 11:30:37 -0700 | |
|---|---|---|
| committer | 2021-07-29 11:30:37 -0700 | |
| commit | 84fcc8963ed5f514dceb02272e5f577069d0b682 (patch) | |
| tree | 52f943b38e7c98537ad025ea0fe3742b38e3b371 | |
| parent | 782b4e9e080bd89badcdfabd37e53740f2f0f2dd (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.java | 6 |
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(); |