Merge "Don't wtf() for reverse profile access; make it a warning." into oc-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
5fd39fad26
@@ -419,7 +419,7 @@ public class LauncherApps {
|
|||||||
*/
|
*/
|
||||||
private void logErrorForInvalidProfileAccess(@NonNull UserHandle target) {
|
private void logErrorForInvalidProfileAccess(@NonNull UserHandle target) {
|
||||||
if (UserHandle.myUserId() != target.getIdentifier() && mUserManager.isManagedProfile()) {
|
if (UserHandle.myUserId() != target.getIdentifier() && mUserManager.isManagedProfile()) {
|
||||||
Log.e(TAG, "Accessing other profiles/users from managed profile is no longer allowed.");
|
Log.w(TAG, "Accessing other profiles/users from managed profile is no longer allowed.");
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -242,18 +242,8 @@ public class LauncherAppsService extends SystemService {
|
|||||||
try {
|
try {
|
||||||
UserInfo callingUserInfo = mUm.getUserInfo(callingUserId);
|
UserInfo callingUserInfo = mUm.getUserInfo(callingUserId);
|
||||||
if (callingUserInfo.isManagedProfile()) {
|
if (callingUserInfo.isManagedProfile()) {
|
||||||
|
Slog.w(TAG, message + " by " + callingPackage + " for another profile "
|
||||||
// STOPSHIP Remove the whitelist.
|
|
||||||
if ("com.google.android.talk".equals(callingPackage)
|
|
||||||
|| "com.google.android.quicksearchbox".equals(callingPackage)
|
|
||||||
|| "com.google.android.googlequicksearchbox".equals(callingPackage)
|
|
||||||
) {
|
|
||||||
return false;
|
|
||||||
}
|
|
||||||
// STOPSHIP Change it to 'e'.
|
|
||||||
Slog.wtfStack(TAG, message + " by " + callingPackage + " for another profile "
|
|
||||||
+ targetUserId + " from " + callingUserId);
|
+ targetUserId + " from " + callingUserId);
|
||||||
|
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user