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

Skip to content
Commit 9bb7d945 authored by Atneya Nair's avatar Atneya Nair Committed by Mohammed Althaf T
Browse files

[RESTRICT AUTOMERGE] [appops] Preflight skip datasource validation

The forDataDelivery logic always skips checking the first attribution in
the chain from datasources, either via singleReceiverFromDataSource, or
internal in startProxyOp, since skipProxy=true is passed for non-trivial
chains.

Make the preflight check consistent with this behavior, by also skipping
a checkOp on the first entry in this case. This avoids cases where
the preflight fails when the delivery would succeed, which should never
happen. This is implicitly relied on by audioserver, as it happens to
fail checkOp due to not having a valid Uid/PackageState.

Test: manual: start and stop recording with toggle restriction
Test: atest CtsMediaAudioPermissionTestCases
Test: atest RuntimePermissionsAppOpTrackingTest
Test: atest SensorPrivacyMicrophoneTest
Bug: 293603271
Bug: 325912429
Flag: EXEMPT bugfix
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:b33e9e68980c2ddd370e46f38406b4b9459823bd)
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:468f7a7fdca36223fcbabaa9be019a231c200401)
Merged-In: I509e7f8da501f5e32d336adb412662e078eab500
Change-Id: I509e7f8da501f5e32d336adb412662e078eab500
parent e5ef4df5
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment