diff options
| author | Martijn Coenen <maco@google.com> | 2018-06-05 11:02:23 +0200 |
|---|---|---|
| committer | Martijn Coenen <maco@google.com> | 2018-06-05 11:18:48 +0200 |
| commit | dfa390e0802bbbdd00d7911f893cc94ec57bf159 (patch) | |
| tree | 3480861ebcc12c01089de62eb395d101d4f232b1 /tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp | |
| parent | dea740bdceb20818d49a0f015c620d1d5d9e5adc (diff) | |
Serialize calls into BinderProxy.
The BinderProxy class is not thread-safe, hence all calls into it
must be serialized. This was achieved by holding the gProxyLock in
JNI code. However, a recent change added calls into BinderProxy
from ActivityManagerService without holding that lock, causing
ConcurrentModificationExceptions.
Instead of dumping debug info from AMS, make the call directly
from JNI, so we can make sure gProxyLock is held correctly.
Also, only dump on debug builds.
Bug: 71353150
Bug: 109701487
Test: sailfish builds, boots, info gets dumped with lowered limits.
Change-Id: I446a71ce4115b9936a01a170401ef98ba3818c0b
Diffstat (limited to 'tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp')
0 files changed, 0 insertions, 0 deletions