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

Merge "Add null check for RemoteToken#getContainer()"

Browse Source
This commit is contained in:
Chris Li
2020-11-05 17:45:06 +00:00
committed by Android (Google) Code Review
parent d72a9640e8 144c6f912c
commit 4fca33f2d8
3 changed files with 39 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
services/core/java/com/android/server/wm/TaskOrganizerController.java
View File

@@ -442,7 +442,11 @@ class TaskOrganizerController extends ITaskOrganizerController.Stub {
final long origId = Binder.clearCallingIdentity();
try {
synchronized (mGlobalLock) {
final Task task = WindowContainer.fromBinder(token.asBinder()).asTask();
final WindowContainer wc = WindowContainer.fromBinder(token.asBinder());
if (wc == null) {
throw new IllegalArgumentException("Can't resolve window from token");
}
final Task task = wc.asTask();
if (task == null) return false;
if (!task.mCreatedByOrganizer) {
throw new IllegalArgumentException(
@@ -563,8 +567,14 @@ class TaskOrganizerController extends ITaskOrganizerController.Stub {
if (defaultTaskDisplayArea == null) {
return;
}
Task task = token == null
? null : WindowContainer.fromBinder(token.asBinder()).asTask();
WindowContainer wc = null;
if (token != null) {
wc = WindowContainer.fromBinder(token.asBinder());
if (wc == null) {
throw new IllegalArgumentException("Can't resolve window from token");
}
}
final Task task = wc == null ? null : wc.asTask();
if (task == null) {
defaultTaskDisplayArea.mLaunchRootTask = null;
return;
@@ -665,7 +675,12 @@ class TaskOrganizerController extends ITaskOrganizerController.Stub {
synchronized (mGlobalLock) {
ProtoLog.v(WM_DEBUG_WINDOW_ORGANIZER, "Set intercept back pressed on root=%b",
interceptBackPressed);
final Task task = WindowContainer.fromBinder(token.asBinder()).asTask();
final WindowContainer wc = WindowContainer.fromBinder(token.asBinder());
if (wc == null) {
Slog.w(TAG, "Could not resolve window from token");
return;
}
final Task task = wc.asTask();
if (task == null) {
Slog.w(TAG, "Could not resolve task from token");
return;

2
services/core/java/com/android/server/wm/WindowContainer.java
View File

@@ -2878,6 +2878,7 @@ class WindowContainer<E extends WindowContainer> extends ConfigurationContainer<
return true;
}
@Nullable
static WindowContainer fromBinder(IBinder binder) {
return RemoteToken.fromBinder(binder).getContainer();
}
@@ -2891,6 +2892,7 @@ class WindowContainer<E extends WindowContainer> extends ConfigurationContainer<
mWeakRef = new WeakReference<>(container);
}
@Nullable
WindowContainer getContainer() {
return mWeakRef.get();
}

21
services/core/java/com/android/server/wm/WindowOrganizerController.java
View File

@@ -249,7 +249,7 @@ class WindowOrganizerController extends IWindowOrganizerController.Stub
for (int i = 0, n = hops.size(); i < n; ++i) {
final WindowContainerTransaction.HierarchyOp hop = hops.get(i);
final WindowContainer wc = WindowContainer.fromBinder(hop.getContainer());
if (!wc.isAttached()) {
if (wc == null || !wc.isAttached()) {
Slog.e(TAG, "Attempt to operate on detached container: " + wc);
continue;
}
@@ -260,7 +260,13 @@ class WindowOrganizerController extends IWindowOrganizerController.Stub
if (transition != null) {
transition.collect(wc);
if (hop.isReparent() && hop.getNewParent() != null) {
transition.collect(WindowContainer.fromBinder(hop.getNewParent()));
final WindowContainer parentWc =
WindowContainer.fromBinder(hop.getNewParent());
if (parentWc == null) {
Slog.e(TAG, "Can't resolve parent window from token");
continue;
}
transition.collect(parentWc);
}
}
}
@@ -269,7 +275,12 @@ class WindowOrganizerController extends IWindowOrganizerController.Stub
entries = t.getChanges().entrySet().iterator();
while (entries.hasNext()) {
final Map.Entry<IBinder, WindowContainerTransaction.Change> entry = entries.next();
final Task task = WindowContainer.fromBinder(entry.getKey()).asTask();
final WindowContainer wc = WindowContainer.fromBinder(entry.getKey());
if (wc == null || !wc.isAttached()) {
Slog.e(TAG, "Attempt to operate on detached container: " + wc);
continue;
}
final Task task = wc.asTask();
final Rect surfaceBounds = entry.getValue().getBoundsChangeSurfaceBounds();
if (task == null || !task.isAttached() || surfaceBounds == null) {
continue;
@@ -429,6 +440,10 @@ class WindowOrganizerController extends IWindowOrganizerController.Stub
WindowContainer newParent = hop.getNewParent() == null
? dc.getDefaultTaskDisplayArea()
: WindowContainer.fromBinder(hop.getNewParent());
if (newParent == null) {
Slog.e(TAG, "Can't resolve parent window from token");
return 0;
}
if (task.getParent() != newParent) {
if (newParent instanceof TaskDisplayArea) {
// For now, reparenting to displayarea is different from other reparents...