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

Commit 0b22b6ba authored by Phil Burk's avatar Phil Burk Committed by android-build-merger
Browse files

aaudio: protect against null client am: ef7eaaf8 

am: 1b835f6a

Change-Id: I09ad84bbdf85b3a8c57b8379a4d36f8c48707239
parents 9a3ce7c3 1b835f6a

media/libaaudio/src/binding/IAAudioService.cpp

+9 −2
Original line number Diff line number Diff line
@@ -251,8 +251,15 @@ status_t BnAAudioService::onTransact(uint32_t code, const Parcel& data, 
            CHECK_INTERFACE(IAAudioService, data, reply);

            sp<IAAudioClient> client = interface_cast<IAAudioClient>(

                    data.readStrongBinder());

            // readStrongBinder() can return null

            if (client.get() == nullptr) {

                ALOGE("BnAAudioService::%s(REGISTER_CLIENT) client is NULL!", __func__);

                android_errorWriteLog(0x534e4554, "116230453");

                return DEAD_OBJECT;

            } else {

                registerClient(client);

                return NO_ERROR;

            }

        } break;



        case OPEN_STREAM: {

services/oboeservice/AAudioClientTracker.cpp

+6 −0
Original line number Diff line number Diff line
@@ -67,6 +67,12 @@ aaudio_result_t AAudioClientTracker::registerClient(pid_t pid, 
                                         const sp<IAAudioClient>& client) {

    ALOGV("registerClient(), calling pid = %d, getpid() = %d\n", pid, getpid());



    if (client.get() == nullptr) {

        ALOGE("AAudioClientTracker::%s() client is NULL!", __func__);

        android_errorWriteLog(0x534e4554, "116230453");

        return AAUDIO_ERROR_NULL;

    }



    std::lock_guard<std::mutex> lock(mLock);

    if (mNotificationClients.count(pid) == 0) {

        sp<NotificationClient> notificationClient = new NotificationClient(pid);