diff options
author | Adam Seaton <aseaton@google.com> | 2017-09-29 22:14:59 +0000 |
---|---|---|
committer | android-build-team Robot <android-build-team-robot@google.com> | 2017-09-29 22:21:57 +0000 |
commit | c8b6aa86b8b3a120189441dd6ec41cf78654af50 (patch) | |
tree | 2f46f2ff81dc4493a137a86fc0adf72d83e89f08 | |
parent | 8d55654bb3db592aec6453d6cfbb211cf962f841 (diff) | |
download | base-c8b6aa86b8b3a120189441dd6ec41cf78654af50.tar.gz |
Revert "Prevent getting data from Clipboard if device is locked"
This reverts commit 93d77b07c34077b6c403c459b7bb75933446a502.
Change-Id: I3ab20d3de1cb1e5fa94fffb8b69683b7508a3503
(cherry picked from commit 4fd5096fffb5aba9cde567981e28558e31e86345)
-rw-r--r-- | services/core/java/com/android/server/clipboard/ClipboardService.java | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/services/core/java/com/android/server/clipboard/ClipboardService.java b/services/core/java/com/android/server/clipboard/ClipboardService.java index efc930ebf8ea..db72c5e4cbab 100644 --- a/services/core/java/com/android/server/clipboard/ClipboardService.java +++ b/services/core/java/com/android/server/clipboard/ClipboardService.java @@ -20,7 +20,6 @@ import android.app.ActivityManager; import android.app.AppGlobals; import android.app.AppOpsManager; import android.app.IActivityManager; -import android.app.KeyguardManager; import android.content.ClipData; import android.content.ClipDescription; import android.content.ContentProvider; @@ -305,7 +304,7 @@ public class ClipboardService extends SystemService { public ClipData getPrimaryClip(String pkg) { synchronized (this) { if (!clipboardAccessAllowed(AppOpsManager.OP_READ_CLIPBOARD, pkg, - Binder.getCallingUid()) || isDeviceLocked()) { + Binder.getCallingUid())) { return null; } addActiveOwnerLocked(Binder.getCallingUid(), pkg); @@ -317,7 +316,7 @@ public class ClipboardService extends SystemService { public ClipDescription getPrimaryClipDescription(String callingPackage) { synchronized (this) { if (!clipboardAccessAllowed(AppOpsManager.OP_READ_CLIPBOARD, callingPackage, - Binder.getCallingUid()) || isDeviceLocked()) { + Binder.getCallingUid())) { return null; } PerUserClipboard clipboard = getClipboard(); @@ -329,7 +328,7 @@ public class ClipboardService extends SystemService { public boolean hasPrimaryClip(String callingPackage) { synchronized (this) { if (!clipboardAccessAllowed(AppOpsManager.OP_READ_CLIPBOARD, callingPackage, - Binder.getCallingUid()) || isDeviceLocked()) { + Binder.getCallingUid())) { return false; } return getClipboard().primaryClip != null; @@ -356,7 +355,7 @@ public class ClipboardService extends SystemService { public boolean hasClipboardText(String callingPackage) { synchronized (this) { if (!clipboardAccessAllowed(AppOpsManager.OP_READ_CLIPBOARD, callingPackage, - Binder.getCallingUid()) || isDeviceLocked()) { + Binder.getCallingUid())) { return false; } PerUserClipboard clipboard = getClipboard(); @@ -434,12 +433,6 @@ public class ClipboardService extends SystemService { } } - private boolean isDeviceLocked() { - final KeyguardManager keyguardManager = getContext().getSystemService( - KeyguardManager.class); - return keyguardManager != null && keyguardManager.isDeviceLocked(); - } - private final void checkUriOwnerLocked(Uri uri, int uid) { if (!"content".equals(uri.getScheme())) { return; |