tree cdcb82a6015a862eabd1030df55d639627432c8b
parent 82150f3f7bff0e80b3a3705aee2f37cde9a5e146
author Philip P. Moltmann <moltmann@google.com> 1561067512 -0700
committer Philip P. Moltmann <moltmann@google.com> 1561073573 -0700

Prevent loop in PermissionPolicyService

The AppOpsModeChangeWatchers call back even if the mode was set to
already set mode. PermissionPolicyService listens to the same app-ops it
changes. I.e. it used to change the mode to default, which then causes a
call-back which made set it to default again, etc...

This exhausts a thread until the app-op is finally set to non-default.
The fix is to prevent the loop by checking if the app-op is already the
correct mode before setting it.

Fixes: 135674928
Test: atest IncidentReportListenerTest CollectorHostsideLibTest on
          cuttlefish and crosshatch
Change-Id: Ic65945e814957ac59495b3da221fabb0f3d42b66