Fix an NPE in BatteryService

mCallback is possible to be null. Avoid registering null callbacks.

Bug: 175184519
Test: TH
Change-Id: Ic382d268f40a8ea31aaeedc7a85b6dc9b4e6c210

1 files changed, 2 insertions, 0 deletions

diff --git a/services/core/java/com/android/server/BatteryService.java b/services/core/java/com/android/server/BatteryService.java
index bb4bbd5bc6d4..57dfc8b7e44e 100644
--- a/services/core/java/com/android/server/BatteryService.java
+++ b/services/core/java/com/android/server/BatteryService.java
@@ -1539,6 +1539,8 @@ public final class BatteryService extends SystemService {
                             if (Objects.equals(newService, oldService)) return;
 
                             Slog.i(TAG, "health: new instance registered " + mInstanceName);
+                            // #init() may be called with null callback. Skip null callbacks.
+                            if (mCallback == null) return;
                             mCallback.onRegistration(oldService, newService, mInstanceName);
                         } catch (NoSuchElementException | RemoteException ex) {
                             Slog.e(TAG, "health: Cannot get instance '" + mInstanceName