Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add dump permission check in TunerResourceManagerService#dump()

Browse Source 
This is the fix for regression introduced by commit: 17c4a944

Bug: 213479108
Test: android.security.cts.ServicePermissionsTest#testDumpProtected
Change-Id: I92e0d9b29a5a8676c02d3bbabec3393651839353
This commit is contained in:
Kensuke Miyagi
2022-01-06 11:54:19 -08:00
parent 238f9d577d
commit c5fabb864d
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
services/core/java/com/android/server/tv/tunerresourcemanager/TunerResourceManagerService.java
View File

@@ -21,6 +21,7 @@ import android.annotation.Nullable;
import android.app.ActivityManager;
import android.app.ActivityManager.RunningAppProcessInfo;
import android.content.Context;
import android.content.pm.PackageManager;
import android.media.IResourceManagerService;
import android.media.tv.TvInputManager;
import android.media.tv.tunerresourcemanager.CasSessionRequest;
@@ -543,6 +544,12 @@ public class TunerResourceManagerService extends SystemService implements IBinde
protected void dump(FileDescriptor fd, final PrintWriter writer, String[] args) {
final IndentingPrintWriter pw = new IndentingPrintWriter(writer, " ");
if (getContext().checkCallingOrSelfPermission(android.Manifest.permission.DUMP)
!= PackageManager.PERMISSION_GRANTED) {
pw.println("Permission Denial: can't dump!");
return;
}
synchronized (mLock) {
if (mClientProfiles != null) {
pw.println("ClientProfiles:");