mirror of
https://github.com/openwrt/openwrt.git
synced 2024-12-26 00:41:17 +00:00
495b237cc4
* Add Authoritative DNS and IPSET to full variant * Remove some bloat from IPSET support * Reintroduce "DHCP no address warning"-patch Signed-off-by: Steven Barth <steven@midlink.org> SVN-Revision: 41246
48 lines
1.7 KiB
Diff
48 lines
1.7 KiB
Diff
--- a/src/dhcp.c
|
|
+++ b/src/dhcp.c
|
|
@@ -146,7 +146,7 @@ void dhcp_packet(time_t now, int pxe_fd)
|
|
struct iovec iov;
|
|
ssize_t sz;
|
|
int iface_index = 0, unicast_dest = 0, is_inform = 0;
|
|
- struct in_addr iface_addr;
|
|
+ struct in_addr iface_addr, *addrp = NULL;
|
|
struct iface_param parm;
|
|
#ifdef HAVE_LINUX_NETWORK
|
|
struct arpreq arp_req;
|
|
@@ -272,11 +272,9 @@ void dhcp_packet(time_t now, int pxe_fd)
|
|
{
|
|
ifr.ifr_addr.sa_family = AF_INET;
|
|
if (ioctl(daemon->dhcpfd, SIOCGIFADDR, &ifr) != -1 )
|
|
- iface_addr = ((struct sockaddr_in *) &ifr.ifr_addr)->sin_addr;
|
|
- else
|
|
{
|
|
- my_syslog(MS_DHCP | LOG_WARNING, _("DHCP packet received on %s which has no address"), ifr.ifr_name);
|
|
- return;
|
|
+ addrp = &iface_addr;
|
|
+ iface_addr = ((struct sockaddr_in *) &ifr.ifr_addr)->sin_addr;
|
|
}
|
|
|
|
for (tmp = daemon->dhcp_except; tmp; tmp = tmp->next)
|
|
@@ -295,7 +293,7 @@ void dhcp_packet(time_t now, int pxe_fd)
|
|
parm.relay_local.s_addr = 0;
|
|
parm.ind = iface_index;
|
|
|
|
- if (!iface_check(AF_INET, (struct all_addr *)&iface_addr, ifr.ifr_name, NULL))
|
|
+ if (!iface_check(AF_INET, (struct all_addr *)addrp, ifr.ifr_name, NULL))
|
|
{
|
|
/* If we failed to match the primary address of the interface, see if we've got a --listen-address
|
|
for a secondary */
|
|
@@ -315,6 +313,12 @@ void dhcp_packet(time_t now, int pxe_fd)
|
|
complete_context(match.addr, iface_index, NULL, match.netmask, match.broadcast, &parm);
|
|
}
|
|
|
|
+ if (!addrp)
|
|
+ {
|
|
+ my_syslog(MS_DHCP | LOG_WARNING, _("DHCP packet received on %s which has no address"), ifr.ifr_name);
|
|
+ return;
|
|
+ }
|
|
+
|
|
if (!iface_enumerate(AF_INET, &parm, complete_context))
|
|
return;
|
|
|