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

Commit 5f7abe2f authored by Austin Borger's avatar Austin Borger
Browse files

Check the device policy for the camera disabled flag before trying to 

open a connection to the camera.

When the camera is disabled from both the device policy and camera
settings, GCA will not receive onError with ERROR_CAMERA_DISABLED. This
is incorrect behavior which is fixed by this patch.

Bug: 230026863, 203063139
Test: Verified camera app closes when both settings are disabled.
Change-Id: I514b38c23a136994de702ea4aacc9be65ab46d70
parent 83302867

camera/aidl/android/hardware/ICameraServiceProxy.aidl

+5 −0
Original line number Diff line number Diff line
@@ -44,4 +44,9 @@ interface ICameraServiceProxy 
     * {@link android.hardware.camera2.CameraMetadata#SCALER_ROTATE_AND_CROP_270}).

     */

    int getRotateAndCropOverride(String packageName, int lensFacing, int userId);



    /**

     * Checks if the camera has been disabled via device policy.

     */

    boolean isCameraDisabled();

}

services/camera/libcameraservice/CameraService.cpp

+7 −0
Original line number Diff line number Diff line
@@ -1688,6 +1688,13 @@ Status CameraService::connectDevice( 
        return STATUS_ERROR(ERROR_ILLEGAL_ARGUMENT, msg.string());

    }



    if (CameraServiceProxyWrapper::isCameraDisabled()) {

        String8 msg =

                String8::format("Camera disabled by device policy");

        ALOGE("%s: %s", __FUNCTION__, msg.string());

        return STATUS_ERROR(ERROR_DISABLED, msg.string());

    }



    // enforce system camera permissions

    if (oomScoreOffset > 0 &&

            !hasPermissionsForSystemCamera(callingPid, CameraThreadState::getCallingUid())) {

services/camera/libcameraservice/utils/CameraServiceProxyWrapper.cpp

+12 −0
Original line number Diff line number Diff line
@@ -260,4 +260,16 @@ void CameraServiceProxyWrapper::logClose(const String8& id, int32_t latencyMs) { 
    sessionStats->onClose(latencyMs);

}



bool CameraServiceProxyWrapper::isCameraDisabled() {

    sp<ICameraServiceProxy> proxyBinder = getCameraServiceProxy();

    if (proxyBinder == nullptr) return true;

    bool ret = false;

    auto status = proxyBinder->isCameraDisabled(&ret);

    if (!status.isOk()) {

        ALOGE("%s: Failed during camera disabled query: %s", __FUNCTION__,

                status.exceptionMessage().c_str());

    }

    return ret;

}



}; // namespace android

services/camera/libcameraservice/utils/CameraServiceProxyWrapper.h

+3 −0
Original line number Diff line number Diff line
@@ -95,6 +95,9 @@ public: 


    // Return the current top activity rotate and crop override.

    static int getRotateAndCropOverride(String16 packageName, int lensFacing, int userId);



    // Detect if the camera is disabled by device policy.

    static bool isCameraDisabled();

};



} // android