Grant adb the BACKUP permission
This also means we should no longer do explicit calling-identity fiddling in dump(), relying instead on the usual permission enforcement mechanism to handle things.
This commit is contained in:
@@ -133,6 +133,7 @@
|
||||
<assign-permission name="android.permission.READ_FRAME_BUFFER" uid="shell" />
|
||||
<assign-permission name="android.permission.DEVICE_POWER" uid="shell" />
|
||||
<assign-permission name="android.permission.INSTALL_LOCATION_PROVIDER" uid="shell" />
|
||||
<assign-permission name="android.permission.BACKUP" uid="shell" />
|
||||
|
||||
<assign-permission name="android.permission.MODIFY_AUDIO_SETTINGS" uid="media" />
|
||||
<assign-permission name="android.permission.ACCESS_DRM" uid="media" />
|
||||
|
||||
@@ -1548,8 +1548,6 @@ class BackupManagerService extends IBackupManager.Stub {
|
||||
@Override
|
||||
public void dump(FileDescriptor fd, PrintWriter pw, String[] args) {
|
||||
synchronized (mQueueLock) {
|
||||
long oldId = Binder.clearCallingIdentity();
|
||||
|
||||
pw.println("Backup Manager is " + (mEnabled ? "enabled" : "disabled")
|
||||
+ " / " + (!mProvisioned ? "not " : "") + "provisioned");
|
||||
pw.println("Available transports:");
|
||||
@@ -1572,8 +1570,6 @@ class BackupManagerService extends IBackupManager.Stub {
|
||||
for (BackupRequest req : mPendingBackups.values()) {
|
||||
pw.println(" " + req);
|
||||
}
|
||||
|
||||
Binder.restoreCallingIdentity(oldId);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user