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

Commit a6bccfc4 authored by Tobias Thierer's avatar Tobias Thierer Committed by Android (Google) Code Review
Browse files

Merge changes I58e5d2a0,I59693819 

* changes:
  BackupManagerService: Make new behavior conditional on ChangeId.
  Enforce BACKUP permission on Service end.
parents ccadd3ec 7671f0d9

core/java/android/app/backup/BackupManager.java

+20 −2
Original line number Diff line number Diff line
@@ -21,10 +21,14 @@ import android.annotation.NonNull; 
import android.annotation.Nullable;

import android.annotation.RequiresPermission;

import android.annotation.SystemApi;

import android.app.compat.CompatChanges;

import android.compat.annotation.ChangeId;

import android.compat.annotation.EnabledAfter;

import android.compat.annotation.UnsupportedAppUsage;

import android.content.ComponentName;

import android.content.Context;

import android.content.Intent;

import android.os.Build;

import android.os.Bundle;

import android.os.Handler;

import android.os.Message;
@@ -435,6 +439,17 @@ public class BackupManager { 
        return false;

    }





    /**

     * If this change is enabled, the {@code BACKUP} permission needed for

     * {@code isBackupServiceActive()} will be enforced on the service end

     * rather than client-side in {@link BackupManager}.

     * @hide

     */

    @ChangeId

    @EnabledAfter(targetSdkVersion = Build.VERSION_CODES.R)

    public static final long IS_BACKUP_SERVICE_ACTIVE_ENFORCE_PERMISSION_IN_SERVICE = 158482162;



    /**

     * Report whether the backup mechanism is currently active.

     * When it is inactive, the device will not perform any backup operations, nor will it
@@ -445,8 +460,11 @@ public class BackupManager { 
    @SystemApi

    @RequiresPermission(android.Manifest.permission.BACKUP)

    public boolean isBackupServiceActive(UserHandle user) {

        if (!CompatChanges.isChangeEnabled(

                IS_BACKUP_SERVICE_ACTIVE_ENFORCE_PERMISSION_IN_SERVICE)) {

            mContext.enforceCallingOrSelfPermission(android.Manifest.permission.BACKUP,

                    "isBackupServiceActive");

        }

        checkServiceBinder();

        if (sService != null) {

            try {

services/backup/java/com/android/server/backup/BackupManagerService.java

+7 −1
Original line number Diff line number Diff line
@@ -32,6 +32,7 @@ import android.app.backup.IBackupObserver; 
import android.app.backup.IFullBackupRestoreObserver;

import android.app.backup.IRestoreSession;

import android.app.backup.ISelectBackupTransportCallback;

import android.app.compat.CompatChanges;

import android.app.job.JobParameters;

import android.app.job.JobScheduler;

import android.app.job.JobService;
@@ -61,7 +62,6 @@ import com.android.internal.annotations.VisibleForTesting; 
import com.android.internal.util.DumpUtils;

import com.android.server.SystemConfig;

import com.android.server.SystemService;

import com.android.server.SystemService.TargetUser;

import com.android.server.backup.utils.RandomAccessFileUtils;



import java.io.File;
@@ -509,6 +509,12 @@ public class BackupManagerService extends IBackupManager.Stub { 
     */

    @Override

    public boolean isBackupServiceActive(int userId) {

        int callingUid = Binder.getCallingUid();

        if (CompatChanges.isChangeEnabled(

                BackupManager.IS_BACKUP_SERVICE_ACTIVE_ENFORCE_PERMISSION_IN_SERVICE, callingUid)) {

            mContext.enforceCallingPermission(android.Manifest.permission.BACKUP,

                    "isBackupServiceActive");

        }

        synchronized (mStateLock) {

            return !mGlobalDisable && isBackupActivatedForUser(userId);

        }