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

Commit e7d45e3f authored by Zhijun He's avatar Zhijun He Committed by Android (Google) Code Review
Browse files

Merge "Camera2: protect the device access" into nyc-dev

parents 0c401511 0effd527

services/camera/libcameraservice/api2/CameraDeviceClient.cpp

+9 −9
Original line number Diff line number Diff line
@@ -310,6 +310,15 @@ binder::Status CameraDeviceClient::endConfigure(bool isConstrainedHighSpeed) { 
    ALOGV("%s: ending configure (%d input stream, %zu output streams)",

            __FUNCTION__, mInputStream.configured ? 1 : 0, mStreamMap.size());



    binder::Status res;

    if (!(res = checkPidStatus(__FUNCTION__)).isOk()) return res;



    Mutex::Autolock icl(mBinderSerializationLock);



    if (!mDevice.get()) {

        return STATUS_ERROR(CameraService::ERROR_DISCONNECTED, "Camera device no longer alive");

    }



    // Sanitize the high speed session against necessary capability bit.

    if (isConstrainedHighSpeed) {

        CameraMetadata staticInfo = mDevice->info();
@@ -332,15 +341,6 @@ binder::Status CameraDeviceClient::endConfigure(bool isConstrainedHighSpeed) { 
        }

    }



    binder::Status res;

    if (!(res = checkPidStatus(__FUNCTION__)).isOk()) return res;



    Mutex::Autolock icl(mBinderSerializationLock);



    if (!mDevice.get()) {

        return STATUS_ERROR(CameraService::ERROR_DISCONNECTED, "Camera device no longer alive");

    }



    status_t err = mDevice->configureStreams(isConstrainedHighSpeed);

    if (err != OK) {

        res = STATUS_ERROR_FMT(CameraService::ERROR_INVALID_OPERATION,