Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6e1a023b authored by Kyunglyul Hyun's avatar Kyunglyul Hyun Committed by Automerger Merge Worker
Browse files

Merge "Fix potential NPE in SapService" into main am: 425ff7b0 

Original change: https://android-review.googlesource.com/c/platform/packages/modules/Bluetooth/+/3124916



Change-Id: Ie7c3654076c00d3a23d22b54b816c88b634ea673
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 76df9596 425ff7b0

android/app/src/com/android/bluetooth/sap/SapService.java

+6 −4
Original line number Diff line number Diff line
@@ -907,12 +907,14 @@ public class SapService extends ProfileService implements AdapterService.Bluetoo 


        @RequiresPermission(android.Manifest.permission.BLUETOOTH_CONNECT)

        private SapService getService(AttributionSource source) {

            if (!Utils.checkServiceAvailable(mService, TAG)

                    || !Utils.checkCallerIsSystemOrActiveOrManagedUser(mService, TAG)

                    || !Utils.checkConnectPermissionForDataDelivery(mService, source, TAG)) {

            SapService sapService = mService;



            if (!Utils.checkServiceAvailable(sapService, TAG)

                    || !Utils.checkCallerIsSystemOrActiveOrManagedUser(sapService, TAG)

                    || !Utils.checkConnectPermissionForDataDelivery(sapService, source, TAG)) {

                return null;

            }

            return mService;

            return sapService;

        }



        SapBinder(SapService service) {

android/app/tests/unit/src/com/android/bluetooth/sap/SapServiceTest.java

+6 −0
Original line number Diff line number Diff line
@@ -25,6 +25,7 @@ import android.bluetooth.BluetoothDevice; 
import android.bluetooth.BluetoothProfile;

import android.content.Context;

import android.content.Intent;

import android.os.Looper;



import androidx.test.filters.MediumTest;

import androidx.test.platform.app.InstrumentationRegistry;
@@ -60,6 +61,11 @@ public class SapServiceTest { 
    public void setUp() throws Exception {

        mTargetContext = InstrumentationRegistry.getInstrumentation().getTargetContext();

        TestUtils.setAdapterService(mAdapterService);



        if (Looper.myLooper() == null) {

            Looper.prepare();

        }



        mService = new SapService(mTargetContext);

        mService.start();

        mService.setAvailable(true);