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

Commit 48f9c731 authored by Lorenzo Colitti's avatar Lorenzo Colitti
Browse files

Remove the IPv4 address from the interface when DHCP fails.

Bug: 19704592
Change-Id: I9b96cdcfc7c9442b3f8d6797aca63559ed4bdb9b
parent e380546a
Loading
Loading
Loading
Loading
+25 −12
Original line number Diff line number Diff line
@@ -165,6 +165,7 @@ public class DhcpClient extends BaseDhcpStateMachine {
    private State mDhcpInitState = new DhcpInitState();
    private State mDhcpSelectingState = new DhcpSelectingState();
    private State mDhcpRequestingState = new DhcpRequestingState();
    private State mDhcpHaveAddressState = new DhcpHaveAddressState();
    private State mDhcpBoundState = new DhcpBoundState();
    private State mDhcpRenewingState = new DhcpRenewingState();
    private State mDhcpRebindingState = new DhcpRebindingState();
@@ -186,10 +187,11 @@ public class DhcpClient extends BaseDhcpStateMachine {
            addState(mWaitBeforeStartState, mDhcpState);
            addState(mDhcpSelectingState, mDhcpState);
            addState(mDhcpRequestingState, mDhcpState);
            addState(mDhcpBoundState, mDhcpState);
            addState(mWaitBeforeRenewalState, mDhcpState);
            addState(mDhcpRenewingState, mDhcpState);
            addState(mDhcpRebindingState, mDhcpState);
            addState(mDhcpHaveAddressState, mDhcpState);
                addState(mDhcpBoundState, mDhcpHaveAddressState);
                addState(mWaitBeforeRenewalState, mDhcpHaveAddressState);
                addState(mDhcpRenewingState, mDhcpHaveAddressState);
                addState(mDhcpRebindingState, mDhcpHaveAddressState);
            addState(mDhcpInitRebootState, mDhcpState);
            addState(mDhcpRebootingState, mDhcpState);

@@ -402,7 +404,7 @@ public class DhcpClient extends BaseDhcpStateMachine {
        }
    }

    private void notifyLease() {
    private void notifySuccess() {
        mController.sendMessage(DhcpStateMachine.CMD_POST_DHCP_ACTION,
                DhcpStateMachine.DHCP_SUCCESS, 0, new DhcpResults(mDhcpLease));
    }
@@ -752,21 +754,32 @@ public class DhcpClient extends BaseDhcpStateMachine {
        }
    }

    class DhcpBoundState extends LoggingState {
    class DhcpHaveAddressState extends LoggingState {
        @Override
        public void enter() {
            super.enter();
            if (setIpAddress(mDhcpLease.ipAddress)) {
                notifyLease();
                // TODO: DhcpStateMachine only supports renewing at 50% of the lease time,
                // and does not support rebinding. Fix this.
                scheduleRenew();
            } else {
            if (!setIpAddress(mDhcpLease.ipAddress)) {
                notifyFailure();
                transitionTo(mStoppedState);
            }
        }

        @Override
        public void exit() {
            setIpAddress(new LinkAddress("0.0.0.0/0"));
        }
    }

    class DhcpBoundState extends LoggingState {
        @Override
        public void enter() {
            super.enter();
            notifySuccess();
            // TODO: DhcpStateMachine only supports renewing at 50% of the lease time, and does not
            // support rebinding. Fix this.
            scheduleRenew();
        }

        @Override
        public boolean processMessage(Message message) {
            super.processMessage(message);