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

Commit dfe71558 authored by Ruben Brunk's avatar Ruben Brunk
Browse files

camera2: Fix race with stream deletion during disconnect. 

Bug: 11856804

- Shutdown order in Camera2Client allows a stream to be
  deleted before the corresponding processing thread has
  quit.  This can result in updates being called on the
  processor thread without a valid stream.
parent 84acd489

services/camera/libcameraservice/api1/Camera2Client.cpp

+8 −6
Original line number Diff line number Diff line
@@ -406,12 +406,6 @@ void Camera2Client::disconnect() { 
        l.mParameters.state = Parameters::DISCONNECTED;

    }



    mStreamingProcessor->deletePreviewStream();

    mStreamingProcessor->deleteRecordingStream();

    mJpegProcessor->deleteStream();

    mCallbackProcessor->deleteStream();

    mZslProcessor->deleteStream();



    mStreamingProcessor->requestExit();

    mFrameProcessor->requestExit();

    mCaptureSequencer->requestExit();
@@ -428,6 +422,14 @@ void Camera2Client::disconnect() { 
    mZslProcessorThread->join();

    mCallbackProcessor->join();



    ALOGV("Camera %d: Deleting streams", mCameraId);



    mStreamingProcessor->deletePreviewStream();

    mStreamingProcessor->deleteRecordingStream();

    mJpegProcessor->deleteStream();

    mCallbackProcessor->deleteStream();

    mZslProcessor->deleteStream();



    ALOGV("Camera %d: Disconnecting device", mCameraId);



    mDevice->disconnect();