Author: cyrus
Date: 2014-09-22 14:07:20 +0200 (Mon, 22 Sep 2014)
New Revision: 42648

Added:
   
trunk/package/network/services/dnsmasq/patches/003-fix-unresponsiveness-on-interface-disappearance.patch
Modified:
   trunk/package/network/services/dnsmasq/Makefile
Log:
dnsmasq: fix lockup when interfaces disappear

Modified: trunk/package/network/services/dnsmasq/Makefile
===================================================================
--- trunk/package/network/services/dnsmasq/Makefile     2014-09-22 11:36:38 UTC 
(rev 42647)
+++ trunk/package/network/services/dnsmasq/Makefile     2014-09-22 12:07:20 UTC 
(rev 42648)
@@ -9,7 +9,7 @@
 
 PKG_NAME:=dnsmasq
 PKG_VERSION:=2.71
-PKG_RELEASE:=4
+PKG_RELEASE:=5
 
 PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.gz
 PKG_SOURCE_URL:=http://thekelleys.org.uk/dnsmasq

Added: 
trunk/package/network/services/dnsmasq/patches/003-fix-unresponsiveness-on-interface-disappearance.patch
===================================================================
--- 
trunk/package/network/services/dnsmasq/patches/003-fix-unresponsiveness-on-interface-disappearance.patch
                            (rev 0)
+++ 
trunk/package/network/services/dnsmasq/patches/003-fix-unresponsiveness-on-interface-disappearance.patch
    2014-09-22 12:07:20 UTC (rev 42648)
@@ -0,0 +1,47 @@
+From 5782649ad95382dd558df97b33b64e854d8789fb Mon Sep 17 00:00:00 2001
+From: Simon Kelley <[email protected]>
+Date: Thu, 18 Sep 2014 22:08:58 +0100
+Subject: [PATCH] Fix bug which caused dnsmasq to become unresponsive when an
+ interface goes.
+
+---
+
+diff --git a/src/util.c b/src/util.c
+index df751c7..a729f33 100644
+--- a/src/util.c
++++ b/src/util.c
+@@ -570,18 +570,28 @@ void bump_maxfd(int fd, int *max)
+ 
+ int retry_send(void)
+ {
+-   struct timespec waiter;
++  /* Linux kernels can return EAGAIN in perpetuity when calling
++     sendmsg() and the relevant interface has gone. Here we loop
++     retrying in EAGAIN for 1 second max, to avoid this hanging 
++     dnsmasq. */
++
++  static int retries = 0;
++  struct timespec waiter;
++
+    if (errno == EAGAIN || errno == EWOULDBLOCK)
+      {
+        waiter.tv_sec = 0;
+        waiter.tv_nsec = 10000;
+        nanosleep(&waiter, NULL);
+-       return 1;
++       if (retries++ < 1000)
++       return 1;
+      }
++
++   retries = 0;
+    
+    if (errno == EINTR)
+      return 1;
+-
++   
+    return 0;
+ }
+ 
+-- 
+1.9.1
+
_______________________________________________
openwrt-commits mailing list
[email protected]
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-commits

Reply via email to