diff options
| author | Kevin Darbyshire-Bryant <kevin@darbyshire-bryant.me.uk> | 2016-09-05 09:01:19 +0100 |
|---|---|---|
| committer | Felix Fietkau <nbd@nbd.name> | 2016-09-06 11:26:05 +0200 |
| commit | 9209f4304b0dd5e5a5337abd69a6dbf6a9cb782e (patch) | |
| tree | c0242abae658b8df1491f74205e52a6be885d1bf /package/network/services | |
| parent | c5913264e7eb9f2bee782b41ab688d002f00d196 (diff) | |
| download | upstream-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.init | 1 |
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" |