From: Kevin Darbyshire-Bryant Date: Mon, 5 Sep 2016 08:01:19 +0000 (+0100) Subject: dnsmasq: fix remove pidfile on shutdown regression X-Git-Url: http://git.cdn.openwrt.org/?a=commitdiff_plain;h=9209f4304b0dd5e5a5337abd69a6dbf6a9cb782e;p=openwrt%2Fstaging%2Fblocktrron.git dnsmasq: fix remove pidfile on shutdown regression Regression introduced by 3481d0d dnsmasq: run as dedicated UID/GID dnsmasq is unable to remove its own pidfile as /var/run/dnsmasq is owned by root and now dnsmasq runs as dnsmasq:dnsmasq. Change directory ownership to match. dnsmasq initially starts as root, creates the pidfile, then drops to requested non-root user. Until this fix dnsmasq had insufficient privilege to remove its own pidfile. Signed-off-by: Kevin Darbyshire-Bryant --- diff --git a/package/network/services/dnsmasq/files/dnsmasq.init b/package/network/services/dnsmasq/files/dnsmasq.init index 2b81a61bcc..4cd5f673d8 100644 --- a/package/network/services/dnsmasq/files/dnsmasq.init +++ b/package/network/services/dnsmasq/files/dnsmasq.init @@ -643,6 +643,7 @@ start_service() { mkdir -p $(dirname $CONFIGFILE) mkdir -p /var/lib/misc touch /tmp/dhcp.leases + chown dnsmasq:dnsmasq /var/run/dnsmasq [ -f "$TIMESTAMPFILE" ] && rm -f "$TIMESTAMPFILE"