From 2ccf88744ccc7a861200f5cdbd0a703cda63a7b3 Mon Sep 17 00:00:00 2001 From: Steven Barth Date: Mon, 22 Sep 2014 12:07:20 +0000 Subject: [PATCH] dnsmasq: fix lockup when interfaces disappear SVN-Revision: 42648 --- package/network/services/dnsmasq/Makefile | 2 +- ...nsiveness-on-interface-disappearance.patch | 47 +++++++++++++++++++ 2 files changed, 48 insertions(+), 1 deletion(-) create mode 100644 package/network/services/dnsmasq/patches/003-fix-unresponsiveness-on-interface-disappearance.patch diff --git a/package/network/services/dnsmasq/Makefile b/package/network/services/dnsmasq/Makefile index 17af3c5d8cb..6262dc5475b 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.71 -PKG_RELEASE:=4 +PKG_RELEASE:=5 PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.gz PKG_SOURCE_URL:=http://thekelleys.org.uk/dnsmasq diff --git a/package/network/services/dnsmasq/patches/003-fix-unresponsiveness-on-interface-disappearance.patch b/package/network/services/dnsmasq/patches/003-fix-unresponsiveness-on-interface-disappearance.patch new file mode 100644 index 00000000000..5ee2090d2c5 --- /dev/null +++ b/package/network/services/dnsmasq/patches/003-fix-unresponsiveness-on-interface-disappearance.patch @@ -0,0 +1,47 @@ +From 5782649ad95382dd558df97b33b64e854d8789fb Mon Sep 17 00:00:00 2001 +From: Simon Kelley +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 +