commit 553d5dbd17df6ae96f878c71e9bd4fe593a47b44
author Gil Cukierman <cukie@google.com> Tue Mar 05 04:39:10 2024 -0500
committer Gil Cukierman <cukie@google.com> Tue Mar 05 04:39:10 2024 -0500
tree 29e16d064ee90d7df42ce7036a5dc5e0a84b05c8
parent d1a97399eba724b51e5bd67641f3570d58a3c153

Handle missing default phone

If default phones have not been created, make the receiver a no-op.

Test: atest SafetySourceReceiverTest
Bug: 325460602
Change-Id: Ic2108ccd3025a0986fa5b12067ad1906faa14193

src/com/android/phone/security/SafetySourceReceiver.java

diff --git a/src/com/android/phone/security/SafetySourceReceiver.java b/src/com/android/phone/security/SafetySourceReceiver.java
index d2be1a1..76f8e72 100644
--- a/src/com/android/phone/security/SafetySourceReceiver.java
+++ b/src/com/android/phone/security/SafetySourceReceiver.java
@@ -28,8 +28,10 @@
 import com.android.internal.telephony.Phone;
 import com.android.internal.telephony.flags.Flags;
 import com.android.phone.PhoneGlobals;
+import com.android.telephony.Rlog;
 
 public class SafetySourceReceiver extends BroadcastReceiver {
+    private static final String TAG = "TelephonySafetySourceReceiver";
     @Override
     public void onReceive(Context context, Intent intent) {
 
@@ -72,6 +74,11 @@
 
     @VisibleForTesting
     public Phone getDefaultPhone() {
-        return PhoneGlobals.getPhone();
+        try {
+            return PhoneGlobals.getPhone();
+        } catch (IllegalStateException e) {
+            Rlog.i(TAG, "Unable to get phone. Skipping safety source refresh: " + e.getMessage());
+        }
+        return null;
     }
 }