fix: ignore carrier config changes during early boot before syncing SIM cards

* During boot up process we get carrier config changes event for each
  SIM card once loaded. But, sometimes we get intermediate carrier
  config changes per SIM card during early boot since SIM cards are
  loaded into the system asynchronously. This forces us to synchronize
  all SIM cards again, and during this phase a SIM subscription present
  a moment ago may disappear again. Unfortunately, for our app this is
  SIM card removal, so we end up resetting the last activated /
  deactivated time fields, which are needed to properly initialize the
  user SIM card enabled state preference after a reboot.

* This patch aims to resolve this SIM card load race by ignoring all
  carrier config changes during early boot in order to allow all SIM
  subscriptions to fully settle up.

Signed-off-by: iusmac <iusico.maxim@libero.it>

Author: iusmac

src/com/github/iusmac/sevensim/DirectBootAwareBroadcastReceiver.java

@@ -10,8 +10,10 @@
 import dagger.hilt.android.AndroidEntryPoint;
 
 import java.time.LocalDateTime;
+import java.util.Optional;
 
 import javax.inject.Inject;
+import javax.inject.Named;
 
 /**
  * <p>This static broadcast receiver is marked as being {@link ComponentInfo#directBootAware}, and
@@ -25,6 +27,10 @@ public final class DirectBootAwareBroadcastReceiver extends Hilt_DirectBootAware
     @Inject
     Logger.Factory mLoggerFactory;
 
+    @Inject
+    @Named("LockedBootCompleted")
+    SysProp mLockedBootCompletedSysProp;
+
     private Logger mLogger;
 
     @Override
@@ -39,6 +45,8 @@ public void onReceive(final Context context, final Intent intent) {
         final String action = intent.getAction() != null ? intent.getAction() : "";
         switch (action) {
             case Intent.ACTION_LOCKED_BOOT_COMPLETED:
+                mLockedBootCompletedSysProp.set(Optional.of("1"));
+
                 // Need to sync the enabled state of all SIM subscriptions available on the device
                 // with their existing weekly repeat schedules after the device has finished booting
                 ForegroundService.syncAllSubscriptionsEnabledState(context, now,
@@ -52,6 +60,13 @@ public void onReceive(final Context context, final Intent intent) {
                 break;
 
             case CarrierConfigManager.ACTION_CARRIER_CONFIG_CHANGED:
+                // Ignore carrier config changes during early boot to allow all SIM subscriptions to
+                // fully settle up
+                if (!mLockedBootCompletedSysProp.isTrue()) {
+                    mLogger.d("onReceive() : Ignoring early boot carrier config changes.");
+                    break;
+                }
+
                 // Avoid rebroadcast after unlocking the device as we're direct boot aware
                 if (Utils.IS_AT_LEAST_R && intent.getBooleanExtra(CarrierConfigManager
                             .EXTRA_REBROADCAST_ON_UNLOCK, false)) {

src/com/github/iusmac/sevensim/inject/SevenSimModule.java

@@ -158,6 +158,13 @@ static DevicePolicyManager provideDevicePolicyManager(
         return ContextCompat.getSystemService(context, DevicePolicyManager.class);
     }
 
+    @Named("LockedBootCompleted")
+    @Singleton
+    @Provides
+    static SysProp provideLockedBootCompletedSysProp() {
+        return new SysProp("locked_boot_completed", /*isPersistent=*/ false);
+    }
+
     /** Do not initialize. */
     private SevenSimModule() {}
 }