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

Skip to content
Commit a95cfc3e authored by Hui Yu's avatar Hui Yu Committed by Android Build Coastguard Worker
Browse files

Revert "BG-FGS-start while-in-use permission restriction improve..."

Revert "Add test cases for background startForeground() improvement."

Revert submission 15081873-cherrypick-security_backport_183147114-an8dvy98fv

Reason for revert: https://b.corp.google.com/issues/197066403#comment15
After the app calls Service.startForeground() to put the app into foreground service mode, the service has mAllowWhileInUsePermissionInFgs set, if it is true, the app/service can access location/camera/micophone.

Some apps may call Service.startForeground() again (for any reason, could be a app's unintentional redundant call, could be that app want to update the notification of the foreground service), but at this moment the app has went into background mode, although the foreground service is still running. The second or later Service.startForeground() call may set mAllowWhileInUsePermissionInFgs to false and the app may lose location/camera/micophone access. This is incorrect because the foreground service is still running and we expect location/camera/micophone access to continue.

The Samsung Voice Recorder app has run into this situation.

Reverted Changes:
I0aca484e5:BG-FGS-start while-in-use permission restriction i...
I4988dbba1:Add test cases for background startForeground() im...
Bug: 183147114
Bug: 197066403
Change-Id: Iad32e4391fa15bc252e50f9b858fe2e5225edb19
Merged-In: Idc88f274c7a323d175d65bb47eca041772ae9bb7
(cherry picked from commit c83a9497)
parent 197a0744
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment