summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Svetoslav Ganov <svetoslavganov@google.com> 2015-11-20 21:10:52 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2015-11-20 21:10:52 +0000
commitc47132c20590d3daa669c89ba9347f27e4dae8db (patch)
tree72d6d965036f984b133bb35868e8712c4e86f831
parent1f683a112847607865ff238ff2c6764075c60834 (diff)
parentd8d25e00386dd768a59e38eb3e1ae0cec0e797b2 (diff)
Merge "Use resolved calling user id"
-rw-r--r--packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java6
1 files changed, 3 insertions, 3 deletions
diff --git a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java
index 8b1caf9411ee..3971706b02d8 100644
--- a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java
+++ b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java
@@ -904,12 +904,12 @@ public class SettingsProvider extends ContentProvider {
}
}
- // Enforce what the calling package can mutate the system settings.
- enforceRestrictedSystemSettingsMutationForCallingPackage(operation, name, runAsUserId);
-
// Resolve the userId on whose behalf the call is made.
final int callingUserId = resolveCallingUserIdEnforcingPermissionsLocked(runAsUserId);
+ // Enforce what the calling package can mutate the system settings.
+ enforceRestrictedSystemSettingsMutationForCallingPackage(operation, name, callingUserId);
+
// Determine the owning user as some profile settings are cloned from the parent.
final int owningUserId = resolveOwningUserIdForSystemSettingLocked(callingUserId, name);