commit d8bde878df9709b25ed4b44b5813c96f12811c6c
author slki.shin <slki.shin@samsung.com> Thu Jul 25 15:14:43 2019 +0900
committer Luca Stefani <luca.stefani.ge1@gmail.com> Sun Aug 23 17:11:59 2020 +0200
tree 21de52637f48416817456d24c12d0c37d1503329
parent c174a8b7ef76c475fdce19ae3f0518978d9497f0

Fix null pointer exception on AvrcpTargetService

This fixes null pointer exception by simply adding null pointer checker.
Problem occurs when sendMediaUpdate() is called just after Avrcp
Service is stop()ed.

Error Message: java.lang.NullPointerException: Attempt to invoke virtual method ’void com.android.bluetooth.avrcp.AvrcpNativeInterface.sendMediaUpdate(boolean, boolean, boolean)’ on a null object reference
Test: Once issue

Change-Id: Id42e48faa28ce6c1ec6d4148705d3342261aa6ca
Signed-off-by: slki.shin <slki.shin@samsung.com>

src/com/android/bluetooth/avrcp/AvrcpTargetService.java

diff --git a/src/com/android/bluetooth/avrcp/AvrcpTargetService.java b/src/com/android/bluetooth/avrcp/AvrcpTargetService.java
index bdc930b..f11622f 100644
--- a/src/com/android/bluetooth/avrcp/AvrcpTargetService.java
+++ b/src/com/android/bluetooth/avrcp/AvrcpTargetService.java
@@ -70,6 +70,8 @@
             MediaPlayerList.FolderUpdateCallback {
         @Override
         public void run(MediaData data) {
+            if (mNativeInterface == null) return;
+
             boolean metadata = !Objects.equals(mCurrentData.metadata, data.metadata);
             boolean state = !MediaPlayerWrapper.playstateEquals(mCurrentData.state, data.state);
             boolean queue = !Objects.equals(mCurrentData.queue, data.queue);
@@ -86,6 +88,8 @@
         @Override
         public void run(boolean availablePlayers, boolean addressedPlayers,
                 boolean uids) {
+            if (mNativeInterface == null) return;
+
             mNativeInterface.sendFolderUpdate(availablePlayers, addressedPlayers, uids);
         }
     }