From 522ed24f428f6cc1f2d1aa9acdc5636f51c2e68c Mon Sep 17 00:00:00 2001 From: Svet Ganov Date: Wed, 7 Mar 2018 21:40:28 -0800 Subject: [PATCH] Use proper start op overload in all places Test: issued a sequences of app ops that led to a crash and after the fix this sequence no longer causes a crash bug: 74209446 Change-Id: I5064b51387ed98e42743ed94ef92114944f28761 --- services/core/java/com/android/server/wm/WindowState.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/services/core/java/com/android/server/wm/WindowState.java b/services/core/java/com/android/server/wm/WindowState.java index 2398661668b1b..f36b692135f37 100644 --- a/services/core/java/com/android/server/wm/WindowState.java +++ b/services/core/java/com/android/server/wm/WindowState.java @@ -2610,7 +2610,7 @@ class WindowState extends WindowContainer implements WindowManagerP setAppOpVisibilityLw(false); } } else { - final int mode = mService.mAppOps.startOpNoThrow(mAppOp, uid, packageName); + final int mode = mService.mAppOps.startOpNoThrow(mAppOp, uid, packageName, true); if (mode == MODE_ALLOWED || mode == MODE_DEFAULT) { setAppOpVisibilityLw(true); }