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

Commit 8ecdb4af authored by Aishwarya Mallampati's avatar Aishwarya Mallampati
Browse files

Return false if radio is off in isCarrierRoamingNtnEligible

Bug: 376273308
Test: atest SatelliteControllerTest
Test: Manually tested SMS/MMS/CALLS/DATA
FLAG: EXEMPT bugfix

Change-Id: I140b474c08f9735b73aae398ab296635a5bbf058
parent a12e4302
Loading
Loading
Loading
Loading
+6 −1
Original line number Diff line number Diff line
@@ -4294,7 +4294,7 @@ public class SatelliteController extends Handler {
        RequestSatelliteEnabledArgument argument =
                (RequestSatelliteEnabledArgument) request.argument;
        handlePersistentLoggingOnSessionStart(argument);
        selectBindingSatelliteSubscription(true);
        selectBindingSatelliteSubscription(argument.enableSatellite);
        SatelliteModemEnableRequestAttributes enableRequestAttributes =
                    createModemEnableRequest(argument);
        if (enableRequestAttributes == null) {
@@ -7140,6 +7140,11 @@ public class SatelliteController extends Handler {
            return false;
        }

        if (!mIsRadioOn) {
            plogd("isCarrierRoamingNtnEligible: radio is off");
            return false;
        }

        if (phone == null) {
            plogd("isCarrierRoamingNtnEligible: phone is null");
            return false;