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

Commit 86389b3e authored by Makoto Onuki's avatar Makoto Onuki
Browse files

Temporarily disable LauncherApps reverse access check 

Bug 34650921
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest1 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest2 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest3 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest4 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest5 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest6 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest7 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest8 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest9 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest10 -w com.android.frameworks.servicestests

With the reverted CTS,
Test: cts-tradefed run cts --skip-device-info --skip-preconditions --skip-system-status-check com.android.compatibility.common.tradefed.targetprep.NetworkConnectivityChecker -a armeabi-v7a -m CtsShortcutHostTestCases -t 'android.content.pm.cts.shortcuthost.ShortcutManagerMultiuserTest#testManagedUser'
Test: cts-tradefed run cts --skip-device-info --skip-preconditions --skip-system-status-check com.android.compatibility.common.tradefed.targetprep.NetworkConnectivityChecker -a armeabi-v7a -m CtsDevicePolicyManagerTestCases -t 'com.android.cts.devicepolicy.LauncherAppsProfileTest'

Change-Id: Id9d13c11816fca10b2dacac92537b2baded3ac9c
parent 22040a53

services/core/java/com/android/server/pm/LauncherAppsService.java

+6 −1
Original line number Diff line number Diff line
@@ -232,7 +232,12 @@ public class LauncherAppsService extends SystemService { 
            try {

                UserInfo callingUserInfo = mUm.getUserInfo(callingUserId);

                if (callingUserInfo.isManagedProfile()) {

                    throw new SecurityException(message + " for another profile " + targetUserId);

                    // TODO: Make it SecurityException.  See b/34650921

                    // throw new SecurityException(message + " for another profile " + targetUserId);



                    // TODO: Report caller package name.

                    Slog.wtfStack(TAG, message + " for another profile " + targetUserId

                            + " from " + callingUserId);

                }



                UserInfo targetUserInfo = mUm.getUserInfo(targetUserId);