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

Commit 4a3db57b authored by Sarah Chin's avatar Sarah Chin
Browse files

Remove checks for system_process in TelephonyManager 

Instead of throwing exceptions, return the default null/empty/void
behavior.

Test: atest TelephonyManagerTest
Fix: 175881713
Change-Id: Ia15c8183212bc67e74b45fc119b5ef9d4c295c94
Merged-In: Ia15c8183212bc67e74b45fc119b5ef9d4c295c94
parent d6eba767

telephony/java/android/telephony/TelephonyManager.java

+13 −53
Original line number Diff line number Diff line
@@ -59,7 +59,6 @@ import android.os.Handler; 
import android.os.IBinder;

import android.os.ParcelFileDescriptor;

import android.os.PersistableBundle;

import android.os.Process;

import android.os.RemoteException;

import android.os.ResultReceiver;

import android.os.SystemProperties;
@@ -414,10 +413,6 @@ public class TelephonyManager { 
        return null;

    }













    private boolean isSystemProcess() {













        return Process.myUid() == Process.SYSTEM_UID;













    }



























    /**















     * Post a runnable to the BackgroundThread.















     *










@@ -4196,19 +4191,12 @@ public class TelephonyManager {













        try {















            IPhoneSubInfo info = getSubscriberInfoService();















            if (info == null) {













                Rlog.e(TAG, "IMSI error: Subscriber Info is null");













                if (!isSystemProcess()) {















                throw new RuntimeException("IMSI error: Subscriber Info is null");















            }













                return;













            }















            int subId = getSubId(SubscriptionManager.getDefaultDataSubscriptionId());















            info.resetCarrierKeysForImsiEncryption(subId, mContext.getOpPackageName());















        } catch (RemoteException ex) {













            Rlog.e(TAG, "getCarrierInfoForImsiEncryption RemoteException" + ex);













            if (!isSystemProcess()) {













                ex.rethrowAsRuntimeException();













            }













            Rlog.e(TAG, "Telephony#getCarrierInfoForImsiEncryption RemoteException" + ex);















        }















    }
























@@ -5247,17 +5235,11 @@ public class TelephonyManager {













        try {















            final ITelephony telephony = getITelephony();















            if (telephony == null) {













                if (!isSystemProcess()) {













                    throw new RuntimeException("Telephony service unavailable");













                }















                return;















            }















            telephony.sendDialerSpecialCode(mContext.getOpPackageName(), inputCode);















        } catch (RemoteException ex) {













            // This could happen if binder process crashes.













            if (!isSystemProcess()) {













                ex.rethrowAsRuntimeException();













            }













            Rlog.e(TAG, "Telephony#sendDialerSpecialCode RemoteException" + ex);















        }















    }
























@@ -9534,9 +9516,7 @@ public class TelephonyManager {













                throw new IllegalStateException("telephony service is null.");















            }















        } catch (RemoteException ex) {













            if (!isSystemProcess()) {













                ex.rethrowAsRuntimeException();













            }













            Rlog.e(TAG, "Telephony#getMobileProvisioningUrl RemoteException" + ex);















        }















        return null;















    }










@@ -13371,9 +13351,7 @@ public class TelephonyManager {













                return service.isDataEnabledForApn(apnType, getSubId(), pkgForDebug);















            }















        } catch (RemoteException ex) {













            if (!isSystemProcess()) {













                ex.rethrowAsRuntimeException();













            }













            Rlog.e(TAG, "Telephony#isDataEnabledForApn RemoteException" + ex);















        }















        return false;















    }









@@ -13393,9 +13371,7 @@ public class TelephonyManager {













                return service.isApnMetered(apnType, getSubId());















            }















        } catch (RemoteException ex) {













            if (!isSystemProcess()) {













                ex.rethrowAsRuntimeException();













            }













            Rlog.e(TAG, "Telephony#isApnMetered RemoteException" + ex);















        }















        return true;















    }










@@ -13455,9 +13431,7 @@ public class TelephonyManager {













                service.setSystemSelectionChannels(specifiers, getSubId(), aidlConsumer);















            }















        } catch (RemoteException ex) {













            if (!isSystemProcess()) {













                ex.rethrowAsRuntimeException();













            }













            Rlog.e(TAG, "Telephony#setSystemSelectionChannels RemoteException" + ex);















        }















    }
























@@ -13485,9 +13459,7 @@ public class TelephonyManager {













                throw new IllegalStateException("telephony service is null.");















            }















        } catch (RemoteException ex) {













            if (!isSystemProcess()) {













                ex.rethrowAsRuntimeException();













            }













            Rlog.e(TAG, "Telephony#getSystemSelectionChannels RemoteException" + ex);















        }















        return new ArrayList<>();















    }










@@ -13516,9 +13488,7 @@ public class TelephonyManager {













                return service.isMvnoMatched(getSubId(), mvnoType, mvnoMatchData);















            }















        } catch (RemoteException ex) {













            if (!isSystemProcess()) {













                ex.rethrowAsRuntimeException();













            }













            Rlog.e(TAG, "Telephony#matchesCurrentSimOperator RemoteException" + ex);















        }















        return false;















    }










@@ -13922,10 +13892,7 @@ public class TelephonyManager {













                service.setMobileDataPolicyEnabledStatus(getSubId(), policy, enabled);















            }















        } catch (RemoteException ex) {













            // This could happen if binder process crashes.













            if (!isSystemProcess()) {













                ex.rethrowAsRuntimeException();













            }













            Rlog.e(TAG, "Telephony#setMobileDataPolicyEnabled RemoteException" + ex);















        }















    }























@@ -13946,10 +13913,7 @@ public class TelephonyManager {













                return service.isMobileDataPolicyEnabled(getSubId(), policy);















            }















        } catch (RemoteException ex) {













            // This could happen if binder process crashes.













            if (!isSystemProcess()) {













                ex.rethrowAsRuntimeException();













            }













            Rlog.e(TAG, "Telephony#isMobileDataPolicyEnabled RemoteException" + ex);















        }















        return false;















    }










@@ -14453,9 +14417,7 @@ public class TelephonyManager {













                throw new IllegalStateException("telephony service is null.");















            }















        } catch (RemoteException ex) {













            if (!isSystemProcess()) {













                ex.rethrowAsRuntimeException();













            }













            Rlog.e(TAG, "Telephony#getEquivalentHomePlmns RemoteException" + ex);















        }





























        return Collections.emptyList();










@@ -14549,9 +14511,7 @@ public class TelephonyManager {













                throw new IllegalStateException("telephony service is null.");















            }















        } catch (RemoteException ex) {













            if (!isSystemProcess()) {













                ex.rethrowAsRuntimeException();













            }













            Rlog.e(TAG, "Telephony#isRadioInterfaceCapabilitySupported RemoteException" + ex);















        }















        return false;















    }