Commit 2f3a07b3 authored Apr 17, 2019 by Jayachandran C

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

Change-Id: I14f3978c37ff02f5016c08efbbb8dec96dde4715

parent 0ea02d7f

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

+3 −6

Original line number	Diff line number	Diff line
		@@ -4418,12 +4418,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() {