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

Commit 38b01ce5 authored by Sooraj Sasindran's avatar Sooraj Sasindran
Browse files

store preferential configs if no error from CS 

store preferential configs in deivice policy only if no error from
connectivity service

Bug: 283999128
Test: Added new test to CTS and ran PreferentialNetworkServiceTest
Change-Id: I8421e4d21b8719426cfb998d96742e065641efcd
parent 10bfdb5c

services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java

+15 −9
Original line number Diff line number Diff line
@@ -12166,6 +12166,9 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager { 
                "Caller is not managed profile owner or device owner;"

                        + " only managed profile owner or device owner may control the preferential"

                        + " network service");













        try {













            updateNetworkPreferenceForUser(caller.getUserId(), preferentialNetworkServiceConfigs);















            synchronized (getLockObject()) {















                final ActiveAdmin requiredAdmin = getDeviceOrProfileOwnerAdminLocked(















                        caller.getUserId());









@@ -12176,7 +12179,10 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager {













                    saveSettingsLocked(caller.getUserId());















                }















            }













        updateNetworkPreferenceForUser(caller.getUserId(), preferentialNetworkServiceConfigs);













        } catch (Exception e) {













            Slogf.e(LOG_TAG, "Failed to set preferential network service configs");













            throw e;













        }















        DevicePolicyEventLogger















                .createEvent(DevicePolicyEnums.SET_PREFERENTIAL_NETWORK_SERVICE_ENABLED)















                .setBoolean(preferentialNetworkServiceConfigs