summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author TreeHugger Robot <treehugger-gerrit@google.com> 2020-05-27 23:03:40 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2020-05-27 23:03:40 +0000
commit10bee631fde96b97a9e23cf50d19369c115ef3d3 (patch)
treebb79371f3bdcf237616e145b7c105fec0e90c6c9
parentc98eb7aecefa369f72198db50c383abd71a3f1d6 (diff)
parent2ffca5a0de56c8ce3826526abd2b7d934356df5e (diff)
Merge "Adding permission check for dumping visible windows" into rvc-dev
-rw-r--r--services/core/java/com/android/server/wm/WindowManagerShellCommand.java5
1 files changed, 5 insertions, 0 deletions
diff --git a/services/core/java/com/android/server/wm/WindowManagerShellCommand.java b/services/core/java/com/android/server/wm/WindowManagerShellCommand.java
index 619d87bc49f0..bdecb8d99752 100644
--- a/services/core/java/com/android/server/wm/WindowManagerShellCommand.java
+++ b/services/core/java/com/android/server/wm/WindowManagerShellCommand.java
@@ -351,6 +351,11 @@ public class WindowManagerShellCommand extends ShellCommand {
}
private int runDumpVisibleWindowViews(PrintWriter pw) {
+ if (!mInternal.checkCallingPermission(android.Manifest.permission.DUMP,
+ "runDumpVisibleWindowViews()")) {
+ throw new SecurityException("Requires DUMP permission");
+ }
+
try (ZipOutputStream out = new ZipOutputStream(getRawOutputStream())) {
ArrayList<Pair<String, ByteTransferPipe>> requestList = new ArrayList<>();
synchronized (mInternal.mGlobalLock) {