Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Remove checks for system_process in TelephonyManager"

Browse Source
This commit is contained in:
Sarah Chin
2021-06-04 02:11:17 +00:00
committed by Gerrit Code Review
parent 6428a56e09 4a3db57ba5
commit 7576e161f1
1 changed files with 13 additions and 53 deletions
Download Patch File Download Diff File Expand all files Collapse all files

66
telephony/java/android/telephony/TelephonyManager.java
View File

@@ -59,7 +59,6 @@ import android.os.Handler;
import android.os.IBinder; import android.os.IBinder;
import android.os.ParcelFileDescriptor; import android.os.ParcelFileDescriptor;
import android.os.PersistableBundle; import android.os.PersistableBundle;
import android.os.Process;
import android.os.RemoteException; import android.os.RemoteException;
import android.os.ResultReceiver; import android.os.ResultReceiver;
import android.os.SystemProperties; import android.os.SystemProperties;
@@ -414,10 +413,6 @@ public class TelephonyManager {
return null; return null;
} }
private boolean isSystemProcess() {
return Process.myUid() == Process.SYSTEM_UID;
}
/** /**
* Post a runnable to the BackgroundThread. * Post a runnable to the BackgroundThread.
* *
@@ -4196,19 +4191,12 @@ public class TelephonyManager {
try { try {
IPhoneSubInfo info = getSubscriberInfoService(); IPhoneSubInfo info = getSubscriberInfoService();
if (info == null) { if (info == null) {
Rlog.e(TAG, "IMSI error: Subscriber Info is null"); throw new RuntimeException("IMSI error: Subscriber Info is null");
if (!isSystemProcess()) {
throw new RuntimeException("IMSI error: Subscriber Info is null");
}
return;
} }
int subId = getSubId(SubscriptionManager.getDefaultDataSubscriptionId()); int subId = getSubId(SubscriptionManager.getDefaultDataSubscriptionId());
info.resetCarrierKeysForImsiEncryption(subId, mContext.getOpPackageName()); info.resetCarrierKeysForImsiEncryption(subId, mContext.getOpPackageName());
} catch (RemoteException ex) { } catch (RemoteException ex) {
Rlog.e(TAG, "getCarrierInfoForImsiEncryption RemoteException" + ex); Rlog.e(TAG, "Telephony#getCarrierInfoForImsiEncryption RemoteException" + ex);
if (!isSystemProcess()) {
ex.rethrowAsRuntimeException();
}
} }
} }
@@ -5247,17 +5235,11 @@ public class TelephonyManager {
try { try {
final ITelephony telephony = getITelephony(); final ITelephony telephony = getITelephony();
if (telephony == null) { if (telephony == null) {
if (!isSystemProcess()) {
throw new RuntimeException("Telephony service unavailable");
}
return; return;
} }
telephony.sendDialerSpecialCode(mContext.getOpPackageName(), inputCode); telephony.sendDialerSpecialCode(mContext.getOpPackageName(), inputCode);
} catch (RemoteException ex) { } catch (RemoteException ex) {
// This could happen if binder process crashes. Rlog.e(TAG, "Telephony#sendDialerSpecialCode RemoteException" + ex);
if (!isSystemProcess()) {
ex.rethrowAsRuntimeException();
}
} }
} }
@@ -9534,9 +9516,7 @@ public class TelephonyManager {
throw new IllegalStateException("telephony service is null."); throw new IllegalStateException("telephony service is null.");
} }
} catch (RemoteException ex) { } catch (RemoteException ex) {
if (!isSystemProcess()) { Rlog.e(TAG, "Telephony#getMobileProvisioningUrl RemoteException" + ex);
ex.rethrowAsRuntimeException();
}
} }
return null; return null;
} }
@@ -13375,9 +13355,7 @@ public class TelephonyManager {
return service.isDataEnabledForApn(apnType, getSubId(), pkgForDebug); return service.isDataEnabledForApn(apnType, getSubId(), pkgForDebug);
} }
} catch (RemoteException ex) { } catch (RemoteException ex) {
if (!isSystemProcess()) { Rlog.e(TAG, "Telephony#isDataEnabledForApn RemoteException" + ex);
ex.rethrowAsRuntimeException();
}
} }
return false; return false;
} }
@@ -13397,9 +13375,7 @@ public class TelephonyManager {
return service.isApnMetered(apnType, getSubId()); return service.isApnMetered(apnType, getSubId());
} }
} catch (RemoteException ex) { } catch (RemoteException ex) {
if (!isSystemProcess()) { Rlog.e(TAG, "Telephony#isApnMetered RemoteException" + ex);
ex.rethrowAsRuntimeException();
}
} }
return true; return true;
} }
@@ -13459,9 +13435,7 @@ public class TelephonyManager {
service.setSystemSelectionChannels(specifiers, getSubId(), aidlConsumer); service.setSystemSelectionChannels(specifiers, getSubId(), aidlConsumer);
} }
} catch (RemoteException ex) { } catch (RemoteException ex) {
if (!isSystemProcess()) { Rlog.e(TAG, "Telephony#setSystemSelectionChannels RemoteException" + ex);
ex.rethrowAsRuntimeException();
}
} }
} }
@@ -13489,9 +13463,7 @@ public class TelephonyManager {
throw new IllegalStateException("telephony service is null."); throw new IllegalStateException("telephony service is null.");
} }
} catch (RemoteException ex) { } catch (RemoteException ex) {
if (!isSystemProcess()) { Rlog.e(TAG, "Telephony#getSystemSelectionChannels RemoteException" + ex);
ex.rethrowAsRuntimeException();
}
} }
return new ArrayList<>(); return new ArrayList<>();
} }
@@ -13520,9 +13492,7 @@ public class TelephonyManager {
return service.isMvnoMatched(getSubId(), mvnoType, mvnoMatchData); return service.isMvnoMatched(getSubId(), mvnoType, mvnoMatchData);
} }
} catch (RemoteException ex) { } catch (RemoteException ex) {
if (!isSystemProcess()) { Rlog.e(TAG, "Telephony#matchesCurrentSimOperator RemoteException" + ex);
ex.rethrowAsRuntimeException();
}
} }
return false; return false;
} }
@@ -13926,10 +13896,7 @@ public class TelephonyManager {
service.setMobileDataPolicyEnabledStatus(getSubId(), policy, enabled); service.setMobileDataPolicyEnabledStatus(getSubId(), policy, enabled);
} }
} catch (RemoteException ex) { } catch (RemoteException ex) {
// This could happen if binder process crashes. Rlog.e(TAG, "Telephony#setMobileDataPolicyEnabled RemoteException" + ex);
if (!isSystemProcess()) {
ex.rethrowAsRuntimeException();
}
} }
} }
@@ -13950,10 +13917,7 @@ public class TelephonyManager {
return service.isMobileDataPolicyEnabled(getSubId(), policy); return service.isMobileDataPolicyEnabled(getSubId(), policy);
} }
} catch (RemoteException ex) { } catch (RemoteException ex) {
// This could happen if binder process crashes. Rlog.e(TAG, "Telephony#isMobileDataPolicyEnabled RemoteException" + ex);
if (!isSystemProcess()) {
ex.rethrowAsRuntimeException();
}
} }
return false; return false;
} }
@@ -14457,9 +14421,7 @@ public class TelephonyManager {
throw new IllegalStateException("telephony service is null."); throw new IllegalStateException("telephony service is null.");
} }
} catch (RemoteException ex) { } catch (RemoteException ex) {
if (!isSystemProcess()) { Rlog.e(TAG, "Telephony#getEquivalentHomePlmns RemoteException" + ex);
ex.rethrowAsRuntimeException();
}
} }
return Collections.emptyList(); return Collections.emptyList();
@@ -14553,9 +14515,7 @@ public class TelephonyManager {
throw new IllegalStateException("telephony service is null."); throw new IllegalStateException("telephony service is null.");
} }
} catch (RemoteException ex) { } catch (RemoteException ex) {
if (!isSystemProcess()) { Rlog.e(TAG, "Telephony#isRadioInterfaceCapabilitySupported RemoteException" + ex);
ex.rethrowAsRuntimeException();
}
} }
return false; return false;
} }