Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Avoid wm<->am deadlock when checking uri permission" into rvc-dev

Browse Source
This commit is contained in:
Wale Ogunwale
2020-05-19 15:00:53 +00:00
committed by Android (Google) Code Review
parent 197709093a d638cd6164
commit 02c67b119e
1 changed files with 31 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

50
services/core/java/com/android/server/am/ActivityManagerService.java
View File

@@ -337,6 +337,7 @@ import com.android.internal.util.Preconditions;
import com.android.internal.util.function.HexFunction;
import com.android.internal.util.function.QuadFunction;
import com.android.internal.util.function.TriFunction;
import com.android.internal.util.function.pooled.PooledLambda;
import com.android.server.AlarmManagerInternal;
import com.android.server.AttributeCache;
import com.android.server.DeviceIdleInternal;
@@ -18831,28 +18832,39 @@ public class ActivityManagerService extends IActivityManager.Stub
@Override
public int checkContentProviderUriPermission(Uri uri, int userId,
int callingUid, int modeFlags) {
// We can find ourselves needing to check Uri permissions while
// already holding the WM lock, which means reaching back here for
// the AM lock would cause an inversion. The WM team has requested
// that we use the strategy below instead of shifting where Uri
// grants are calculated.
final Object wmLock = mActivityTaskManager.getGlobalLock();
if (Thread.currentThread().holdsLock(wmLock)
&& !Thread.currentThread().holdsLock(ActivityManagerService.this)) {
// We can find ourselves needing to check Uri permissions while already holding the
// WM lock, which means reaching back here for the AM lock would cause an inversion.
// The WM team has requested that we use the strategy below instead of shifting
// where Uri grants are calculated.
synchronized (wmLock) {
final int[] result = new int[1];
final Message msg = PooledLambda.obtainMessage(
LocalService::checkContentProviderUriPermission,
this, uri, userId, callingUid, modeFlags, wmLock, result);
mHandler.sendMessage(msg);
try {
wmLock.wait();
} catch (InterruptedException ignore) {
// Since we could also arrive here while holding the AM lock, we
// can't always delegate the call through the handler, and we need
// to delicately dance between the deadlocks.
if (Thread.currentThread().holdsLock(ActivityManagerService.this)) {
}
return result[0];
}
} else {
return ActivityManagerService.this.checkContentProviderUriPermission(uri,
userId, callingUid, modeFlags);
} else {
final CompletableFuture<Integer> res = new CompletableFuture<>();
mHandler.post(() -> {
res.complete(ActivityManagerService.this.checkContentProviderUriPermission(uri,
userId, callingUid, modeFlags));
});
try {
return res.get();
} catch (InterruptedException | ExecutionException e) {
throw new RuntimeException(e);
}
}
void checkContentProviderUriPermission(
Uri uri, int userId, int callingUid, int modeFlags, Object wmLock, int[] result) {
synchronized (ActivityManagerService.this) {
synchronized (wmLock) {
result[0] = ActivityManagerService.this.checkContentProviderUriPermission(
uri, userId, callingUid, modeFlags);
wmLock.notify();
}
}
}