From 9209f4304b0dd5e5a5337abd69a6dbf6a9cb782e Mon Sep 17 00:00:00 2001 From: Kevin Darbyshire-Bryant Date: Mon, 5 Sep 2016 09:01:19 +0100 Subject: [PATCH] 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 --- package/network/services/dnsmasq/files/dnsmasq.init | 1 + 1 file changed, 1 insertion(+) 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" -- 2.30.2