Merge "overrideConfig should use existing notification flow for CCC" into qt-dev
diff --git a/src/com/android/phone/CarrierConfigLoader.java b/src/com/android/phone/CarrierConfigLoader.java
index af54f1f..cf7bafd 100644
--- a/src/com/android/phone/CarrierConfigLoader.java
+++ b/src/com/android/phone/CarrierConfigLoader.java
@@ -924,8 +924,12 @@
@Override
public void overrideConfig(int subscriptionId, PersistableBundle overrides) {
+ // SHELL UID already has MODIFY_PHONE_STATE implicitly so we do not have to check it, but
+ // the API signature explicitly declares that the method caller have MODIFY_PHONE_STATE, so
+ // calling this as well to be safe.
mContext.enforceCallingOrSelfPermission(
android.Manifest.permission.MODIFY_PHONE_STATE, null);
+ TelephonyPermissions.enforceShellOnly(Binder.getCallingUid(), "overrideConfig");
int phoneId = SubscriptionManager.getPhoneId(subscriptionId);
if (!SubscriptionManager.isValidPhoneId(phoneId)) {
log("Ignore invalid phoneId: " + phoneId + " for subId: " + subscriptionId);
@@ -942,7 +946,8 @@
} else {
mOverrideConfigs[phoneId].putAll(overrides);
}
- broadcastConfigChangedIntent(phoneId);
+
+ notifySubscriptionInfoUpdater(phoneId);
}
@Override