diff options
| author | Christopher Tate <ctate@google.com> | 2015-02-24 15:08:08 -0800 |
|---|---|---|
| committer | Christopher Tate <ctate@google.com> | 2015-02-24 15:16:57 -0800 |
| commit | 8dd7d01c1a07d9893ce1207c90869fe3d40fbced (patch) | |
| tree | db5ed59b6923f3674af382d629c98bafcd79d01e /services/backup | |
| parent | a1879609890640bd26639fc3ce1fa482efd17e8d (diff) | |
| download | frameworks_base-8dd7d01c1a07d9893ce1207c90869fe3d40fbced.zip frameworks_base-8dd7d01c1a07d9893ce1207c90869fe3d40fbced.tar.gz frameworks_base-8dd7d01c1a07d9893ce1207c90869fe3d40fbced.tar.bz2 | |
Check DUMP permission in the backup service trampoline
Make sure that even if backup is disabled outright (and hence there
is no underlying service implementation for the trampoline to delegate
to), the DUMP permission exception is thrown as expected.
Bug 19422232
Change-Id: I6d1a17c5f85adcfad75af969b521920e786c05a8
Diffstat (limited to 'services/backup')
| -rw-r--r-- | services/backup/java/com/android/server/backup/Trampoline.java | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/services/backup/java/com/android/server/backup/Trampoline.java b/services/backup/java/com/android/server/backup/Trampoline.java index 8bd7132..2e84fbe 100644 --- a/services/backup/java/com/android/server/backup/Trampoline.java +++ b/services/backup/java/com/android/server/backup/Trampoline.java @@ -318,6 +318,8 @@ public class Trampoline extends IBackupManager.Stub { @Override public void dump(FileDescriptor fd, PrintWriter pw, String[] args) { + mContext.enforceCallingOrSelfPermission(android.Manifest.permission.DUMP, TAG); + BackupManagerService svc = mService; if (svc != null) { svc.dump(fd, pw, args); |