commit 8c38b252d48c16287e956021fa0c1e576bbbec9c
author Andy Yu <chingtangyu@google.com> Sat Jan 15 01:13:19 2022 -0800
committer Andy Yu <chingtangyu@google.com> Thu Jan 27 19:05:07 2022 -0800
tree 83888c496b1825929a3745c471353ce7df79158a
parent bc5f68fd1e3c3874a6c0832530254f8f1081fa2e

Add test for setOverrideFrameRate

- Create test to verify the API call in SurfaceComposerClient
- Create test to verify frame rate override DisplayEventReceiver events
are correctly received
- Add permission AID_ROOT to setOverrideFrameRate

Bug: b/204322816
Test: atest Surfaceflinger_test
Change-Id: I5b1782fcb7fc4896139ad56fcaee61853480f153

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 73a5f58..5ca75d5 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -5520,7 +5520,13 @@
             }
             return PERMISSION_DENIED;
         }
-        case SET_OVERRIDE_FRAME_RATE:
+        case SET_OVERRIDE_FRAME_RATE: {
+            const int uid = IPCThreadState::self()->getCallingUid();
+            if (uid == AID_ROOT || uid == AID_SYSTEM) {
+                return OK;
+            }
+            return PERMISSION_DENIED;
+        }
         case ON_PULL_ATOM: {
             const int uid = IPCThreadState::self()->getCallingUid();
             if (uid == AID_SYSTEM) {