diff options
author | TreeHugger Robot <treehugger-gerrit@google.com> | 2017-06-08 20:16:35 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2017-06-08 20:16:35 +0000 |
commit | 7c8a759e56faa087d6fcc888d4317a8937b3b812 (patch) | |
tree | 7221ab2fd85611a9c902571737ef9333e0917e01 | |
parent | 13c50710bee832b83ebf7dcd7cc6bf36d9fcfc82 (diff) | |
parent | dff5a4bb3bc1f473bfe5c492b130ba00d6891ebe (diff) | |
download | telephony-7c8a759e56faa087d6fcc888d4317a8937b3b812.tar.gz |
Merge "Clear calling identity in getCarrierImsPackage()" into oc-dev
-rw-r--r-- | src/java/com/android/internal/telephony/CarrierSmsUtils.java | 17 |
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() {} |