Commit 1649f709 authored by Simon Kelley's avatar Simon Kelley

Fix DHCP relay, broken by ff325644

Thanks to John Fitzgibbon for initial patch.
parent 50ca8550
version 2.78
Fix logic of appending ".<layer>" to PXE basename. Thanks
to Chris Novakovic for the patch.
Revert ping-check of address in DHCPDISCOVER if there
already exists a lease for the address. Under some
circumstances, and netbooted windows installation can reply
to pings before if has a DHCP lease and block allocation
of the address it already used during netboot. Thanks to
Jan Psota for spotting this.
Fix DHCP relaying, broken in 2.76 and 2.77 by commit
ff325644c7afae2588583f935f4ea9b9694eb52e. Thanks to
John Fitzgibbon for the diagnosis and patch.
version 2.77 version 2.77
Generate an error when configured with a CNAME loop, Generate an error when configured with a CNAME loop,
rather than a crash. Thanks to George Metz for rather than a crash. Thanks to George Metz for
......
...@@ -273,8 +273,8 @@ void dhcp_packet(time_t now, int pxe_fd) ...@@ -273,8 +273,8 @@ void dhcp_packet(time_t now, int pxe_fd)
if ((relay = relay_reply4((struct dhcp_packet *)daemon->dhcp_packet.iov_base, ifr.ifr_name))) if ((relay = relay_reply4((struct dhcp_packet *)daemon->dhcp_packet.iov_base, ifr.ifr_name)))
{ {
/* Reply from server, using us as relay. */ /* Reply from server, using us as relay. */
iface_index = relay->iface_index; rcvd_iface_index = relay->iface_index;
if (!indextoname(daemon->dhcpfd, iface_index, ifr.ifr_name)) if (!indextoname(daemon->dhcpfd, rcvd_iface_index, ifr.ifr_name))
return; return;
is_relay_reply = 1; is_relay_reply = 1;
iov.iov_len = sz; iov.iov_len = sz;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment