commit 041a075262ef5fc886e46fd4eec3dd79ee2e60c0
author Mathias Agopian <mathias@google.com> Fri Mar 15 18:31:56 2013 -0700
committer Mathias Agopian <mathias@google.com> Fri Mar 15 18:39:16 2013 -0700
tree 495a923b01bc1b81ca0d780b69af3d24d168b0d0
parent 7670d3cb2b5d38c60135f34a2446e1ae809d1b32

don't allow screenshots without the READ_FRAMEBUFFER permission

the recent screenshot rework allowed the older screenshot
interface to work without that permission

Change-Id: I6c4743f4591c81106e3b823d55a055f7b4907de1

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 1daadcc..edf01f7 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -2399,6 +2399,7 @@
 {
     switch (code) {
         case CREATE_CONNECTION:
+        case CREATE_DISPLAY:
         case SET_TRANSACTION_STATE:
         case BOOT_FINISHED:
         case BLANK:
@@ -2417,6 +2418,7 @@
             break;
         }
         case CAPTURE_SCREEN:
+        case CAPTURE_SCREEN_DEPRECATED:
         {
             // codes that require permission check
             IPCThreadState* ipc = IPCThreadState::self();