summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBeniamino Galvani <bgalvani@redhat.com>2020-01-27 21:44:36 +0100
committerBeniamino Galvani <bgalvani@redhat.com>2020-01-30 15:23:04 +0100
commit182a8021f39c943af87d2bfc703d09dbc7e25bf7 (patch)
tree6523f780ebb542f47b9e8bed5a69ccf11fde1213
parentaee78ca788a6df66166505c6f7342655773409cc (diff)
downloadNetworkManager-182a8021f39c943af87d2bfc703d09dbc7e25bf7.tar.gz
n-dhcp4: move back to INIT after lease expires
Move back to INIT state after the lease expires, as per section 4.4.5 of RFC 2131. Previously the client just moved to EXPIRED, closed the connection and cleared the probe, leaving to the caller of the library the choice to create a new client instance and to start from scratch. However, it seems more useful that the client, once initialized, always tries to get a lease even after an expiration.
-rw-r--r--shared/n-dhcp4/src/n-dhcp4-c-probe.c7
1 files changed, 3 insertions, 4 deletions
diff --git a/shared/n-dhcp4/src/n-dhcp4-c-probe.c b/shared/n-dhcp4/src/n-dhcp4-c-probe.c
index f4962d6995..bfead036bd 100644
--- a/shared/n-dhcp4/src/n-dhcp4-c-probe.c
+++ b/shared/n-dhcp4/src/n-dhcp4-c-probe.c
@@ -841,11 +841,10 @@ static int n_dhcp4_client_probe_transition_lifetime(NDhcp4ClientProbe *probe) {
return r;
c_assert(probe->client->current_probe == probe);
- probe->client->current_probe = NULL;
-
- n_dhcp4_c_connection_close(&probe->connection);
+ probe->current_lease = n_dhcp4_client_lease_unref(probe->current_lease);
- probe->state = N_DHCP4_CLIENT_PROBE_STATE_EXPIRED;
+ probe->state = N_DHCP4_CLIENT_PROBE_STATE_INIT;
+ probe->ns_deferred = n_dhcp4_gettime(CLOCK_BOOTTIME) + UINT64_C(1);
break;