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

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

Merge changes from topic "bug158482162_rvc-qpr-dev-rvc-qpr-dev-plus-aosp" into... 

Merge changes from topic "bug158482162_rvc-qpr-dev-rvc-qpr-dev-plus-aosp" into rvc-qpr-dev-plus-aosp

* changes:
  BackupManagerService: Make new behavior conditional on ChangeId.
  Enforce BACKUP permission on Service end.
parents 1de15941 c6d18392

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

+20 −2
Original line number Diff line number Diff line
@@ -20,10 +20,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;
@@ -390,6 +394,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
@@ -400,8 +415,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 −0
Original line number Diff line number Diff line
@@ -30,6 +30,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;
@@ -506,6 +507,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);

        }