Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2b02b53e authored by Guojing Yuan's avatar Guojing Yuan
Browse files

[CDM][CMD] Check permissions for CDM shell commands 

Override handleShellCommand instead of onShellCommand because
Binder.onShellCommand checks the necessary permissions of the caller.

Bug: 313428840

Test: manually tested CDM shell commands
Change-Id: I5539b3594feb5544c458c0fd1061b51a0a808900
Merged-In: I5539b3594feb5544c458c0fd1061b51a0a808900
(cherry picked from commit 1761a0fe)
parent b614ab29

services/companion/java/com/android/server/companion/CompanionDeviceManagerService.java

+7 −6
Original line number Diff line number Diff line
@@ -87,12 +87,11 @@ import android.os.Handler; 
import android.os.IBinder;

import android.os.IInterface;

import android.os.Parcel;

import android.os.ParcelFileDescriptor;

import android.os.PowerWhitelistManager;

import android.os.Process;

import android.os.RemoteException;

import android.os.ResultReceiver;

import android.os.ServiceManager;

import android.os.ShellCallback;

import android.os.ShellCommand;

import android.os.UserHandle;

import android.os.UserManager;
@@ -708,10 +707,12 @@ public class CompanionDeviceManagerService extends SystemService implements Bind 
        }



        @Override

        public void onShellCommand(FileDescriptor in, FileDescriptor out, FileDescriptor err,

                String[] args, ShellCallback callback, ResultReceiver resultReceiver)

                throws RemoteException {

            new ShellCmd().exec(this, in, out, err, args, callback, resultReceiver);

        public int handleShellCommand(@NonNull ParcelFileDescriptor in,

                @NonNull ParcelFileDescriptor out, @NonNull ParcelFileDescriptor err,

                @NonNull String[] args) {

            return new ShellCmd()

                    .exec(this, in.getFileDescriptor(), out.getFileDescriptor(),

                            err.getFileDescriptor(), args);

        }



        @Override