aboutsummaryrefslogtreecommitdiffstats
path: root/package/network/services
diff options
context:
space:
mode:
authorKevin Darbyshire-Bryant <kevin@darbyshire-bryant.me.uk>2016-09-05 09:01:19 +0100
committerFelix Fietkau <nbd@nbd.name>2016-09-06 11:26:05 +0200
commit9209f4304b0dd5e5a5337abd69a6dbf6a9cb782e (patch)
treec0242abae658b8df1491f74205e52a6be885d1bf /package/network/services
parentc5913264e7eb9f2bee782b41ab688d002f00d196 (diff)
downloadupstream-9209f4304b0dd5e5a5337abd69a6dbf6a9cb782e.tar.gz
upstream-9209f4304b0dd5e5a5337abd69a6dbf6a9cb782e.tar.bz2
upstream-9209f4304b0dd5e5a5337abd69a6dbf6a9cb782e.zip
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 <kevin@darbyshire-bryant.me.uk>
Diffstat (limited to 'package/network/services')
-rw-r--r--package/network/services/dnsmasq/files/dnsmasq.init1
1 files changed, 1 insertions, 0 deletions
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"