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

Commit fd7db44a authored by Sooraj Sasindran's avatar Sooraj Sasindran Committed by Robert Greenwalt
Browse files

Issue Finish command even if a transaction failed 

Was only sending FINISH to modems that hadn't errored.

bug: 19996651
Change-Id: Ib646be5c0807c0cf9f824e71bead2e192c550c6d
parent 3af34e15

src/java/com/android/internal/telephony/ProxyController.java

+8 −13
Original line number Diff line number Diff line
@@ -466,7 +466,6 @@ public class ProxyController { 
        // Issue FINISH

        synchronized(mSetRadioAccessFamilyStatus) {

            for (int i = 0; i < mProxyPhones.length; i++) {

                if (mSetRadioAccessFamilyStatus[i] != SET_RC_STATUS_FAIL) {

                logd("issueFinish: phoneId=" + i + " sessionId=" + sessionId

                        + " mTransactionFailed=" + mTransactionFailed);

                mRadioAccessFamilyStatusCounter++;
@@ -484,10 +483,6 @@ public class ProxyController { 
                    // At least one failed, mark them all failed.

                    mSetRadioAccessFamilyStatus[i] = SET_RC_STATUS_FAIL;

                }

                } else {

                    logd("issueFinish: Ignore already FAIL, Phone" + i + " sessionId=" + sessionId

                            + " mTransactionFailed =" + mTransactionFailed);

                }

            }

        }

    }