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

Commit a5e1db3c authored by Jayachandran C's avatar Jayachandran C Committed by Jack Yu
Browse files

Check for attached state explicitly before triggering data stall recovery 

The current implementation only checks for isDataAllowed() which could
return true based on mAutoAttached flag even though GPRS is detached.
This CL adds additional check for mAttached flag to prevent data stall
recovery for such cases.

Test: Manually verified by changing preferred network type
      atest com.android.internal.telephony.dataconnection.DcTrackerTest
Bug: 127425834

Merged-In: I14f3978c37ff02f5016c08efbbb8dec96dde4715
Change-Id: I14f3978c37ff02f5016c08efbbb8dec96dde4715
(cherry picked from commit 2f3a07b3)
parent f82d0598

src/java/com/android/internal/telephony/dataconnection/DcTracker.java

+3 −6
Original line number Diff line number Diff line
@@ -4481,12 +4481,9 @@ public class DcTracker extends Handler { 
                if (VDBG_STALL) log("skip back to back data stall recovery");

                return false;

            }

            // Data is not allowed in current environment

            if (!isDataAllowed(null, null)) {

                log("skipped data stall recovery due to data is not allowd");

                return false;

            }

            return true;



            // Allow recovery if data is expected to work

            return mAttached.get() && isDataAllowed(null);

        }



        private void triggerRecovery() {