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

Commit 79e762c7 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Check if sensor is accessible on flush" into rvc-dev am: 9aacd0cf 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/12091816

Change-Id: I0261b41b2c93ea76c0b9961fd1d5b92b65be6b2c
parents 2a09f624 9aacd0cf

services/sensorservice/SensorEventConnection.cpp

+10 −5
Original line number Diff line number Diff line
@@ -272,12 +272,17 @@ void SensorService::SensorEventConnection::updateLooperRegistrationLocked( 
    }

}



void SensorService::SensorEventConnection::incrementPendingFlushCount(int32_t handle) {

bool SensorService::SensorEventConnection::incrementPendingFlushCountIfHasAccess(int32_t handle) {

    if (hasSensorAccess()) {

        Mutex::Autolock _l(mConnectionLock);

        if (mSensorInfo.count(handle) > 0) {

            FlushInfo& flushInfo = mSensorInfo[handle];

            flushInfo.mPendingFlushEventsToSend++;

        }

        return true;

    } else {

        return false;

    }

}



status_t SensorService::SensorEventConnection::sendEvents(

services/sensorservice/SensorEventConnection.h

+3 −2
Original line number Diff line number Diff line
@@ -116,8 +116,9 @@ private: 
    // for writing send the data from the cache.

    virtual int handleEvent(int fd, int events, void* data);



    // Increment mPendingFlushEventsToSend for the given sensor handle.

    void incrementPendingFlushCount(int32_t handle);

    // Increment mPendingFlushEventsToSend for the given handle if the connection has sensor access.

    // Returns true if this connection does have sensor access.

    bool incrementPendingFlushCountIfHasAccess(int32_t handle);



    // Add or remove the file descriptor associated with the BitTube to the looper. If mDead is set

    // to true or there are no more sensors for this connection, the file descriptor is removed if

services/sensorservice/SensorService.cpp

+4 −1
Original line number Diff line number Diff line
@@ -1774,7 +1774,10 @@ status_t SensorService::flushSensor(const sp<SensorEventConnection>& connection, 
        if (halVersion <= SENSORS_DEVICE_API_VERSION_1_0 || isVirtualSensor(handle)) {

            // For older devices just increment pending flush count which will send a trivial

            // flush complete event.

            connection->incrementPendingFlushCount(handle);

            if (!connection->incrementPendingFlushCountIfHasAccess(handle)) {

                ALOGE("flush called on an inaccessible sensor");

                err = INVALID_OPERATION;

            }

        } else {

            if (!canAccessSensor(sensor->getSensor(), "Tried flushing", opPackageName)) {

                err = INVALID_OPERATION;