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

Commit 870f4e7e authored by Siarhei Vishniakou's avatar Siarhei Vishniakou
Browse files

InputDispatcher: check consistency of filtered injected events -- try 3 

This reverts commit d604df19.

Reason for revert: fixed flags in b/309829647

Change-Id: I191eb04c95d1316cb505ba0861669ad004d02d67
parent ae427688

services/inputflinger/dispatcher/InputDispatcher.cpp

+26 −0
Original line number Diff line number Diff line
@@ -4732,6 +4732,7 @@ void InputDispatcher::notifySwitch(const NotifySwitchArgs& args) { 
}



void InputDispatcher::notifyDeviceReset(const NotifyDeviceResetArgs& args) {

    // TODO(b/308677868) Remove device reset from the InputListener interface

    if (debugInboundEventDetails()) {

        ALOGD("notifyDeviceReset - eventTime=%" PRId64 ", deviceId=%d", args.eventTime,

              args.deviceId);
@@ -4883,6 +4884,30 @@ InputEventInjectionResult InputDispatcher::injectInputEvent(const InputEvent* ev 
            }



            mLock.lock();



            if (policyFlags & POLICY_FLAG_FILTERED) {

                // The events from InputFilter impersonate real hardware devices. Check these

                // events for consistency and print an error. An inconsistent event sent from

                // InputFilter could cause a crash in the later stages of dispatching pipeline.

                auto [it, _] =

                        mInputFilterVerifiersByDisplay

                                .try_emplace(displayId,

                                             StringPrintf("Injection on %" PRId32, displayId));

                InputVerifier& verifier = it->second;



                Result<void> result =

                        verifier.processMovement(resolvedDeviceId, motionEvent.getSource(),

                                                 motionEvent.getAction(),

                                                 motionEvent.getPointerCount(),

                                                 motionEvent.getPointerProperties(),

                                                 motionEvent.getSamplePointerCoords(), flags);

                if (!result.ok()) {

                    logDispatchStateLocked();

                    LOG(ERROR) << "Inconsistent event: " << motionEvent

                               << ", reason: " << result.error();

                }

            }



            const nsecs_t* sampleEventTimes = motionEvent.getSampleEventTimes();

            const size_t pointerCount = motionEvent.getPointerCount();

            const std::vector<PointerProperties>
@@ -6962,6 +6987,7 @@ void InputDispatcher::displayRemoved(ui::LogicalDisplayId displayId) { 
        // Remove the associated touch mode state.

        mTouchModePerDisplay.erase(displayId);

        mVerifiersByDisplay.erase(displayId);

        mInputFilterVerifiersByDisplay.erase(displayId);

    } // release lock



    // Wake up poll loop since it may need to make new input dispatching choices.

services/inputflinger/dispatcher/InputDispatcher.h

+3 −1
Original line number Diff line number Diff line
@@ -301,7 +301,9 @@ private: 
    void transformMotionEntryForInjectionLocked(MotionEntry&,

                                                const ui::Transform& injectedTransform) const

            REQUIRES(mLock);



    // Per-display correction of injected events

    std::map<android::ui::LogicalDisplayId, InputVerifier> mInputFilterVerifiersByDisplay

            GUARDED_BY(mLock);

    std::condition_variable mInjectionSyncFinished;

    void incrementPendingForegroundDispatches(const EventEntry& entry);

    void decrementPendingForegroundDispatches(const EventEntry& entry);