aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTreeHugger Robot <treehugger-gerrit@google.com>2017-06-08 20:16:35 +0000
committerAndroid (Google) Code Review <android-gerrit@google.com>2017-06-08 20:16:35 +0000
commit7c8a759e56faa087d6fcc888d4317a8937b3b812 (patch)
tree7221ab2fd85611a9c902571737ef9333e0917e01
parent13c50710bee832b83ebf7dcd7cc6bf36d9fcfc82 (diff)
parentdff5a4bb3bc1f473bfe5c492b130ba00d6891ebe (diff)
downloadtelephony-7c8a759e56faa087d6fcc888d4317a8937b3b812.tar.gz
Merge "Clear calling identity in getCarrierImsPackage()" into oc-dev
-rw-r--r--src/java/com/android/internal/telephony/CarrierSmsUtils.java17
1 files changed, 11 insertions, 6 deletions
diff --git a/src/java/com/android/internal/telephony/CarrierSmsUtils.java b/src/java/com/android/internal/telephony/CarrierSmsUtils.java
index 845860cdfa..a64aea7ce3 100644
--- a/src/java/com/android/internal/telephony/CarrierSmsUtils.java
+++ b/src/java/com/android/internal/telephony/CarrierSmsUtils.java
@@ -21,6 +21,7 @@ import android.content.Context;
import android.content.Intent;
import android.content.pm.PackageManager;
import android.content.pm.ResolveInfo;
+import android.os.Binder;
import android.os.PersistableBundle;
import android.telephony.CarrierConfigManager;
import android.telephony.Rlog;
@@ -78,13 +79,17 @@ public class CarrierSmsUtils {
return null;
}
- PersistableBundle config = cm.getConfigForSubId(phone.getSubId());
- if (config == null) {
- if (VDBG) Rlog.v(TAG, "No CarrierConfig for subId:" + phone.getSubId());
- return null;
+ final long identity = Binder.clearCallingIdentity();
+ try {
+ PersistableBundle config = cm.getConfigForSubId(phone.getSubId());
+ if (config == null) {
+ if (VDBG) Rlog.v(TAG, "No CarrierConfig for subId:" + phone.getSubId());
+ return null;
+ }
+ return config.getString(CARRIER_IMS_PACKAGE_KEY, null);
+ } finally {
+ Binder.restoreCallingIdentity(identity);
}
-
- return config.getString(CARRIER_IMS_PACKAGE_KEY, null);
}
private CarrierSmsUtils() {}