[source] dnsmasq: backport tweak ICMP ping logic for DHCPv4

LEDE Commits lede-commits at lists.infradead.org
Mon Jun 26 01:58:00 PDT 2017


dedeckeh pushed a commit to source.git, branch master:
https://git.lede-project.org/f33de8023228d7d6545ff12c4f2ef8febb486f37

commit f33de8023228d7d6545ff12c4f2ef8febb486f37
Author: Hans Dedecker <dedeckeh at gmail.com>
AuthorDate: Mon Jun 26 10:23:08 2017 +0200

    dnsmasq: backport tweak ICMP ping logic for DHCPv4
    
    Don't start ping-check of address in DHCP discover if there already
    exists a lease for the address. It has been reported under some
    circumstances android and netbooted windows devices can reply to
    ICMP pings if they have a lease and thus block the allocation of
    the IP address the device already has during boot.
    
    Signed-off-by: Hans Dedecker <dedeckeh at gmail.com>
---
 package/network/services/dnsmasq/Makefile          |  2 +-
 ...10-Tweak-ICMP-ping-check-logic-for-DHCPv4.patch | 25 ++++++++++++++++++++++
 2 files changed, 26 insertions(+), 1 deletion(-)

diff --git a/package/network/services/dnsmasq/Makefile b/package/network/services/dnsmasq/Makefile
index 35ac6b2..11baff3 100644
--- a/package/network/services/dnsmasq/Makefile
+++ b/package/network/services/dnsmasq/Makefile
@@ -9,7 +9,7 @@ include $(TOPDIR)/rules.mk
 
 PKG_NAME:=dnsmasq
 PKG_VERSION:=2.77
-PKG_RELEASE:=4
+PKG_RELEASE:=5
 
 PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.xz
 PKG_SOURCE_URL:=http://thekelleys.org.uk/dnsmasq/
diff --git a/package/network/services/dnsmasq/patches/010-Tweak-ICMP-ping-check-logic-for-DHCPv4.patch b/package/network/services/dnsmasq/patches/010-Tweak-ICMP-ping-check-logic-for-DHCPv4.patch
new file mode 100644
index 0000000..571ff36
--- /dev/null
+++ b/package/network/services/dnsmasq/patches/010-Tweak-ICMP-ping-check-logic-for-DHCPv4.patch
@@ -0,0 +1,25 @@
+From 4bb68866a8aeb31db8100492bceae051e33be5d0 Mon Sep 17 00:00:00 2001
+From: Simon Kelley <simon at thekelleys.org.uk>
+Date: Thu, 15 Jun 2017 23:18:44 +0100
+Subject: [PATCH] Tweak ICMP ping check logic for DHCPv4.
+
+---
+ src/rfc2131.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/rfc2131.c b/src/rfc2131.c
+index 1c850e5..75792da 100644
+--- a/src/rfc2131.c
++++ b/src/rfc2131.c
+@@ -1040,7 +1040,7 @@ size_t dhcp_reply(struct dhcp_context *context, char *iface_name, int int_index,
+ 		  else if (have_config(config, CONFIG_DECLINED) &&
+ 			   difftime(now, config->decline_time) < (float)DECLINE_BACKOFF)
+ 		    my_syslog(MS_DHCP | LOG_WARNING, _("not using configured address %s because it was previously declined"), addrs);
+-		  else if (!do_icmp_ping(now, config->addr, 0, loopback))
++		  else if ((!lease || lease->addr.s_addr != config->addr.s_addr) && !do_icmp_ping(now, config->addr, 0, loopback))
+ 		    my_syslog(MS_DHCP | LOG_WARNING, _("not using configured address %s because it is in use by another host"), addrs);
+ 		  else
+ 		    conf = config->addr;
+-- 
+1.9.1
+



More information about the lede-commits mailing list