X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=blobdiff_plain;f=src%2Flibsystemd-dhcp%2Fdhcp-client.c;h=102353894c08089e2675a1a85ba9c22237f95343;hp=7dc154612816ac3902070cce9bbecdba76efe5f4;hb=aba26854e5d47e36fd78a263b271d727d308f0dd;hpb=77e8d29dd2ecfa7f11c31c8d96021d323ab3fe3a diff --git a/src/libsystemd-dhcp/dhcp-client.c b/src/libsystemd-dhcp/dhcp-client.c index 7dc154612..102353894 100644 --- a/src/libsystemd-dhcp/dhcp-client.c +++ b/src/libsystemd-dhcp/dhcp-client.c @@ -22,6 +22,7 @@ #include #include #include +#include #include "util.h" #include "list.h" @@ -77,6 +78,9 @@ static const uint8_t default_req_opts[] = { DHCP_OPTION_NTP_SERVER, }; +static int client_receive_message(sd_event_source *s, int fd, + uint32_t revents, void *userdata); + int sd_dhcp_client_set_callback(sd_dhcp_client *client, sd_dhcp_client_cb_t cb, void *userdata) { @@ -324,6 +328,9 @@ static int client_packet_init(sd_dhcp_client *client, uint8_t type, refuse to issue an DHCP lease if 'secs' is set to zero */ message->secs = htobe16(secs); + if (client->state == DHCP_STATE_RENEWING) + message->ciaddr = client->lease->address; + memcpy(&message->chaddr, &client->mac_addr, ETH_ALEN); (*opt)[0] = 0x63; (*opt)[1] = 0x82; @@ -382,10 +389,13 @@ static uint16_t client_checksum(void *buf, int len) if (len & 0x01) { odd = buf; - sum += odd[len]; + sum += odd[len - 1]; } - return ~((sum & 0xffff) + (sum >> 16)); + while (sum >> 16) + sum = (sum & 0xffff) + (sum >> 16); + + return ~sum; } static void client_append_ip_headers(DHCPPacket *packet, uint16_t len) @@ -487,10 +497,17 @@ static int client_send_request(sd_dhcp_client *client, uint16_t secs) if (err < 0) return err; - client_append_ip_headers(request, len); + if (client->state == DHCP_STATE_RENEWING) { + err = dhcp_network_send_udp_socket(client->fd, + client->lease->server_address, + &request->dhcp, + len - DHCP_IP_UDP_SIZE); + } else { + client_append_ip_headers(request, len); - err = dhcp_network_send_raw_socket(client->fd, &client->link, - request, len); + err = dhcp_network_send_raw_socket(client->fd, &client->link, + request, len); + } return err; } @@ -499,17 +516,39 @@ static int client_timeout_resend(sd_event_source *s, uint64_t usec, void *userdata) { sd_dhcp_client *client = userdata; - usec_t next_timeout; + usec_t next_timeout = 0; + uint32_t time_left; uint16_t secs; int err = 0; - secs = (usec - client->start_time) / USEC_PER_SEC; + switch (client->state) { + case DHCP_STATE_RENEWING: + + time_left = (client->lease->t2 - client->lease->t1)/2; + if (time_left < 60) + time_left = 60; + + next_timeout = usec + time_left * USEC_PER_SEC; + + break; + + case DHCP_STATE_INIT: + case DHCP_STATE_INIT_REBOOT: + case DHCP_STATE_REBOOTING: + case DHCP_STATE_SELECTING: + case DHCP_STATE_REQUESTING: + case DHCP_STATE_BOUND: + case DHCP_STATE_REBINDING: - if (client->attempt < 64) - client->attempt *= 2; + if (client->attempt < 64) + client->attempt *= 2; + + next_timeout = usec + (client->attempt - 1) * USEC_PER_SEC; + + break; + } - next_timeout = usec + (client->attempt - 1) * USEC_PER_SEC + - (random_u() & 0x1fffff); + next_timeout += (random_u() & 0x1fffff); err = sd_event_add_monotonic(client->event, next_timeout, 10 * USEC_PER_MSEC, @@ -518,6 +557,8 @@ static int client_timeout_resend(sd_event_source *s, uint64_t usec, if (err < 0) goto error; + secs = (usec - client->start_time) / USEC_PER_SEC; + switch (client->state) { case DHCP_STATE_INIT: err = client_send_discover(client, secs); @@ -539,6 +580,7 @@ static int client_timeout_resend(sd_event_source *s, uint64_t usec, break; case DHCP_STATE_REQUESTING: + case DHCP_STATE_RENEWING: err = client_send_request(client, secs); if (err < 0 && client->attempt >= 64) goto error; @@ -550,7 +592,6 @@ static int client_timeout_resend(sd_event_source *s, uint64_t usec, case DHCP_STATE_INIT_REBOOT: case DHCP_STATE_REBOOTING: case DHCP_STATE_BOUND: - case DHCP_STATE_RENEWING: case DHCP_STATE_REBINDING: break; @@ -583,6 +624,35 @@ static int client_timeout_t2(sd_event_source *s, uint64_t usec, void *userdata) static int client_timeout_t1(sd_event_source *s, uint64_t usec, void *userdata) { + sd_dhcp_client *client = userdata; + int r; + + client->state = DHCP_STATE_RENEWING; + client->attempt = 1; + + r = dhcp_network_bind_udp_socket(client->index, + client->lease->address); + if (r < 0) + goto error; + + client->fd = r; + r = sd_event_add_io(client->event, client->fd, EPOLLIN, + client_receive_message, client, + &client->receive_message); + if (r < 0) + goto error; + + r = sd_event_add_monotonic(client->event, usec, 0, + client_timeout_resend, client, + &client->timeout_resend); + if (r < 0) + goto error; + + return 0; + +error: + client_stop(client, r); + return 0; } @@ -714,22 +784,32 @@ error: return -ENOMSG; } -static int client_receive_ack(sd_dhcp_client *client, DHCPPacket *offer, +static int client_receive_ack(sd_dhcp_client *client, const uint8_t *buf, size_t len) { int r; + DHCPPacket *ack; + DHCPMessage *dhcp; DHCPLease *lease; - r = client_verify_headers(client, offer, len); - if (r < 0) - return r; + if (client->state == DHCP_STATE_RENEWING) { + dhcp = (DHCPMessage *)buf; + } else { + ack = (DHCPPacket *)buf; + + r = client_verify_headers(client, ack, len); + if (r < 0) + return r; + + dhcp = &ack->dhcp; + len -= DHCP_IP_UDP_SIZE; + } lease = new0(DHCPLease, 1); if (!lease) return -ENOMEM; - len = len - DHCP_IP_UDP_SIZE; - r = dhcp_option_parse(&offer->dhcp, len, client_parse_offer, lease); + r = dhcp_option_parse(dhcp, len, client_parse_offer, lease); if (r == DHCP_NAK) { r = DHCP_EVENT_NO_LEASE; @@ -741,7 +821,7 @@ static int client_receive_ack(sd_dhcp_client *client, DHCPPacket *offer, goto error; } - lease->address = offer->dhcp.yiaddr; + lease->address = dhcp->yiaddr; if (lease->address == INADDR_ANY || lease->server_address == INADDR_ANY || @@ -786,6 +866,10 @@ static int client_set_lease_timeouts(sd_dhcp_client *client, uint64_t usec) if (client->lease->lifetime < 10) return -EINVAL; + client->timeout_t1 = sd_event_source_unref(client->timeout_t1); + client->timeout_t2 = sd_event_source_unref(client->timeout_t2); + client->timeout_expire = sd_event_source_unref(client->timeout_expire); + if (!client->lease->t1) client->lease->t1 = client->lease->lifetime / 2; @@ -837,13 +921,13 @@ static int client_set_lease_timeouts(sd_dhcp_client *client, uint64_t usec) return 0; } -static int client_receive_raw_message(sd_event_source *s, int fd, - uint32_t revents, void *userdata) +static int client_receive_message(sd_event_source *s, int fd, + uint32_t revents, void *userdata) { sd_dhcp_client *client = userdata; uint8_t buf[sizeof(DHCPPacket) + DHCP_CLIENT_MIN_OPTIONS_SIZE]; int buflen = sizeof(buf); - int len, r = 0; + int len, r = 0, notify_event = 0; DHCPPacket *message; usec_t time_now; @@ -855,11 +939,11 @@ static int client_receive_raw_message(sd_event_source *s, int fd, if (r < 0) goto error; - message = (DHCPPacket *)&buf; - switch (client->state) { case DHCP_STATE_SELECTING: + message = (DHCPPacket *)&buf; + if (client_receive_offer(client, message, len) >= 0) { client->timeout_resend = @@ -879,8 +963,10 @@ static int client_receive_raw_message(sd_event_source *s, int fd, break; case DHCP_STATE_REQUESTING: + case DHCP_STATE_RENEWING: + + r = client_receive_ack(client, buf, len); - r = client_receive_ack(client, message, len); if (r == DHCP_EVENT_NO_LEASE) goto error; @@ -888,16 +974,22 @@ static int client_receive_raw_message(sd_event_source *s, int fd, client->timeout_resend = sd_event_source_unref(client->timeout_resend); + if (client->state == DHCP_STATE_REQUESTING) + notify_event = DHCP_EVENT_IP_ACQUIRE; + else if (r != DHCP_EVENT_IP_ACQUIRE) + notify_event = r; + client->state = DHCP_STATE_BOUND; client->attempt = 1; client->last_addr = client->lease->address; r = client_set_lease_timeouts(client, time_now); - if (r < 0 ) + if (r < 0) goto error; - client_notify(client, DHCP_EVENT_IP_ACQUIRE); + if (notify_event) + client_notify(client, notify_event); client->receive_message = sd_event_source_unref(client->receive_message); @@ -913,7 +1005,6 @@ static int client_receive_raw_message(sd_event_source *s, int fd, case DHCP_STATE_INIT_REBOOT: case DHCP_STATE_REBOOTING: case DHCP_STATE_BOUND: - case DHCP_STATE_RENEWING: case DHCP_STATE_REBINDING: break; @@ -946,7 +1037,7 @@ int sd_dhcp_client_start(sd_dhcp_client *client) } err = sd_event_add_io(client->event, client->fd, EPOLLIN, - client_receive_raw_message, client, + client_receive_message, client, &client->receive_message); if (err < 0) goto error;