diff options
| author | Todd Kennedy <toddke@google.com> | 2016-04-26 17:06:38 -0700 |
|---|---|---|
| committer | Todd Kennedy <toddke@google.com> | 2016-04-26 17:06:38 -0700 |
| commit | d5e295f9192522176796271cb29558a53d37c875 (patch) | |
| tree | 42e8b8bba2121d902a4995b78a771dc63d6d0d16 /tools/aapt2/java/JavaClassGenerator_test.cpp | |
| parent | 34510eb2933b98f0c8c73f9a7be5eae911a14210 (diff) | |
Don't hold lock when calling externally
Wound up in a deadlock. We held mPackages lock and called the mount
service. This called into the AppOps service which tried to obtain
its lock. On another thread, we held the AppOps service lock which
called into PackageManager service and tried to take mPackages.
Ideally, we'd never hold an internal lock when calling externally.
In practice, that's not always fixable [for example, code change
would be too great to not hold the AppOps lock when calling into
the PackageManager service].
Bug: 28169075
Change-Id: If0a7f1361c7be9385b1f629f68ebc0556b08ab06
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator_test.cpp')
0 files changed, 0 insertions, 0 deletions